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

Side by Side Diff: chrome/test/base/testing_profile.h

Issue 7967024: Profile shouldn't own PersonalDataManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addresses isherman #11 Created 9 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/test/base/profile_mock.h ('k') | chrome/test/base/testing_profile.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_TEST_BASE_TESTING_PROFILE_H_ 5 #ifndef CHROME_TEST_BASE_TESTING_PROFILE_H_
6 #define CHROME_TEST_BASE_TESTING_PROFILE_H_ 6 #define CHROME_TEST_BASE_TESTING_PROFILE_H_
7 #pragma once 7 #pragma once
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 169 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 // Sets the profile's PrefService. If a pref service hasn't been explicitly 180 // Sets the profile's PrefService. If a pref service hasn't been explicitly
181 // set GetPrefs creates one, so normally you need not invoke this. If you need 181 // set GetPrefs creates one, so normally you need not invoke this. If you need
182 // to set a pref service you must invoke this before GetPrefs. 182 // to set a pref service you must invoke this before GetPrefs.
183 // TestingPrefService takes ownership of |prefs|. 183 // TestingPrefService takes ownership of |prefs|.
184 void SetPrefService(PrefService* prefs); 184 void SetPrefService(PrefService* prefs);
185 virtual PrefService* GetPrefs(); 185 virtual PrefService* GetPrefs();
186 virtual TemplateURLFetcher* GetTemplateURLFetcher(); 186 virtual TemplateURLFetcher* GetTemplateURLFetcher();
187 virtual history::TopSites* GetTopSites(); 187 virtual history::TopSites* GetTopSites();
188 virtual history::TopSites* GetTopSitesWithoutCreating(); 188 virtual history::TopSites* GetTopSitesWithoutCreating();
189 virtual DownloadManager* GetDownloadManager(); 189 virtual DownloadManager* GetDownloadManager();
190 virtual PersonalDataManager* GetPersonalDataManager();
191 virtual fileapi::FileSystemContext* GetFileSystemContext(); 190 virtual fileapi::FileSystemContext* GetFileSystemContext();
192 virtual void SetQuotaManager(quota::QuotaManager* manager); 191 virtual void SetQuotaManager(quota::QuotaManager* manager);
193 virtual quota::QuotaManager* GetQuotaManager(); 192 virtual quota::QuotaManager* GetQuotaManager();
194 virtual bool HasCreatedDownloadManager() const; 193 virtual bool HasCreatedDownloadManager() const;
195 194
196 // Returns a testing ContextGetter (if one has been created via 195 // Returns a testing ContextGetter (if one has been created via
197 // CreateRequestContext) or NULL. This is not done on-demand for two reasons: 196 // CreateRequestContext) or NULL. This is not done on-demand for two reasons:
198 // (1) Some tests depend on GetRequestContext() returning NULL. (2) Because 197 // (1) Some tests depend on GetRequestContext() returning NULL. (2) Because
199 // of the special memory management considerations for the 198 // of the special memory management considerations for the
200 // TestURLRequestContextGetter class, many tests would find themseleves 199 // TestURLRequestContextGetter class, many tests would find themseleves
(...skipping 202 matching lines...) Expand 10 before | Expand all | Expand 10 after
403 // testing. 402 // testing.
404 ProfileDependencyManager* profile_dependency_manager_; 403 ProfileDependencyManager* profile_dependency_manager_;
405 404
406 scoped_refptr<ChromeAppCacheService> appcache_service_; 405 scoped_refptr<ChromeAppCacheService> appcache_service_;
407 406
408 // The QuotaManager, only available if set explicitly via SetQuotaManager. 407 // The QuotaManager, only available if set explicitly via SetQuotaManager.
409 scoped_refptr<quota::QuotaManager> quota_manager_; 408 scoped_refptr<quota::QuotaManager> quota_manager_;
410 }; 409 };
411 410
412 #endif // CHROME_TEST_BASE_TESTING_PROFILE_H_ 411 #endif // CHROME_TEST_BASE_TESTING_PROFILE_H_
OLDNEW
« no previous file with comments | « chrome/test/base/profile_mock.h ('k') | chrome/test/base/testing_profile.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698