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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/background_page_tracker_unittest.cc

Issue 5915004: Introduce incognito preference settings. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Whitespaces + fixes for trybot Created 10 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
« no previous file with comments | « chrome/browser/about_flags_unittest.cc ('k') | chrome/browser/chromeos/login/signed_settings_temp_storage_unittest.cc » ('j') | chrome/browser/policy/configuration_policy_pref_store.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/background_page_tracker_unittest.cc
diff --git a/chrome/browser/background_page_tracker_unittest.cc b/chrome/browser/background_page_tracker_unittest.cc
index b7039bdadbc79c8ab19a106cf6d8d6ccc3f44dad..b100019c8cf047e75b7eaab5614c096773df7382 100644
--- a/chrome/browser/background_page_tracker_unittest.cc
+++ b/chrome/browser/background_page_tracker_unittest.cc
@@ -15,15 +15,16 @@
class MockBackgroundPageTracker : public BackgroundPageTracker {
public:
- MockBackgroundPageTracker() {
- BackgroundPageTracker::RegisterPrefs(&prefs_);
+ MockBackgroundPageTracker()
+ : prefs_(TestingPrefService::CreateTestingPrefService()) {
+ BackgroundPageTracker::RegisterPrefs(prefs_.get());
}
~MockBackgroundPageTracker() {}
// Overridden from BackgroundPageTracker to mock out functionality.
virtual bool IsEnabled() { return true; }
- virtual PrefService* GetPrefService() { return &prefs_; }
+ virtual PrefService* GetPrefService() { return prefs_.get(); }
private:
- TestingPrefService prefs_;
+ scoped_ptr<TestingPrefService> prefs_;
};
TEST(BackgroundPageTrackerTest, Create) {
« no previous file with comments | « chrome/browser/about_flags_unittest.cc ('k') | chrome/browser/chromeos/login/signed_settings_temp_storage_unittest.cc » ('j') | chrome/browser/policy/configuration_policy_pref_store.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698