Fix AllSources observers for EXTENSION_READY_TO_INSTALL and BROWSER_ACTION_VISIBILITY_CHANGED.

Make NOTIFICATION_EXTENSION_READY_FOR_INSTALL sent from the Profile.  We need this because the profile can't be gotten from the DownloadManager, and the notification can be sent from the incognito profile.

BUG=94034
TEST=existing test


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

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

1 files changed, 13 insertions, 11 deletions

diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index b21a519e..0b10414 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -4035,17 +4035,19 @@ void Browser::Observe(int type,
       break;
 
     case chrome::NOTIFICATION_EXTENSION_READY_FOR_INSTALL: {
-      // Handle EXTENSION_READY_FOR_INSTALL for last active tabbed browser.
-      if (BrowserList::FindTabbedBrowser(profile(), true) != this)
-        break;
-
-      // We only want to show the loading dialog for themes, but we don't want
-      // to wait until unpack to find out an extension is a theme, so we test
-      // the download_url GURL instead. This means that themes in the extensions
-      // gallery won't get the loading dialog.
-      GURL download_url = *(Details<GURL>(details).ptr());
-      if (ExtensionService::IsDownloadFromMiniGallery(download_url))
-        window()->ShowThemeInstallBubble();
+      Profile* profile = Source<Profile>(source).ptr();
+      if (profile_->IsSameProfile(profile)) {
+        // Handle EXTENSION_READY_FOR_INSTALL for last active tabbed browser.
+        if (BrowserList::FindTabbedBrowser(profile, true) == this) {
+          // We only want to show the loading dialog for themes, but we don't
+          // want to wait until unpack to find out an extension is a theme, so
+          // we test the download_url GURL instead. This means that themes in
+          // the extensions gallery won't get the loading dialog.
+          GURL download_url = *(Details<GURL>(details).ptr());
+          if (ExtensionService::IsDownloadFromMiniGallery(download_url))
+            window()->ShowThemeInstallBubble();
+        }
+      }
       break;
     }