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

Unified Diff: ash/shelf/scoped_observer_with_duplicated_sources_unittest.cc

Issue 2185743002: Remove ash::ScopedObserverWithDuplicatedSources (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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: ash/shelf/scoped_observer_with_duplicated_sources_unittest.cc
diff --git a/ash/shelf/scoped_observer_with_duplicated_sources_unittest.cc b/ash/shelf/scoped_observer_with_duplicated_sources_unittest.cc
deleted file mode 100644
index ef9c167c847b669b0897979b1f260a8455c6d55a..0000000000000000000000000000000000000000
--- a/ash/shelf/scoped_observer_with_duplicated_sources_unittest.cc
+++ /dev/null
@@ -1,75 +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/shelf/scoped_observer_with_duplicated_sources.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));
-}

Powered by Google App Engine
This is Rietveld 408576698