Desktop chrome tab capture-chooseDesktopMedia()

chrome/browser/ui/views/desktop_media_picker_views.cc

 // Copyright 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.
 
 #include "chrome/browser/ui/views/desktop_media_picker_views.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/callback.h"
@@ skipping 505 matching lines @@
 bool DesktopMediaPickerDialogView::Accept() {
   DesktopMediaSourceView* selection = list_view_->GetSelection();
 
   // Ok button should only be enabled when a source is selected.
   DCHECK(selection);
 
   DesktopMediaID source;
   if (selection)
     source = selection->source_id();
 
-  if (parent_)
+  // Temporarily disable tab capture. This will be enabled later.

[Avi (use Gerrit), 2016/01/11 16:27:57]

"enabled later" meaning:
1) It's in the code, and will happen later in the flow, or
2) It will be added in a later CL?

If 1, OK, but if 2 then turn this into a TODO.

[GeorgeZ, 2016/01/11 19:16:19]

It is 2. Therefore I use TODO.

+  if (parent_ && source.type != DesktopMediaID::TYPE_TAB)
     parent_->NotifyDialogResult(source);
 
   // Return true to close the window.
   return true;
 }
 
 void DesktopMediaPickerDialogView::DeleteDelegate() {
   // If the dialog is being closed then notify the parent about it.
   if (parent_)
     parent_->NotifyDialogResult(DesktopMediaID());
@@ skipping 66 matching lines @@
   content::BrowserThread::PostTask(
       content::BrowserThread::UI, FROM_HERE,
       base::Bind(callback_, source));
   callback_.Reset();
 }
 
 // static
 scoped_ptr<DesktopMediaPicker> DesktopMediaPicker::Create() {
   return scoped_ptr<DesktopMediaPicker>(new DesktopMediaPickerViews());
 }