• Henrique Nakashima's avatar
    Revert "Flatten UpdateMetricsUsagePrefs - skip Metrics and Reporting service." · 4b9de7c9
    Henrique Nakashima authored
    This reverts commit a061c829.
    
    Reason for revert: Suspect in crbug.com/906242
    
    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}
    
    TBR=asvitkine@chromium.org,bcwhite@chromium.org,rajendrant@chromium.org,hnakashima@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 902791
    Change-Id: Iad1f49a8bae42e60ae5a9c68509d4cb38db683ef
    Reviewed-on: https://chromium-review.googlesource.com/c/1344817Reviewed-by: default avatarHenrique Nakashima <hnakashima@chromium.org>
    Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#609861}
    4b9de7c9
metrics_service.cc 36.9 KB