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

Side by Side Diff: chrome/browser/notifications/login_state_notification_blocker_chromeos_browsertest.cc

Issue 287083003: Multi-profile flag cleanup (Closed) Base URL: https://chromium.googlesource.com/chromium/src
Patch Set: Created 6 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ash/shell.h" 5 #include "ash/shell.h"
6 #include "ash/system/system_notifier.h" 6 #include "ash/system/system_notifier.h"
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "chrome/browser/chromeos/login/login_manager_test.h" 8 #include "chrome/browser/chromeos/login/login_manager_test.h"
9 #include "chrome/browser/chromeos/login/startup_utils.h" 9 #include "chrome/browser/chromeos/login/startup_utils.h"
10 #include "chrome/browser/chromeos/login/ui/user_adding_screen.h" 10 #include "chrome/browser/chromeos/login/ui/user_adding_screen.h"
11 #include "chrome/browser/notifications/login_state_notification_blocker_chromeos .h" 11 #include "chrome/browser/notifications/login_state_notification_blocker_chromeos .h"
12 #include "chrome/common/chrome_switches.h" 12 #include "chrome/common/chrome_switches.h"
Nikita (slow) 2014/05/19 14:27:15 nit: not used
13 #include "content/public/test/test_utils.h" 13 #include "content/public/test/test_utils.h"
14 #include "ui/message_center/message_center.h" 14 #include "ui/message_center/message_center.h"
15 15
16 using namespace testing; 16 using namespace testing;
17 17
18 namespace { 18 namespace {
19 19
20 const char* kTestUsers[] = {"test-user@gmail.com", 20 const char* kTestUsers[] = {"test-user@gmail.com",
21 "test-user1@gmail.com"}; 21 "test-user1@gmail.com"};
22 22
23 } // anonymous namespace 23 } // anonymous namespace
24 24
25 class LoginStateNotificationBlockerChromeOSBrowserTest 25 class LoginStateNotificationBlockerChromeOSBrowserTest
26 : public chromeos::LoginManagerTest, 26 : public chromeos::LoginManagerTest,
27 public message_center::NotificationBlocker::Observer { 27 public message_center::NotificationBlocker::Observer {
28 public: 28 public:
29 LoginStateNotificationBlockerChromeOSBrowserTest() 29 LoginStateNotificationBlockerChromeOSBrowserTest()
30 : chromeos::LoginManagerTest(false), 30 : chromeos::LoginManagerTest(false),
31 state_changed_count_(0) {} 31 state_changed_count_(0) {}
32 virtual ~LoginStateNotificationBlockerChromeOSBrowserTest() {} 32 virtual ~LoginStateNotificationBlockerChromeOSBrowserTest() {}
33 33
34 // InProcessBrowserTest overrides:
35 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
36 LoginManagerTest::SetUpCommandLine(command_line);
37 command_line->AppendSwitch(::switches::kMultiProfiles);
38 }
39
40 virtual void SetUpOnMainThread() OVERRIDE { 34 virtual void SetUpOnMainThread() OVERRIDE {
41 chromeos::LoginState::Get()->set_always_logged_in(false); 35 chromeos::LoginState::Get()->set_always_logged_in(false);
42 chromeos::LoginManagerTest::SetUpOnMainThread(); 36 chromeos::LoginManagerTest::SetUpOnMainThread();
43 } 37 }
44 38
45 virtual void CleanUpOnMainThread() OVERRIDE { 39 virtual void CleanUpOnMainThread() OVERRIDE {
46 if (blocker_) 40 if (blocker_)
47 blocker_->RemoveObserver(this); 41 blocker_->RemoveObserver(this);
48 blocker_.reset(); 42 blocker_.reset();
49 chromeos::LoginManagerTest::CleanUpOnMainThread(); 43 chromeos::LoginManagerTest::CleanUpOnMainThread();
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
139 content::RunAllPendingInMessageLoop(); 133 content::RunAllPendingInMessageLoop();
140 EXPECT_EQ(1, GetStateChangedCountAndReset()); 134 EXPECT_EQ(1, GetStateChangedCountAndReset());
141 EXPECT_TRUE(ShouldShowNotificationAsPopup(notifier_id)); 135 EXPECT_TRUE(ShouldShowNotificationAsPopup(notifier_id));
142 136
143 // Multi-login user switch off. 137 // Multi-login user switch off.
144 chromeos::UserAddingScreen::Get()->Cancel(); 138 chromeos::UserAddingScreen::Get()->Cancel();
145 content::RunAllPendingInMessageLoop(); 139 content::RunAllPendingInMessageLoop();
146 EXPECT_EQ(1, GetStateChangedCountAndReset()); 140 EXPECT_EQ(1, GetStateChangedCountAndReset());
147 EXPECT_TRUE(ShouldShowNotificationAsPopup(notifier_id)); 141 EXPECT_TRUE(ShouldShowNotificationAsPopup(notifier_id));
148 } 142 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698