[Media Galleries] Add an ImageCaptureCore listener for Mac. (part 2)

chrome/browser/chrome_browser_main_mac.h

Index: chrome/browser/chrome_browser_main_mac.h
diff --git a/chrome/browser/chrome_browser_main_mac.h b/chrome/browser/chrome_browser_main_mac.h
index 46388d887b06bcdaab27845b741cda91e506f203..616932ab448321223571c95dc6d18e1a78ad21da 100644
--- a/chrome/browser/chrome_browser_main_mac.h
+++ b/chrome/browser/chrome_browser_main_mac.h
@@ -31,6 +31,11 @@ class ChromeBrowserMainPartsMac : public ChromeBrowserMainPartsPosix {
   scoped_refptr<chrome::RemovableDeviceNotificationsMac>
       removable_device_notifications_mac_;
 
+  class Inner;
+
+  // Hide scoped_nsobject members from C++.
+  scoped_ptr<Inner> inner_;

[sail, 2012/12/12 20:53:20]

I think a better way to do this would be to expose a C++ class in
chrome/browser/system_monitor. Make the Objective-C class internal to the
implementation.

[Greg Billock, 2012/12/13 00:31:58]

I've sketched that out separately -- I think we actually need to move these
notifications out of SystemMonitor. They aren't base:: namespaced notifications;
you have to know chrome:: code to handle them, so they don't belong in base::.

I tried working SystemMonitor into something that maintained list of
notification types, but it ends up being complex and relatively pointless -- why
not just get notifications from the sources themselves?

I think the best option is for things these BrowserMainParts use should be
available in BrowserContext. But that has its own problems....

+
   DISALLOW_COPY_AND_ASSIGN(ChromeBrowserMainPartsMac);
 };