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

Unified Diff: chrome/browser/status_icons/status_icon_unittest.cc

Issue 10408039: status_icons: Factor out status icon observer into its own header file. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/status_icons/status_icon_unittest.cc
diff --git a/chrome/browser/status_icons/status_icon_unittest.cc b/chrome/browser/status_icons/status_icon_unittest.cc
index 832db095e87a36bac65e1ac90b61e16f78926bca..5897be8e705cccd0d80ec2e34fe2d945baf81274 100644
--- a/chrome/browser/status_icons/status_icon_unittest.cc
+++ b/chrome/browser/status_icons/status_icon_unittest.cc
@@ -2,14 +2,16 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "base/compiler_specific.h"
#include "chrome/browser/status_icons/status_icon.h"
+
+#include "base/compiler_specific.h"
+#include "chrome/browser/status_icons/status_icon_observer.h"
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
-class MockStatusIconObserver : public StatusIcon::Observer {
+class MockStatusIconObserver : public StatusIconObserver {
public:
- MOCK_METHOD0(OnClicked, void());
+ MOCK_METHOD0(OnStatusIconClicked, void());
};
// Define pure virtual functions so we can test base class functionality.
@@ -29,8 +31,8 @@ TEST(StatusIconTest, ObserverAdd) {
// Make sure that observers are invoked when we click items.
TestStatusIcon icon;
MockStatusIconObserver observer, observer2;
- EXPECT_CALL(observer, OnClicked()).Times(2);
- EXPECT_CALL(observer2, OnClicked());
+ EXPECT_CALL(observer, OnStatusIconClicked()).Times(2);
+ EXPECT_CALL(observer2, OnStatusIconClicked());
icon.AddObserver(&observer);
icon.DispatchClickEvent();
icon.AddObserver(&observer2);
@@ -43,7 +45,7 @@ TEST(StatusIconTest, ObserverRemove) {
// Make sure that observers are no longer invoked after they are removed.
TestStatusIcon icon;
MockStatusIconObserver observer;
- EXPECT_CALL(observer, OnClicked());
+ EXPECT_CALL(observer, OnStatusIconClicked());
icon.AddObserver(&observer);
icon.DispatchClickEvent();
icon.RemoveObserver(&observer);

Powered by Google App Engine
This is Rietveld 408576698