diff options
| author | erg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-03-22 18:05:22 +0000 |
|---|---|---|
| committer | erg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-03-22 18:05:22 +0000 |
| commit | 18280378d78cc83d8e83de21f28411cb4fa777bc (patch) | |
| tree | a820592ddb8899a850e75061fc2e4e533e8b149e /chrome/browser/profiles/profile.h | |
| parent | 33a85f181bf099afe9067d50277ea0e85e35f302 (diff) | |
| download | chromium_src-18280378d78cc83d8e83de21f28411cb4fa777bc.zip chromium_src-18280378d78cc83d8e83de21f28411cb4fa777bc.tar.gz chromium_src-18280378d78cc83d8e83de21f28411cb4fa777bc.tar.bz2 | |
Profile shouldn't own BrowserThemeProviders; A ThemeService should and should give back a BTP when presented with a Profile.
This removes all Profile methods that deal with themes. A profile is now just the key that ThemeService maps to a BrowserThemeProvider. Any user of a BrowserThemeProvider asks ThemeService::GetForProfile() instead of the now deleted Profile::GetThemeProvider() method.
Why do this? Because this will drastically reduce coupling. Right now, the Profile knows about pretty much every major object in chrome/browser/. This should allow easier compile time removal of certain systems.
BUG=profile effort
TEST=existing tests
Review URL: http://codereview.chromium.org/6714003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79001 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/profiles/profile.h')
| -rw-r--r-- | chrome/browser/profiles/profile.h | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/chrome/browser/profiles/profile.h b/chrome/browser/profiles/profile.h index 236e0e5..35f8e5488 100644 --- a/chrome/browser/profiles/profile.h +++ b/chrome/browser/profiles/profile.h @@ -315,25 +315,6 @@ class Profile { // Returns the BrowserSignin object assigned to this profile. virtual BrowserSignin* GetBrowserSignin() = 0; - // Init our themes system. - virtual void InitThemes() = 0; - - // Set the theme to the specified extension. - virtual void SetTheme(const Extension* extension) = 0; - - // Set the theme to the machine's native theme. - virtual void SetNativeTheme() = 0; - - // Clear the theme and reset it to default. - virtual void ClearTheme() = 0; - - // Gets the theme that was last set. Returns NULL if the theme is no longer - // installed, if there is no installed theme, or the theme was cleared. - virtual const Extension* GetTheme() = 0; - - // Returns or creates the ThemeProvider associated with this profile - virtual BrowserThemeProvider* GetThemeProvider() = 0; - // Returns the request context information associated with this profile. Call // this only on the UI thread, since it can send notifications that should // happen on the UI thread. |