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

Side by Side Diff: chrome/browser/prefs/pref_notifier_impl_unittest.cc

Issue 10494012: Move mock notification observer header from content\test to content\public\test. This way we can en… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 6 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "chrome/browser/prefs/pref_notifier_impl.h" 5 #include "chrome/browser/prefs/pref_notifier_impl.h"
6 #include "chrome/browser/prefs/pref_observer_mock.h" 6 #include "chrome/browser/prefs/pref_observer_mock.h"
7 #include "chrome/browser/prefs/pref_service.h" 7 #include "chrome/browser/prefs/pref_service.h"
8 #include "chrome/browser/prefs/pref_value_store.h" 8 #include "chrome/browser/prefs/pref_value_store.h"
9 #include "chrome/common/chrome_notification_types.h" 9 #include "chrome/common/chrome_notification_types.h"
10 #include "chrome/test/base/testing_pref_service.h" 10 #include "chrome/test/base/testing_pref_service.h"
11 #include "content/public/browser/notification_details.h" 11 #include "content/public/browser/notification_details.h"
12 #include "content/public/browser/notification_registrar.h" 12 #include "content/public/browser/notification_registrar.h"
13 #include "content/public/browser/notification_source.h" 13 #include "content/public/browser/notification_source.h"
14 #include "content/test/notification_observer_mock.h" 14 #include "content/public/test/mock_notification_observer.h"
15 #include "testing/gmock/include/gmock/gmock.h" 15 #include "testing/gmock/include/gmock/gmock.h"
16 #include "testing/gtest/include/gtest/gtest.h" 16 #include "testing/gtest/include/gtest/gtest.h"
17 17
18 using testing::_; 18 using testing::_;
19 using testing::Field; 19 using testing::Field;
20 using testing::Invoke; 20 using testing::Invoke;
21 using testing::Mock; 21 using testing::Mock;
22 using testing::Truly; 22 using testing::Truly;
23 23
24 namespace { 24 namespace {
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 }; 73 };
74 74
75 TEST_F(PrefNotifierTest, OnPreferenceChanged) { 75 TEST_F(PrefNotifierTest, OnPreferenceChanged) {
76 MockPrefNotifier notifier(&pref_service_); 76 MockPrefNotifier notifier(&pref_service_);
77 EXPECT_CALL(notifier, FireObservers(kChangedPref)).Times(1); 77 EXPECT_CALL(notifier, FireObservers(kChangedPref)).Times(1);
78 notifier.OnPreferenceChanged(kChangedPref); 78 notifier.OnPreferenceChanged(kChangedPref);
79 } 79 }
80 80
81 TEST_F(PrefNotifierTest, OnInitializationCompleted) { 81 TEST_F(PrefNotifierTest, OnInitializationCompleted) {
82 MockPrefNotifier notifier(&pref_service_); 82 MockPrefNotifier notifier(&pref_service_);
83 content::NotificationObserverMock observer; 83 content::MockNotificationObserver observer;
84 content::NotificationRegistrar registrar; 84 content::NotificationRegistrar registrar;
85 registrar.Add(&observer, chrome::NOTIFICATION_PREF_INITIALIZATION_COMPLETED, 85 registrar.Add(&observer, chrome::NOTIFICATION_PREF_INITIALIZATION_COMPLETED,
86 content::Source<PrefService>(&pref_service_)); 86 content::Source<PrefService>(&pref_service_));
87 EXPECT_CALL(observer, Observe( 87 EXPECT_CALL(observer, Observe(
88 int(chrome::NOTIFICATION_PREF_INITIALIZATION_COMPLETED), 88 int(chrome::NOTIFICATION_PREF_INITIALIZATION_COMPLETED),
89 content::Source<PrefService>(&pref_service_), 89 content::Source<PrefService>(&pref_service_),
90 Property(&content::Details<bool>::ptr, testing::Pointee(true)))); 90 Property(&content::Details<bool>::ptr, testing::Pointee(true))));
91 notifier.OnInitializationCompleted(true); 91 notifier.OnInitializationCompleted(true);
92 } 92 }
93 93
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
191 obs2_.Expect(&pref_service_, kChangedPref, &value_true); 191 obs2_.Expect(&pref_service_, kChangedPref, &value_true);
192 notifier.OnPreferenceChanged(kChangedPref); 192 notifier.OnPreferenceChanged(kChangedPref);
193 Mock::VerifyAndClearExpectations(&obs1_); 193 Mock::VerifyAndClearExpectations(&obs1_);
194 Mock::VerifyAndClearExpectations(&obs2_); 194 Mock::VerifyAndClearExpectations(&obs2_);
195 195
196 notifier.RemovePrefObserver(kChangedPref, &obs2_); 196 notifier.RemovePrefObserver(kChangedPref, &obs2_);
197 notifier.RemovePrefObserver(kUnchangedPref, &obs2_); 197 notifier.RemovePrefObserver(kUnchangedPref, &obs2_);
198 } 198 }
199 199
200 } // namespace 200 } // namespace
OLDNEW
« no previous file with comments | « chrome/browser/prefs/pref_change_registrar_unittest.cc ('k') | chrome/browser/prefs/pref_set_observer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698