More.

chrome/browser/ui/cocoa/browser_window_cocoa.mm

Index: chrome/browser/ui/cocoa/browser_window_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.mm b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
index 2c63978f0eacceee23a3e2d62925a6410ba6bd53..292e1b437c0ba7dec52c0f501792c348358bfbb6 100644
--- a/chrome/browser/ui/cocoa/browser_window_cocoa.mm
+++ b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
@@ -57,7 +57,7 @@ BrowserWindowCocoa::BrowserWindowCocoa(Browser* browser,
                  chrome::NOTIFICATION_BOOKMARK_BAR_VISIBILITY_PREF_CHANGED,
                  NotificationService::AllSources());

[Peter Kasting, 2011/08/10 21:03:21]

Don't you want to change this (and the observer code) similarly to browser.cc?

[Elliot Glaysher, 2011/08/10 21:19:23]

I'm doing this in a per-notification way instead of per-file. (See our tracking
spreadsheet
https://docs.google.com/a/google.com/spreadsheet/ccc?key=0Ar4V_eAZl8_SdDB2THV...
) 

   registrar_.Add(this, chrome::NOTIFICATION_SIDEBAR_CHANGED,
-                 NotificationService::AllSources());
+                 Source<SidebarManager>(SidebarManager::GetInstance()));

[Peter Kasting, 2011/08/10 21:03:21]

Shouldn't the sidebar state be synced between normal and incognito profiles?  Or
am I mistaken about what the sidebar is...

[Elliot Glaysher, 2011/08/10 21:19:23]

The SidebarManager is currently a Singleton. It maps a TabContents to a Sidebar
object. This is just a case where it's easy to remove the AllSources() call and
listen directly on the single object that we're interested in.

(I believe this is an extensions feature. This is an extension API, not the side
tab stuff.)

 }
 
 BrowserWindowCocoa::~BrowserWindowCocoa() {