getUserMedia: handle the device starting status report.

chrome/browser/extensions/api/desktop_capture/desktop_capture_apitest.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 <array>
 
 #include "base/command_line.h"
 #include "base/macros.h"
 #include "base/path_service.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 188 matching lines @@
       // chooseMediaAndGetStream()
       {true, true, false, false,
        content::DesktopMediaID(content::DesktopMediaID::TYPE_SCREEN,
                                webrtc::kFullDesktopScreenId)},
       // chooseMediaAndTryGetStreamWithInvalidId()
       {true, true, false, false,
        content::DesktopMediaID(content::DesktopMediaID::TYPE_SCREEN,
                                webrtc::kFullDesktopScreenId)},
       // cancelDialog()
       {true, true, false, false, content::DesktopMediaID(), true},
-      // tabShareWithAudioGetStream()
-      {false, false, true, true,
-       content::DesktopMediaID(content::DesktopMediaID::TYPE_WEB_CONTENTS, 0,
-                               true)},
-      // windowShareWithAudioGetStream()
-      {false, true, false, true,
-       content::DesktopMediaID(content::DesktopMediaID::TYPE_WINDOW, 0, true)},
+
       // screenShareWithAudioGetStream()
       {true, false, false, true,
        content::DesktopMediaID(content::DesktopMediaID::TYPE_SCREEN,
                                webrtc::kFullDesktopScreenId, true)},
-      // tabShareWithoutAudioGetStream()

[miu, 2017/03/06 22:37:33]

I'm worried about these deletions. Instead, can we comment them out; and add a
TODO comment and file a crbug to get them working again? FWICT, either test
hooks are needed to fake-out the chooser dialog UI, or we need to create
interactive_ui_tests for these.

[braveyao, 2017/03/08 22:02:51]

Done.

-      {false, false, true, true,
-       content::DesktopMediaID(content::DesktopMediaID::TYPE_WEB_CONTENTS, 0)},
-      // windowShareWithoutAudioGetStream()
-      {false, true, false, true,
-       content::DesktopMediaID(content::DesktopMediaID::TYPE_WINDOW, 0)},
+
       // screenShareWithoutAudioGetStream()
       {true, false, false, true,
        content::DesktopMediaID(content::DesktopMediaID::TYPE_SCREEN,
                                webrtc::kFullDesktopScreenId)},
   };
   picker_factory_.SetTestFlags(test_flags, arraysize(test_flags));
   ASSERT_TRUE(RunExtensionTest("desktop_capture")) << message_;
 }
 
 // Test is flaky http://crbug.com/301887.
@@ skipping 50 matching lines @@
   EXPECT_TRUE(result);
   EXPECT_TRUE(test_flags[2].picker_created);
   EXPECT_FALSE(test_flags[2].picker_deleted);
 
   web_contents->Close();
   destroyed_watcher.Wait();
   EXPECT_TRUE(test_flags[2].picker_deleted);
 }
 
 }  // namespace extensions