Media Engagement Preload: measure load time and lookup time for DB |
|||||||
Issue descriptionDB load time and lookup time can have an impact on Chrome performances. We need to measure these in order to have an idea of the impact and look for improvements.
,
Jan 18 2018
,
Jan 19 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c7d8bcf2e7e27dc311efbc3ed997c9172a4a450d commit c7d8bcf2e7e27dc311efbc3ed997c9172a4a450d Author: Mounir Lamouri <mlamouri@chromium.org> Date: Fri Jan 19 10:34:30 2018 Media Engagement: add metrics for time to load and time to lookup. Bug: 800758 Change-Id: Iea3614bd17563ef0106be267e5f597a79c23bec0 Reviewed-on: https://chromium-review.googlesource.com/874690 Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Reviewed-by: Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#530477} [modify] https://crrev.com/c7d8bcf2e7e27dc311efbc3ed997c9172a4a450d/chrome/browser/media/media_engagement_preloaded_list.cc [modify] https://crrev.com/c7d8bcf2e7e27dc311efbc3ed997c9172a4a450d/chrome/browser/media/media_engagement_preloaded_list.h [modify] https://crrev.com/c7d8bcf2e7e27dc311efbc3ed997c9172a4a450d/chrome/browser/media/media_engagement_preloaded_list_unittest.cc [modify] https://crrev.com/c7d8bcf2e7e27dc311efbc3ed997c9172a4a450d/tools/metrics/histograms/histograms.xml
,
Jan 19 2018
,
Jan 22 2018
Requesting merge to M65 as it's only metrics and we would like to be able to catch perf regressions early.
,
Jan 23 2018
Your change meets the bar and is auto-approved for M65. Please go ahead and merge the CL to branch 3325 manually. Please contact milestone owner if you have questions. Owners: cmasso@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 23 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2200cdd0825947a0823092ca0d7cca782900669d commit 2200cdd0825947a0823092ca0d7cca782900669d Author: Mounir Lamouri <mlamouri@chromium.org> Date: Tue Jan 23 10:46:21 2018 Media Engagement: add metrics for time to load and time to lookup. Bug: 800758 Change-Id: Iea3614bd17563ef0106be267e5f597a79c23bec0 Reviewed-on: https://chromium-review.googlesource.com/874690 Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Reviewed-by: Ilya Sherman <isherman@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#530477}(cherry picked from commit c7d8bcf2e7e27dc311efbc3ed997c9172a4a450d) Reviewed-on: https://chromium-review.googlesource.com/880702 Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/branch-heads/3325@{#26} Cr-Branched-From: bc084a8b5afa3744a74927344e304c02ae54189f-refs/heads/master@{#530369} [modify] https://crrev.com/2200cdd0825947a0823092ca0d7cca782900669d/chrome/browser/media/media_engagement_preloaded_list.cc [modify] https://crrev.com/2200cdd0825947a0823092ca0d7cca782900669d/chrome/browser/media/media_engagement_preloaded_list.h [modify] https://crrev.com/2200cdd0825947a0823092ca0d7cca782900669d/chrome/browser/media/media_engagement_preloaded_list_unittest.cc [modify] https://crrev.com/2200cdd0825947a0823092ca0d7cca782900669d/tools/metrics/histograms/histograms.xml
,
Jan 23 2018
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by mlamouri@chromium.org
, Jan 18 2018Labels: -OS-Android