Desktop Capture Picker New UI: Preliminary Refactor

chrome/browser/media/desktop_media_picker.h

Index: chrome/browser/media/desktop_media_picker.h
diff --git a/chrome/browser/media/desktop_media_picker.h b/chrome/browser/media/desktop_media_picker.h
index cb144c5f4a23a6d846054e4817dbb5506b29a0d7..3312037f8d11df900ff92cc262a87648d4f1e506 100644
--- a/chrome/browser/media/desktop_media_picker.h
+++ b/chrome/browser/media/desktop_media_picker.h
@@ -33,14 +33,17 @@ class DesktopMediaPicker {
   virtual ~DesktopMediaPicker() {}
 
   // Shows dialog with list of desktop media sources (screens, windows, tabs)
-  // provided by |list| and calls |done_callback| when user chooses one of the
+  // provided by |screen_list|, |window_list| and |tab_list|.
+  // Dialog window will call |done_callback| when user chooses one of the
   // sources or closes the dialog.
   virtual void Show(content::WebContents* web_contents,
                     gfx::NativeWindow context,
                     gfx::NativeWindow parent,
                     const base::string16& app_name,
                     const base::string16& target_name,
-                    std::unique_ptr<DesktopMediaList> list,
+                    std::unique_ptr<DesktopMediaList> screen_list,
+                    std::unique_ptr<DesktopMediaList> window_list,
+                    std::unique_ptr<DesktopMediaList> tab_list,
                     bool request_audio,

[Sergey Ulanov, 2016/04/14 22:39:52]

nit, not for this cl: there 4 parameters can be bundled together in a struct.

[qiangchen, 2016/04/15 19:33:30]

Good suggestion.

                     const DoneCallback& done_callback) = 0;