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

Unified Diff: ash/launcher/scoped_observer_with_duplicated_sources_unittest.cc

Issue 23534023: ash: Minor cleanup, move more files from launcher to shelf. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 4 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
« no previous file with comments | « ash/launcher/scoped_observer_with_duplicated_sources.h ('k') | ash/shelf/overflow_bubble.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/launcher/scoped_observer_with_duplicated_sources_unittest.cc
diff --git a/ash/launcher/scoped_observer_with_duplicated_sources_unittest.cc b/ash/launcher/scoped_observer_with_duplicated_sources_unittest.cc
deleted file mode 100644
index d7d13f3602a5b42a2d8a7f0152da13b6a8a18fd4..0000000000000000000000000000000000000000
--- a/ash/launcher/scoped_observer_with_duplicated_sources_unittest.cc
+++ /dev/null
@@ -1,82 +0,0 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "ash/launcher/scoped_observer_with_duplicated_sources.h"
-
-#include "base/memory/scoped_ptr.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-class TestObserver {
- public:
- TestObserver() {}
- ~TestObserver() {}
-
- private:
- DISALLOW_COPY_AND_ASSIGN(TestObserver);
-};
-
-class TestSource {
- public:
- TestSource() : observer_count_(0) {}
- ~TestSource() {}
-
- void AddObserver(TestObserver* observer) {
- observer_count_++;
- }
- void RemoveObserver(TestObserver* observer) {
- observer_count_--;
- }
-
- int GetObserverCount() {
- return observer_count_;
- }
-
- private:
- int observer_count_;
-
- DISALLOW_COPY_AND_ASSIGN(TestSource);
-};
-
-TEST(ScopedObserverWithDuplicatedSourcesTest, DuplicatedSource) {
- TestObserver observer;
- TestSource source1;
- TestSource source2;
-
- ScopedObserverWithDuplicatedSources<TestSource, TestObserver>
- observers(&observer);
- EXPECT_EQ(0, source1.GetObserverCount());
- EXPECT_FALSE(observers.IsObserving(&source1));
- EXPECT_EQ(0, source2.GetObserverCount());
- EXPECT_FALSE(observers.IsObserving(&source2));
-
- // Add |source1|.
- observers.Add(&source1);
- EXPECT_EQ(1, source1.GetObserverCount());
- EXPECT_TRUE(observers.IsObserving(&source1));
- // AddObserver of TestSource is called only once.
- observers.Add(&source1);
- EXPECT_EQ(1, source1.GetObserverCount());
- EXPECT_TRUE(observers.IsObserving(&source1));
-
- // Add |source2|.
- observers.Add(&source2);
- EXPECT_EQ(1, source2.GetObserverCount());
- EXPECT_TRUE(observers.IsObserving(&source2));
-
- // Remove |source1|.
- observers.Remove(&source1);
- EXPECT_EQ(1, source1.GetObserverCount());
- EXPECT_TRUE(observers.IsObserving(&source1));
-
- // Remove |source2|.
- observers.Remove(&source2);
- EXPECT_EQ(0, source2.GetObserverCount());
- EXPECT_FALSE(observers.IsObserving(&source2));
-
- // Remove |source1| again.
- observers.Remove(&source1);
- // In this time, |observer| is removed from |source1|.
- EXPECT_EQ(0, source1.GetObserverCount());
- EXPECT_FALSE(observers.IsObserving(&source1));
-}
« no previous file with comments | « ash/launcher/scoped_observer_with_duplicated_sources.h ('k') | ash/shelf/overflow_bubble.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698