Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(258)

Unified Diff: ios/chrome/browser/metrics/mobile_session_shutdown_metrics_provider_unittest.mm

Issue 2592173002: Fix ios_chrome_unittests to crash on DCHECK failure. (Closed)
Patch Set: Address comments Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ios/chrome/browser/metrics/mobile_session_shutdown_metrics_provider_unittest.mm
diff --git a/ios/chrome/browser/metrics/mobile_session_shutdown_metrics_provider_unittest.mm b/ios/chrome/browser/metrics/mobile_session_shutdown_metrics_provider_unittest.mm
index 4c942adaa4211b44eb051c7959b7d6561a6396ef..c02088a2af3ecd3b684417c3a1f798171d1a0abe 100644
--- a/ios/chrome/browser/metrics/mobile_session_shutdown_metrics_provider_unittest.mm
+++ b/ios/chrome/browser/metrics/mobile_session_shutdown_metrics_provider_unittest.mm
@@ -69,9 +69,7 @@ class MobileSessionShutdownMetricsProviderForTesting
class MobileSessionShutdownMetricsProviderTest
: public testing::TestWithParam<int> {
public:
- MobileSessionShutdownMetricsProviderTest()
- : task_runner_(new base::TestSimpleTaskRunner) {
- base::SetRecordActionTaskRunner(task_runner_);
+ MobileSessionShutdownMetricsProviderTest() {
metrics::MetricsService::RegisterPrefs(local_state_.registry());
}
@@ -82,7 +80,6 @@ class MobileSessionShutdownMetricsProviderTest
std::unique_ptr<metrics::MetricsService> metrics_service_;
std::unique_ptr<MobileSessionShutdownMetricsProviderForTesting>
metrics_provider_;
- scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
private:
DISALLOW_COPY_AND_ASSIGN(MobileSessionShutdownMetricsProviderTest);

Powered by Google App Engine
This is Rietveld 408576698