We should have FirstMeaningfulPaint UMAs for ServiceWorkers |
|||||
Issue descriptionCurrently we have FirstContentfulPaint UMAs for ServiceWorkers recorded by ServiceWorkerPageLoadMetricsObserver. But we don't have FirstMeaningfulPaint UMAs yet.
,
May 31 2017
,
Jun 8 2017
,
Jun 8 2017
Your change meets the bar and is auto-approved for M60. Please go ahead and merge the CL to branch 3112 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e09a471170947024596d2b490b4d3c43165c4bf7 commit e09a471170947024596d2b490b4d3c43165c4bf7 Author: Tsuyoshi Horo <horo@chromium.org> Date: Thu Jun 08 02:12:32 2017 [Merge to M60] Add UMA for ServiceWorker FirstMeaningfulPaint BUG= 727599 Review-Url: https://codereview.chromium.org/2908323002 Cr-Original-Commit-Position: refs/heads/master@{#475760} Review-Url: https://codereview.chromium.org/2931753002 . Cr-Commit-Position: refs/branch-heads/3112@{#246} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} [modify] https://crrev.com/e09a471170947024596d2b490b4d3c43165c4bf7/chrome/browser/page_load_metrics/observers/service_worker_page_load_metrics_observer.cc [modify] https://crrev.com/e09a471170947024596d2b490b4d3c43165c4bf7/chrome/browser/page_load_metrics/observers/service_worker_page_load_metrics_observer.h [modify] https://crrev.com/e09a471170947024596d2b490b4d3c43165c4bf7/chrome/browser/page_load_metrics/observers/service_worker_page_load_metrics_observer_unittest.cc [modify] https://crrev.com/e09a471170947024596d2b490b4d3c43165c4bf7/tools/metrics/histograms/histograms.xml |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by bugdroid1@chromium.org
, May 31 2017