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

Unified Diff: chrome/browser/background/background_mode_manager_unittest.cc

Issue 2528013002: Initialize the message center as part of the testing browser process (Closed)
Patch Set: re-uploding due to time outs Created 4 years, 1 month 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: chrome/browser/background/background_mode_manager_unittest.cc
diff --git a/chrome/browser/background/background_mode_manager_unittest.cc b/chrome/browser/background/background_mode_manager_unittest.cc
index 6da58ea0297cc791a5bba4c131e1650d02ea114c..7f95687fbfa786ded5edbe24f331e5d36ee23a37 100644
--- a/chrome/browser/background/background_mode_manager_unittest.cc
+++ b/chrome/browser/background/background_mode_manager_unittest.cc
@@ -228,9 +228,6 @@ class BackgroundModeManagerWithExtensionsTest : public testing::Test {
profile_manager_ = CreateTestingProfileManager();
profile_ = profile_manager_->CreateTestingProfile("p1");
- // Aura clears notifications from the message center at shutdown.
- message_center::MessageCenter::Initialize();
-
test_keep_alive_.reset(
new ScopedKeepAlive(KeepAliveOrigin::BACKGROUND_MODE_MANAGER,
KeepAliveRestartOption::DISABLED));
@@ -276,10 +273,6 @@ class BackgroundModeManagerWithExtensionsTest : public testing::Test {
// before tearing down the Message Center.
profile_manager_.reset();
- // Message Center shutdown must occur after the KeepAlive is released
- // because clearing it will end up referencing the message center.
- message_center::MessageCenter::Shutdown();
-
// Clear the shutdown flag to isolate the remaining effect of this test.
browser_shutdown::SetTryingToQuit(false);
}
« no previous file with comments | « chrome/browser/background/background_contents_service_unittest.cc ('k') | chrome/browser/browser_process_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698