• Henrique Nakashima's avatar
    Reland "Flatten UpdateMetricsUsagePrefs - skip Metrics and Reporting service." · 0ff71979
    Henrique Nakashima authored
    This is a reland of a061c829
    
    Reverted due to being a suspect in crbug.com/906242, but after reverting the
    bug persists and was bisected to another CL.
    
    Original change's description:
    > Flatten UpdateMetricsUsagePrefs - skip Metrics and Reporting service.
    >
    > UpdateMetricsUsagePrefs() is a chain of calls:
    > - ChromeDataUseMeasurement::UpdateDataUseToMetricsService()
    >   -> UpdateMetricsUsagePrefs()
    >   -> MetricsService::UpdateMetricsUsagePrefs()
    >   -> ReportingService::UpdateMetricsUsagePrefs()
    >   -> DataUseTracker::UpdateMetricsUsagePrefs()
    >
    > It can be reduced to:
    > - ChromeDataUseMeasurement::UpdateDataUseToMetricsService()
    >   -> UpdateMetricsUsagePrefs()
    >   -> DataUseTracker::UpdateMetricsUsagePrefs()
    >
    > This removes the dependency from ChromeDataUseMeasurement to
    > MetricsService and ReportingService.
    >
    > Bug: 902791
    > Change-Id: I38a6d22d1ff823134c79bc342ea32be0f89cda77
    > Reviewed-on: https://chromium-review.googlesource.com/c/1334267
    > Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
    > Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
    > Reviewed-by: rajendrant <rajendrant@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#608567}
    
    Bug: 902791
    Change-Id: I27abaac1d265e63ae5a921ab2839a77ae8582720
    Reviewed-on: https://chromium-review.googlesource.com/c/1348249Reviewed-by: default avatarrajendrant <rajendrant@chromium.org>
    Reviewed-by: default avatarAlexei Svitkine <asvitkine@chromium.org>
    Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#611250}
    0ff71979
reporting_service.cc 6.72 KB