Desktop chrome tab capture-chooseDesktopMedia()

chrome/browser/media/native_desktop_media_list_unittest.cc

Index: chrome/browser/media/native_desktop_media_list_unittest.cc
diff --git a/chrome/browser/media/native_desktop_media_list_unittest.cc b/chrome/browser/media/native_desktop_media_list_unittest.cc
index a5925e979fdafc55f220c2038c904da97c41e671..80bc63b7632c8b201e35282ed91c6108010f9931 100644
--- a/chrome/browser/media/native_desktop_media_list_unittest.cc
+++ b/chrome/browser/media/native_desktop_media_list_unittest.cc
@@ -149,7 +149,7 @@ class DesktopMediaListTest : public testing::Test {
     window_capturer_ = new FakeWindowCapturer();
     model_.reset(new NativeDesktopMediaList(
         scoped_ptr<webrtc::ScreenCapturer>(new FakeScreenCapturer()),
-        scoped_ptr<webrtc::WindowCapturer>(window_capturer_)));
+        scoped_ptr<webrtc::WindowCapturer>(window_capturer_), false));
 
     // Set update period to reduce the time it takes to run tests.
     model_->SetUpdatePeriod(base::TimeDelta::FromMilliseconds(0));
@@ -231,14 +231,14 @@ TEST_F(DesktopMediaListTest, WindowsOnly) {
   window_capturer_ = new FakeWindowCapturer();
   model_.reset(new NativeDesktopMediaList(
       scoped_ptr<webrtc::ScreenCapturer>(),
-      scoped_ptr<webrtc::WindowCapturer>(window_capturer_)));
+      scoped_ptr<webrtc::WindowCapturer>(window_capturer_), false));
   AddWindowsAndVerify(1, true);
 }
 
 TEST_F(DesktopMediaListTest, ScreenOnly) {
   model_.reset(new NativeDesktopMediaList(
       scoped_ptr<webrtc::ScreenCapturer>(new FakeScreenCapturer),
-      scoped_ptr<webrtc::WindowCapturer>()));
+      scoped_ptr<webrtc::WindowCapturer>(), false));

[qiangchen, 2015/12/07 22:45:56]

Consider adding a test case say "TabOnly"

[GeorgeZ, 2015/12/09 19:36:38]

Need modify unit tests desktop_meida_list_ash_uninttest.cc and
native_desktop_meida_list_uninttest.cc and associated desktop_meida_list_ash.cc
and native_desktop_meida_list.cc to cover tab cases.

How about that I create another CL to cover units test to limit the scope of
this CL?

 
   {
     testing::InSequence dummy;