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

Unified Diff: chrome/browser/themes/theme_service_factory.h

Issue 11416324: Revert changes to have ExtensionService notify ThemeService directly (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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/themes/theme_service.cc ('k') | chrome/browser/themes/theme_service_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/themes/theme_service_factory.h
diff --git a/chrome/browser/themes/theme_service_factory.h b/chrome/browser/themes/theme_service_factory.h
index f74f167ff2a0fe4869e3054c1bb63fb5b5002306..be7b956843ac4ebbd01416e7ca9ce9efd1400c98 100644
--- a/chrome/browser/themes/theme_service_factory.h
+++ b/chrome/browser/themes/theme_service_factory.h
@@ -26,15 +26,10 @@ class ThemeServiceFactory : public ProfileKeyedServiceFactory {
// still needs a ThemeService to hand back the default theme images.
static ThemeService* GetForProfile(Profile* profile);
- // If the theme service for |profile| has been instantiated,
- // immediately sets its theme to |theme|. Otherwise, saves
- // |theme|'s ID so that the theme service picks it up when it gets
- // initialized.
- //
- // |theme| must already be installed in the extension service.
- static void SetThemeForProfile(
- Profile* profile,
- const extensions::Extension* theme);
+ // Returns the Extension that implements the theme associated with
+ // |profile|. Returns NULL if the theme is no longer installed, if there is
+ // no installed theme, or the theme was cleared.
+ static const extensions::Extension* GetThemeForProfile(Profile* profile);
static ThemeServiceFactory* GetInstance();
« no previous file with comments | « chrome/browser/themes/theme_service.cc ('k') | chrome/browser/themes/theme_service_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698