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

chrome/browser/chrome_browser_main_mac.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_CHROME_BROWSER_MAIN_MAC_H_
 #define CHROME_BROWSER_CHROME_BROWSER_MAIN_MAC_H_
 
 #include "chrome/browser/chrome_browser_main_posix.h"
 #include "base/memory/ref_counted.h"
 
@@ skipping 13 matching lines @@
   virtual void PreProfileInit() OVERRIDE;
 
   // Perform platform-specific work that needs to be done after the main event
   // loop has ended. The embedder must be sure to call this.
   static void DidEndMainMessageLoop();
 
  private:
   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);
 };
 
 #endif  // CHROME_BROWSER_CHROME_BROWSER_MAIN_MAC_H_