commit | 6378752b810e6122744cefac544c8d84278dcc71 | [log] [tgz] |
---|---|---|
author | Charles Zhao <charleszhao@chromium.org> | Fri Dec 07 06:26:07 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Dec 07 06:26:07 2018 |
tree | e240e61004dbaadfd80fc27769e4f244d9362260 | |
parent | 28d424ea311f180535b9a5aceb6facfc61c9b59e [diff] |
TabRanker: Fix Query time logging. TabRanker query time logging was landed in this cl: https://chromium-review.googlesource.com/c/chromium/src/+/1350445 TabManager::GetSortedLifecycleUnitsFromTabRanker() was created to bypass calls to TabManager::GetSortedLifecycleUnits() But this was wrong on two aspects: (1) TabManagerDelegate::LowMemoryKillImpl is still calling GetSortedLifecycleUnits(), so ChromeOS discards are all skipped. (2) TabManagerDelegate::Candidate is calling lifecycle_unit_->GetSortKey() for inferring sorting, Which means TabRanker model won't work for ChromeOS discards. The fix: (1) Change TabManager, TabLifecycleUnit back to their previous code, which were sorting LifecycleUnits by TabRanker when it's enabled. (2) Add TabActivityWatcher::LogOldestNTabFeatures() and call this function in TabManager::DiscardTab for query time logging. Bug: 911474 Change-Id: I103e8237cd96ea87667536b7b44b27fe8772abb4 Reviewed-on: https://chromium-review.googlesource.com/c/1360471 Reviewed-by: Christopher Morin <cmtm@google.com> Reviewed-by: Kentaro Hara <haraken@chromium.org> Reviewed-by: Michael Giuffrida <michaelpg@chromium.org> Commit-Queue: Charles . <charleszhao@chromium.org> Cr-Commit-Position: refs/heads/master@{#614623}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .