Download.Service.Driver.InterruptReason metric should not record all downloads. |
||
Issue descriptionCurrently Download.Service.Driver.InterruptReason tracks all downloads, instead, it should only track downloads fired by download service.
,
Sep 19 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2bdd5022f640fb43f6494b9b44169d0df26e3e5c commit 2bdd5022f640fb43f6494b9b44169d0df26e3e5c Author: Xing Liu <xingliu@chromium.org> Date: Tue Sep 19 17:02:40 2017 Download Service: Fixed an issue in interrupt reason metric. In Download.Service.Driver.InterruptReason, it tracks all downloads, this CL changes to only track downloads with guid in download service's database. Bug: 765430 Change-Id: Id2ddf1a5bb39f5fa47c4d301171aa5704c1633a0 Reviewed-on: https://chromium-review.googlesource.com/667246 Commit-Queue: Xing Liu <xingliu@chromium.org> Reviewed-by: David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#502869} [modify] https://crrev.com/2bdd5022f640fb43f6494b9b44169d0df26e3e5c/components/download/content/internal/download_driver_impl.cc [modify] https://crrev.com/2bdd5022f640fb43f6494b9b44169d0df26e3e5c/components/download/content/internal/download_driver_impl_unittest.cc [modify] https://crrev.com/2bdd5022f640fb43f6494b9b44169d0df26e3e5c/components/download/internal/controller_impl.cc [modify] https://crrev.com/2bdd5022f640fb43f6494b9b44169d0df26e3e5c/components/download/internal/controller_impl.h [modify] https://crrev.com/2bdd5022f640fb43f6494b9b44169d0df26e3e5c/components/download/internal/download_driver.h
,
Sep 21 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by xingliu@chromium.org
, Sep 14 2017