Desktop Capture Picker Window New UI For Mac

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

Index: chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.h
diff --git a/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.h b/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.h
index f0931e07113c5d358128055d01ebe450e6dda6d2..775e80e9e6359bd119a8e29fd1ebc76c6efdb7e3 100644
--- a/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.h
+++ b/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.h
@@ -20,11 +20,20 @@
 // A controller for the Desktop Media Picker. Presents the user with a list of
 // media sources for screen-capturing, and reports the result.
 @interface DesktopMediaPickerController
-    : NSWindowController<NSWindowDelegate, DesktopMediaPickerObserver> {
+    : NSWindowController<NSWindowDelegate,
+                         DesktopMediaPickerObserver,
+                         NSTableViewDataSource,
+                         NSTableViewDelegate> {
  @private
-  // The image browser view to present sources to the user (thumbnails and
-  // names).
-  base::scoped_nsobject<IKImageBrowserView> sourceBrowser_;
+  // The image browser view or table view to present sources to the user
+  // (thumbnails and names).
+  base::scoped_nsobject<IKImageBrowserView> screenBrowser_;
+  base::scoped_nsobject<IKImageBrowserView> windowBrowser_;
+  base::scoped_nsobject<NSTableView> tabBrowser_;
+
+  base::scoped_nsobject<NSScrollView> imageBrowserScroll_;
+
+  base::scoped_nsobject<NSSegmentedControl> sourceTypeControl_;
 
   // The button used to confirm the selection.
   NSButton* shareButton_;  // weak; owned by contentView
@@ -37,18 +46,24 @@
   base::scoped_nsobject<NSButton> audioShareCheckbox_;
   NSCellStateValue audioShareState_;
 
-  // Provides source information (including thumbnails) to fill up |items_| and
-  // to render in |sourceBrowser_|.
-  std::unique_ptr<DesktopMediaList> media_list_;
+  // Provides source information (including thumbnails) to fill up the array of
+  // |screen_items_|, |window_items_| and |tab_items_|, and to render in
+  // |screenBrowser_|, |windowBrowser_| and |tabBrowser_|.
+  std::unique_ptr<DesktopMediaList> screen_list_;

[tapted, 2016/06/20 12:18:34]

These need to be screenList_ naming style. Same with the rest

[qiangchen, 2016/06/21 23:31:26]

Done.

+  std::unique_ptr<DesktopMediaList> window_list_;
+  std::unique_ptr<DesktopMediaList> tab_list_;
 
   // To be called with the user selection.
   DesktopMediaPicker::DoneCallback doneCallback_;
 
-  // Array of |DesktopMediaPickerItem| used as data for |sourceBrowser_|.
-  base::scoped_nsobject<NSMutableArray> items_;
+  // Arrays of |DesktopMediaPickerItem| used as data for |screenBrowser|,
+  // |windowBrowser_| and |tabBrowser|.
+  base::scoped_nsobject<NSMutableArray> screen_items_;
+  base::scoped_nsobject<NSMutableArray> window_items_;
+  base::scoped_nsobject<NSMutableArray> tab_items_;
 
-  // C++ bridge to use as an observer to |media_list_|, that forwards obj-c
-  // notifications to this object.
+  // C++ bridge to use as an observer to |screen_list_|, |window_list_| and
+  // |tab_list_|, that forwards obj-c notifications to this object.
   std::unique_ptr<DesktopMediaPickerBridge> bridge_;
 
   // Used to create |DesktopMediaPickerItem|s with unique IDs.