Send notifications to off-the-record profile about error bubble being added/removed.

chrome/browser/ui/global_error_service.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/global_error_service.h"
 
 #include <algorithm>
 
 #include "base/stl_util.h"
+#include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/global_error.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "content/public/browser/notification_service.h"
 
 GlobalErrorService::GlobalErrorService(Profile* profile) : profile_(profile) {
 }
 
 GlobalErrorService::~GlobalErrorService() {
   STLDeleteElements(&errors_);
 }
@@ skipping 33 matching lines @@
   for (std::vector<GlobalError*>::const_iterator
        it = errors_.begin(); it != errors_.end(); ++it) {
     GlobalError* error = *it;
     if (error->HasBubbleView() && !error->HasShownBubbleView())
       return error;
   }
   return NULL;
 }
 
 void GlobalErrorService::NotifyErrorsChanged(GlobalError* error) {
-  content::NotificationService::current()->Notify(
-      chrome::NOTIFICATION_GLOBAL_ERRORS_CHANGED,
-      content::Source<Profile>(profile_),
-      content::Details<GlobalError>(error));
+  std::vector<Profile*> profiles_to_notify;

[sail, 2011/12/08 17:44:45]

can you add a DCHECK() for ServiceRedirectedInIncognito().

Also can you add a comment that you need to broadcast 2 set of notifications so
that both normal windows and incognito windows get updated.

[whywhat, 2011/12/09 11:14:33]

I've added comment. How can I check ServiceRedirectedInIncognito here if it's
private in the factory class?

On 2011/12/08 17:44:45, sail wrote:

[sail, 2011/12/09 17:43:51]

Ahh damn, never mind.

+  if (profile_ != NULL) {
+    profiles_to_notify.push_back(profile_);
+    if (profile_->IsOffTheRecord())
+      profiles_to_notify.push_back(profile_->GetOriginalProfile());
+    else if (profile_->HasOffTheRecordProfile())
+      profiles_to_notify.push_back(profile_->GetOffTheRecordProfile());
+    for (size_t i = 0; i < profiles_to_notify.size(); ++i) {
+      content::NotificationService::current()->Notify(
+        chrome::NOTIFICATION_GLOBAL_ERRORS_CHANGED,
+        content::Source<Profile>(profiles_to_notify[i]),
+        content::Details<GlobalError>(error));
+    }
+  }
 }