[Sync] Simplify themes notifications and themes sync

Change ThemesService to listen to EXTENSION_LOADED instead of
THEME_INSTALLED.  Remove THEME_INSTALLED notification and always send
EXTENSION_INSTALLED.

Remove Extension details from BROWSER_THEME_CHANGED notification.

The two changes above make it so that listeners to BROWSER_THEME_CHANGED
can simply query the ThemeService about the new theme; previously this
was not possible since the theme was only installed and not loaded.

Cut out the now-unneeded code in ThemeChangeProcessor (yay!).

BUG=74831
TEST=

Review URL: http://codereview.chromium.org/6874001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81795 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 4 deletions

diff --git a/chrome/browser/ui/gtk/gtk_theme_service.cc b/chrome/browser/ui/gtk/gtk_theme_service.cc
index df6d913..8e433d6 100644
--- a/chrome/browser/ui/gtk/gtk_theme_service.cc
+++ b/chrome/browser/ui/gtk/gtk_theme_service.cc
@@ -348,7 +348,7 @@ void GtkThemeService::SetNativeTheme() {
   profile()->GetPrefs()->SetBoolean(prefs::kUsesSystemTheme, true);
   ClearAllThemeData();
   LoadGtkValues();
-  NotifyThemeChanged(NULL);
+  NotifyThemeChanged();
 }
 
 bool GtkThemeService::UsingDefaultTheme() {
@@ -631,8 +631,8 @@ void GtkThemeService::LoadThemePrefs() {
   RebuildMenuIconSets();
 }
 
-void GtkThemeService::NotifyThemeChanged(const Extension* extension) {
-  ThemeService::NotifyThemeChanged(extension);
+void GtkThemeService::NotifyThemeChanged() {
+  ThemeService::NotifyThemeChanged();
 
   // Notify all GtkChromeButtons of their new rendering mode:
   for (std::vector<GtkWidget*>::iterator it = chrome_buttons_.begin();
@@ -664,7 +664,7 @@ void GtkThemeService::OnStyleSet(GtkWidget* widget,
   if (profile()->GetPrefs()->GetBoolean(prefs::kUsesSystemTheme)) {
     ClearAllThemeData();
     LoadGtkValues();
-    NotifyThemeChanged(NULL);
+    NotifyThemeChanged();
   }
 
   RebuildMenuIconSets();