Implement Desktop Media Picker (Mac version) for Desktop Capture API

chrome/browser/ui/cocoa/media_picker/desktop_media_picker_cocoa.h

Index: chrome/browser/ui/cocoa/media_picker/desktop_media_picker_cocoa.h
diff --git a/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_cocoa.h b/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_cocoa.h
new file mode 100644
index 0000000000000000000000000000000000000000..cd75dc961d1e2843bb760aa4324e447bc3ee2f12
--- /dev/null
+++ b/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_cocoa.h
@@ -0,0 +1,30 @@
+// Copyright (c) 2013 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_UI_COCOA_MEDIA_PICKER_DESKTOP_MEDIA_PICKER_COCOA_H_
+#define CHROME_BROWSER_UI_COCOA_MEDIA_PICKER_DESKTOP_MEDIA_PICKER_COCOA_H_
+
+#include "base/mac/scoped_nsobject.h"

[Robert Sesek, 2013/09/04 20:23:19]

Use #import for ObjC headers

[dcaiafa, 2013/09/04 21:39:35]

Done.

+#include "chrome/browser/media/desktop_media_picker.h"
+
+@class DesktopMediaPickerController;
+
+// Cocoa's DesktopMediaPicker implementation.
+class DesktopMediaPickerCocoa : public DesktopMediaPicker {
+ public:
+  DesktopMediaPickerCocoa();
+  virtual ~DesktopMediaPickerCocoa();
+
+  // Overridden from DesktopMediaPicker:
+  virtual void Show(gfx::NativeWindow context,
+                    gfx::NativeWindow parent,
+                    const string16& app_name,
+                    scoped_ptr<DesktopMediaPickerModel> model,
+                    const DoneCallback& done_callback) OVERRIDE;
+
+ private:
+  base::scoped_nsobject<DesktopMediaPickerController> controller_;

[Robert Sesek, 2013/09/04 20:23:19]

We usually have controllers owning themselves. How does this get torn down?

[dcaiafa, 2013/09/04 21:39:35]

The Desktop Media Picker API will destroy the DesktopMediaPicker when the
callback is received or when the operation is cancelled (e.g. the app/extension
is closed). The latter is the main reason for ownership.

On 2013/09/04 20:23:19, rsesek wrote:

+};
+
+#endif  // CHROME_BROWSER_UI_COCOA_MEDIA_PICKER_DESKTOP_MEDIA_PICKER_COCOA_H_