Make ~TabStatsTracker remove PowerMonitor observer.

This fixes TestTabStatsTracker failures running unit_tests.exe, because PowerMonitor dchecks if the same observer is added multiple times. Bug: 956719 Change-Id: Ic01197209d65a7d272f7e943ccc403a1c49107fa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2618221Reviewed-by: Brian White <bcwhite@chromium.org> Commit-Queue: David Bienvenu <davidbienvenu@chromium.org> Cr-Commit-Position: refs/heads/master@{#841623}

Make ~TabStatsTracker remove PowerMonitor observer.
This fixes TestTabStatsTracker failures running unit_tests.exe, because PowerMonitor dchecks if the same observer is added multiple times. Bug: 956719 Change-Id: Ic01197209d65a7d272f7e943ccc403a1c49107fa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2618221Reviewed-by: Brian White <bcwhite@chromium.org> Commit-Queue: David Bienvenu <davidbienvenu@chromium.org> Cr-Commit-Position: refs/heads/master@{#841623}
f85442a8 · David Bienvenu · Chromium LUCI CQ · 2e80d6f6 · f85442a8
Commit f85442a8 authored Jan 08, 2021 by David Bienvenu Committed by Chromium LUCI CQ Jan 08, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 0 deletions

chrome/browser/metrics/tab_stats_tracker.cc chrome/browser/metrics/tab_stats_tracker.cc +1 -0

No files found.
--- a/chrome/browser/metrics/tab_stats_tracker.cc
+++ b/chrome/browser/metrics/tab_stats_tracker.cc
@@ -182,6 +182,7 @@ TabStatsTracker::TabStatsTracker(PrefService* pref_service)
 TabStatsTracker::~TabStatsTracker() {
  DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
  BrowserList::GetInstance()->RemoveObserver(this);
+  base::PowerMonitor::RemoveObserver(this);
 }
 // static