Change matching of filename - db entry to use maps |
||
Issue descriptionMore context at: https://chromium-review.googlesource.com/c/chromium/src/+/770430/1/components/offline_pages/core/model/persistent_pages_consistency_check_task.cc#139 Currently we're using a M*N approach, and we can reduce the time cost by applying a map there. Should be changing this for all consistency tasks: TemporaryPages* PersistentPages* [Maybe]ClearLegacy*.
,
Dec 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c3ce5d5452593ec5b6d1156d8e801cb813f92e1e commit c3ce5d5452593ec5b6d1156d8e801cb813f92e1e Author: Yafei Duan <romax@chromium.org> Date: Mon Dec 04 23:56:12 2017 [Offline Pages] Minor improvement on consistency check. Use set::find instead of std::find_if when looking for match of file paths in order to make the process faster Bug: 786240 Change-Id: I84dfdb7eb910bc120671520c6aca9b52a4971c7a Reviewed-on: https://chromium-review.googlesource.com/802520 Commit-Queue: Yafei Duan <romax@chromium.org> Reviewed-by: Peter Williamson <petewil@chromium.org> Cr-Commit-Position: refs/heads/master@{#521534} [modify] https://crrev.com/c3ce5d5452593ec5b6d1156d8e801cb813f92e1e/components/offline_pages/core/model/persistent_pages_consistency_check_task.cc [modify] https://crrev.com/c3ce5d5452593ec5b6d1156d8e801cb813f92e1e/components/offline_pages/core/model/temporary_pages_consistency_check_task.cc
,
Dec 4 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by romax@chromium.org
, Dec 4 2017Status: Started (was: Available)