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

Unified Diff: chrome/browser/extensions/api/system_info_storage/system_info_storage_apitest.cc

Issue 11415177: Implements the storage free space watching functionality into storage info provider (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: avoid naming conflict in browser_tests Created 8 years 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/extensions/api/system_info_storage/system_info_storage_apitest.cc
diff --git a/chrome/browser/extensions/api/system_info_storage/system_info_storage_apitest.cc b/chrome/browser/extensions/api/system_info_storage/system_info_storage_apitest.cc
index 3231c3d1e7fbd1fa11723d2133cca69fa6c6af4f..a03b8e9686432f72c0193b88d2071802aa7195ba 100644
--- a/chrome/browser/extensions/api/system_info_storage/system_info_storage_apitest.cc
+++ b/chrome/browser/extensions/api/system_info_storage/system_info_storage_apitest.cc
@@ -24,7 +24,7 @@ class MockStorageInfoProvider : public StorageInfoProvider {
MockStorageInfoProvider() : is_watching_(false) {
}
virtual ~MockStorageInfoProvider() {
- StopWatching();
+ Stop();
}
virtual bool QueryInfo(StorageInfo* info) OVERRIDE {
@@ -46,7 +46,7 @@ class MockStorageInfoProvider : public StorageInfoProvider {
return false;
}
- bool StartWatching() {
+ bool Start() {
if (is_watching_) return false;
// Start the timer to emulate storage.onChanged event.
@@ -58,7 +58,7 @@ class MockStorageInfoProvider : public StorageInfoProvider {
return true;
}
- bool StopWatching() {
+ bool Stop() {
if (!is_watching_) return false;
is_watching_ = false;
timer_.Stop();
@@ -110,7 +110,7 @@ IN_PROC_BROWSER_TEST_F(SystemInfoStorageApiTest, Storage) {
ui_test_utils::NavigateToURL(browser(), page_url);
EXPECT_TRUE(listener.WaitUntilSatisfied());
- provider->StartWatching();
+ provider->Start();
listener.Reply("go");
EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
}

Powered by Google App Engine
This is Rietveld 408576698