Add a receiver interface to RemovableStorageNotifications.

chrome/browser/system_monitor/media_transfer_protocol_device_observer_linux.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_SYSTEM_MONITOR_MEDIA_TRANSFER_PROTOCOL_DEVICE_OBSERVER_LINUX_H_
 #define CHROME_BROWSER_SYSTEM_MONITOR_MEDIA_TRANSFER_PROTOCOL_DEVICE_OBSERVER_LINUX_H_
 
 #include <map>
 #include <string>
 
@@ skipping 22 matching lines @@
   virtual ~MediaTransferProtocolDeviceObserverLinux();
 
   static MediaTransferProtocolDeviceObserverLinux* GetInstance();
 
   // Finds the storage that contains |path| and populates |storage_info|.
   // Returns false if unable to find the storage.
   bool GetStorageInfoForPath(
       const FilePath& path,
       RemovableStorageNotifications::StorageInfo* storage_info) const;
 
+  // Set the volume notifications object to be used when new
+  // MTP devices are found.
+  void SetNotifications(RemovableStorageNotifications::Receiver* notifications);

[vandebo (ex-Chrome), 2013/01/31 23:51:29]

Pass in constructor?

[Greg Billock, 2013/02/01 18:28:42]

see above.

On 2013/01/31 23:51:29, vandebo wrote:

+
  protected:
   // Only used in unit tests.
   explicit MediaTransferProtocolDeviceObserverLinux(
       GetStorageInfoFunc get_storage_info_func);
 
   // device::MediaTransferProtocolManager::Observer implementation.
   // Exposed for unit tests.
   virtual void StorageChanged(bool is_attached,
                               const std::string& storage_name) OVERRIDE;
 
  private:
   // Mapping of storage location and mtp storage info object.
   typedef std::map<std::string, RemovableStorageNotifications::StorageInfo>
       StorageLocationToInfoMap;
 
   // Enumerate existing mtp storage devices.
   void EnumerateStorages();
 
   // Map of all attached mtp devices.
   StorageLocationToInfoMap storage_map_;
 
   // Function handler to get storage information. This is useful to set a mock
   // handler for unit testing.
   GetStorageInfoFunc get_storage_info_func_;
 
+  // The notifications object to use to signal newly attached devices.

[vandebo (ex-Chrome), 2013/01/31 23:51:29]

Note lifetime and todo to update when it changes.

[Greg Billock, 2013/02/01 18:28:43]

Done.

+  RemovableStorageNotifications::Receiver* notifications_;
+
   DISALLOW_COPY_AND_ASSIGN(MediaTransferProtocolDeviceObserverLinux);
 };
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_SYSTEM_MONITOR_MEDIA_TRANSFER_PROTOCOL_DEVICE_OBSERVER_LINUX_H_