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

Unified Diff: sync/notifier/sync_system_resources_unittest.cc

Issue 10907070: [Sync] Rename classes in sync/ that start with Chrome (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rename var Created 8 years, 3 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: sync/notifier/sync_system_resources_unittest.cc
diff --git a/sync/notifier/chrome_system_resources_unittest.cc b/sync/notifier/sync_system_resources_unittest.cc
similarity index 84%
rename from sync/notifier/chrome_system_resources_unittest.cc
rename to sync/notifier/sync_system_resources_unittest.cc
index 7d05db8da10ab2e7ff242d6beda75ed810441d39..08a57755d9f3d2c471d8d04e583dc84e149a9ec3 100644
--- a/sync/notifier/chrome_system_resources_unittest.cc
+++ b/sync/notifier/sync_system_resources_unittest.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "sync/notifier/chrome_system_resources.h"
+#include "sync/notifier/sync_system_resources.h"
#include <string>
@@ -46,14 +46,14 @@ class MockStorageCallback {
}
};
-class ChromeSystemResourcesTest : public testing::Test {
+class SyncSystemResourcesTest : public testing::Test {
protected:
- ChromeSystemResourcesTest()
+ SyncSystemResourcesTest()
: chrome_system_resources_(
rlarocque 2012/09/05 00:58:31 Missed one?
scoped_ptr<notifier::PushClient>(new notifier::FakePushClient()),
&mock_state_writer_) {}
- virtual ~ChromeSystemResourcesTest() {}
+ virtual ~SyncSystemResourcesTest() {}
void ScheduleShouldNotRun() {
{
@@ -85,49 +85,49 @@ class ChromeSystemResourcesTest : public testing::Test {
// Needed by |chrome_system_resources_|.
MessageLoop message_loop_;
MockStateWriter mock_state_writer_;
- ChromeSystemResources chrome_system_resources_;
+ SyncSystemResources chrome_system_resources_;
private:
- DISALLOW_COPY_AND_ASSIGN(ChromeSystemResourcesTest);
+ DISALLOW_COPY_AND_ASSIGN(SyncSystemResourcesTest);
};
// Make sure current_time() doesn't crash or leak.
-TEST_F(ChromeSystemResourcesTest, CurrentTime) {
+TEST_F(SyncSystemResourcesTest, CurrentTime) {
invalidation::Time current_time =
chrome_system_resources_.internal_scheduler()->GetCurrentTime();
DVLOG(1) << "current_time returned: " << current_time.ToInternalValue();
}
// Make sure Log() doesn't crash or leak.
-TEST_F(ChromeSystemResourcesTest, Log) {
- chrome_system_resources_.logger()->Log(ChromeLogger::INFO_LEVEL,
+TEST_F(SyncSystemResourcesTest, Log) {
+ chrome_system_resources_.logger()->Log(SyncLogger::INFO_LEVEL,
__FILE__, __LINE__, "%s %d",
"test string", 5);
}
-TEST_F(ChromeSystemResourcesTest, ScheduleBeforeStart) {
+TEST_F(SyncSystemResourcesTest, ScheduleBeforeStart) {
ScheduleShouldNotRun();
chrome_system_resources_.Start();
}
-TEST_F(ChromeSystemResourcesTest, ScheduleAfterStop) {
+TEST_F(SyncSystemResourcesTest, ScheduleAfterStop) {
chrome_system_resources_.Start();
chrome_system_resources_.Stop();
ScheduleShouldNotRun();
}
-TEST_F(ChromeSystemResourcesTest, ScheduleAndStop) {
+TEST_F(SyncSystemResourcesTest, ScheduleAndStop) {
chrome_system_resources_.Start();
ScheduleShouldNotRun();
chrome_system_resources_.Stop();
}
-TEST_F(ChromeSystemResourcesTest, ScheduleAndDestroy) {
+TEST_F(SyncSystemResourcesTest, ScheduleAndDestroy) {
chrome_system_resources_.Start();
ScheduleShouldNotRun();
}
-TEST_F(ChromeSystemResourcesTest, ScheduleImmediately) {
+TEST_F(SyncSystemResourcesTest, ScheduleImmediately) {
chrome_system_resources_.Start();
MockClosure mock_closure;
EXPECT_CALL(mock_closure, Run());
@@ -136,7 +136,7 @@ TEST_F(ChromeSystemResourcesTest, ScheduleImmediately) {
message_loop_.RunAllPending();
}
-TEST_F(ChromeSystemResourcesTest, ScheduleOnListenerThread) {
+TEST_F(SyncSystemResourcesTest, ScheduleOnListenerThread) {
chrome_system_resources_.Start();
MockClosure mock_closure;
EXPECT_CALL(mock_closure, Run());
@@ -147,7 +147,7 @@ TEST_F(ChromeSystemResourcesTest, ScheduleOnListenerThread) {
message_loop_.RunAllPending();
}
-TEST_F(ChromeSystemResourcesTest, ScheduleWithZeroDelay) {
+TEST_F(SyncSystemResourcesTest, ScheduleWithZeroDelay) {
chrome_system_resources_.Start();
MockClosure mock_closure;
EXPECT_CALL(mock_closure, Run());
@@ -158,7 +158,7 @@ TEST_F(ChromeSystemResourcesTest, ScheduleWithZeroDelay) {
// TODO(akalin): Figure out how to test with a non-zero delay.
-TEST_F(ChromeSystemResourcesTest, WriteState) {
+TEST_F(SyncSystemResourcesTest, WriteState) {
chrome_system_resources_.Start();
EXPECT_CALL(mock_state_writer_, WriteState(_));
// Owned by WriteState.

Powered by Google App Engine
This is Rietveld 408576698