Add UMA to record loading failed error codes and final per-request status codes |
||||
Issue descriptionIn order to get better understanding of our higher LOADING_FAILED and SAVED UMA differences, we should record what's causing them and record status on a per-request basis to understand what is going on This bug is to track a merge request. The CL in question is https://codereview.chromium.org/2751253004/
,
Mar 24 2017
Please add applicable OSs. Thanks!
,
Mar 24 2017
,
Mar 25 2017
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 27 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8b28e3a4dc7090a2ad9cbcf66fc427d941b8c1dd commit 8b28e3a4dc7090a2ad9cbcf66fc427d941b8c1dd Author: Pete Williamson <petewil@chromium.org> Date: Mon Mar 27 18:36:19 2017 [Offline Pages] Add UMA to record final offline request status and loading failed error code. BUG= 704999 Review-Url: https://codereview.chromium.org/2751253004 Cr-Commit-Position: refs/heads/master@{#458001} (cherry picked from commit 642d1848697b731de396d6df1577443e33e17ad0) Review-Url: https://codereview.chromium.org/2779783002 . Cr-Commit-Position: refs/branch-heads/3029@{#433} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} [modify] https://crrev.com/8b28e3a4dc7090a2ad9cbcf66fc427d941b8c1dd/chrome/browser/android/offline_pages/background_loader_offliner.cc [modify] https://crrev.com/8b28e3a4dc7090a2ad9cbcf66fc427d941b8c1dd/chrome/browser/android/offline_pages/background_loader_offliner_unittest.cc [modify] https://crrev.com/8b28e3a4dc7090a2ad9cbcf66fc427d941b8c1dd/components/offline_pages/core/background/request_coordinator.cc [modify] https://crrev.com/8b28e3a4dc7090a2ad9cbcf66fc427d941b8c1dd/components/offline_pages/core/background/request_coordinator_unittest.cc [modify] https://crrev.com/8b28e3a4dc7090a2ad9cbcf66fc427d941b8c1dd/components/offline_pages/core/background/request_notifier.h [modify] https://crrev.com/8b28e3a4dc7090a2ad9cbcf66fc427d941b8c1dd/tools/metrics/histograms/histograms.xml |
||||
►
Sign in to add a comment |
||||
Comment 1 by chili@chromium.org
, Mar 24 2017