Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #ifndef CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MANAGER_H_ | 5 #ifndef CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MANAGER_H_ |
| 6 #define CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MANAGER_H_ | 6 #define CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MANAGER_H_ |
| 7 | 7 |
| 8 #import <Foundation/Foundation.h> | 8 #import <Foundation/Foundation.h> |
| 9 #include <string> | 9 #include <string> |
| 10 | 10 |
| (...skipping 15 matching lines...) Expand all Loading... | |
| 26 ~ImageCaptureDeviceManager(); | 26 ~ImageCaptureDeviceManager(); |
| 27 | 27 |
| 28 // The UUIDs passed here are available in the device attach notifications | 28 // The UUIDs passed here are available in the device attach notifications |
| 29 // given through SystemMonitor. They're gotten by cracking the device ID | 29 // given through SystemMonitor. They're gotten by cracking the device ID |
| 30 // and taking the unique ID output. | 30 // and taking the unique ID output. |
| 31 static ImageCaptureDevice* deviceForUUID(const std::string& uuid); | 31 static ImageCaptureDevice* deviceForUUID(const std::string& uuid); |
| 32 | 32 |
| 33 // Returns a weak pointer to the internal ImageCapture interface protocol. | 33 // Returns a weak pointer to the internal ImageCapture interface protocol. |
| 34 id<ICDeviceBrowserDelegate> device_browser(); | 34 id<ICDeviceBrowserDelegate> device_browser(); |
| 35 | 35 |
| 36 // Sets the receiver for device attach/detach notifications. | |
| 37 void SetNotifications(RemovableStorageNotifications::Receiver* notifications); | |
|
vandebo (ex-Chrome)
2013/01/31 23:51:29
Why not pass this in to the constructor?
Greg Billock
2013/02/01 18:28:42
They should be. I used these setters because of th
vandebo (ex-Chrome)
2013/02/01 18:59:55
Ok, can you add a TODO here then, so it's clear th
Greg Billock
2013/02/01 23:05:42
Done.
| |
| 38 | |
| 36 private: | 39 private: |
| 37 scoped_nsobject<ImageCaptureDeviceManagerImpl> device_browser_; | 40 scoped_nsobject<ImageCaptureDeviceManagerImpl> device_browser_; |
| 38 }; | 41 }; |
| 39 | 42 |
| 40 } // namespace chrome | 43 } // namespace chrome |
| 41 | 44 |
| 42 #endif // CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MANAGER_H_ | 45 #endif // CHROME_BROWSER_SYSTEM_MONITOR_IMAGE_CAPTURE_DEVICE_MANAGER_H_ |
| OLD | NEW |