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

Unified Diff: chrome/browser/ui/webui/sync_internals_message_handler_unittest.cc

Issue 2657673004: Add shutdown notification and service refs to SyncServiceObserver. (Closed)
Patch Set: Chromeos fix Created 3 years, 11 months 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/ui/webui/sync_internals_message_handler_unittest.cc
diff --git a/chrome/browser/ui/webui/sync_internals_message_handler_unittest.cc b/chrome/browser/ui/webui/sync_internals_message_handler_unittest.cc
index 1505a7e2677bcb888c347819572e79229ed68256..466b6e2fb3b21cf5c9468ae4890ca4b714b86de3 100644
--- a/chrome/browser/ui/webui/sync_internals_message_handler_unittest.cc
+++ b/chrome/browser/ui/webui/sync_internals_message_handler_unittest.cc
@@ -112,7 +112,7 @@ class SyncInternalsMessageHandlerTest : public ::testing::Test {
} // namespace
TEST_F(SyncInternalsMessageHandlerTest, SendAboutInfoWithService) {
- handler()->OnStateChanged();
+ handler()->OnStateChanged(nullptr);
EXPECT_EQ(1, fake_extractor()->call_count());
EXPECT_NE(nullptr, fake_extractor()->last_service());
EXPECT_NE(nullptr, fake_extractor()->last_signin());
@@ -121,7 +121,7 @@ TEST_F(SyncInternalsMessageHandlerTest, SendAboutInfoWithService) {
TEST_F(SyncInternalsMessageHandlerTest, SendAboutInfoWithoutService) {
base::CommandLine::ForCurrentProcess()->AppendSwitch(switches::kDisableSync);
- handler()->OnStateChanged();
+ handler()->OnStateChanged(nullptr);
EXPECT_EQ(1, fake_extractor()->call_count());
EXPECT_EQ(nullptr, fake_extractor()->last_service());
EXPECT_EQ(nullptr, fake_extractor()->last_signin());
« no previous file with comments | « chrome/browser/ui/webui/sync_internals_message_handler.cc ('k') | components/browser_sync/profile_sync_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698