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

Unified Diff: chrome/browser/storage_monitor/storage_monitor_unittest.cc

Issue 14556015: [Media Galleries] Lazily initialize the storage monitor. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Adjustments Created 7 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/storage_monitor/storage_monitor_unittest.cc
diff --git a/chrome/browser/storage_monitor/storage_monitor_unittest.cc b/chrome/browser/storage_monitor/storage_monitor_unittest.cc
index bc57f4afa9474bf5f244f44861fcaaadbe61c520..21ee0fbfc6812e37ea817b0d4f27d51d20e7f525 100644
--- a/chrome/browser/storage_monitor/storage_monitor_unittest.cc
+++ b/chrome/browser/storage_monitor/storage_monitor_unittest.cc
@@ -8,11 +8,14 @@
#include "chrome/browser/storage_monitor/mock_removable_storage_observer.h"
#include "chrome/browser/storage_monitor/storage_monitor.h"
#include "chrome/browser/storage_monitor/test_storage_monitor.h"
+#include "content/public/test/test_browser_thread.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace chrome {
TEST(StorageMonitorTest, TestInitialize) {
+ MessageLoopForUI message_loop_;
+ content::TestBrowserThread(content::BrowserThread::UI, &message_loop_);
tommycli 2013/05/22 21:13:12 Since you no longer DCHECK that you're on UI threa
Greg Billock 2013/05/23 00:55:59 No. It doesn't work anyway. :-/
test::TestStorageMonitor monitor;
EXPECT_FALSE(monitor.init_called_);

Powered by Google App Engine
This is Rietveld 408576698