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

Unified Diff: chrome/browser/extensions/extension_prefs_unittest.h

Issue 11027070: Moved JsonPrefStore to use SequencedWorkerPool (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 2 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/extensions/extension_prefs_unittest.h
diff --git a/chrome/browser/extensions/extension_prefs_unittest.h b/chrome/browser/extensions/extension_prefs_unittest.h
index ba5832a009b2734c69355e580a0624e13f789d05..28b9ef0f39091e2e29511d56ddcf5d61e5fd9333 100644
--- a/chrome/browser/extensions/extension_prefs_unittest.h
+++ b/chrome/browser/extensions/extension_prefs_unittest.h
@@ -11,6 +11,7 @@
#include "testing/gtest/include/gtest/gtest.h"
namespace base {
+class SequencedWorkerPool;
akalin 2012/10/19 02:00:51 not needed anymore, right?
zel 2012/10/19 18:45:07 Done.
class Value;
}
@@ -40,13 +41,13 @@ class ExtensionPrefsTest : public testing::Test {
virtual void TearDown() OVERRIDE;
protected:
- ExtensionPrefs* prefs() { return prefs_.prefs(); }
+ ExtensionPrefs* prefs() { return prefs_->prefs(); }
MessageLoop message_loop_;
content::TestBrowserThread ui_thread_;
content::TestBrowserThread file_thread_;
- TestExtensionPrefs prefs_;
+ scoped_ptr<TestExtensionPrefs> prefs_;
akalin 2012/10/19 02:00:51 why did you make this a scoped_ptr? It looks like
zel 2012/10/19 18:45:07 Done.
private:
DISALLOW_COPY_AND_ASSIGN(ExtensionPrefsTest);

Powered by Google App Engine
This is Rietveld 408576698