Add scheme exceptions for isSecureContext

chrome/browser/extensions/api/webrtc_from_web_accessible_resource_browsertest.cc

+// Copyright 2015 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 "base/command_line.h"
+#include "chrome/browser/extensions/extension_apitest.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/browser/ui/website_settings/permission_bubble_manager.h"
+#include "chrome/common/chrome_switches.h"
+#include "chrome/test/base/ui_test_utils.h"
+#include "content/public/common/content_switches.h"
+#include "media/base/media_switches.h"
+#include "net/dns/mock_host_resolver.h"
+#include "net/test/embedded_test_server/embedded_test_server.h"
+
+namespace extensions {
+
+namespace {
+
+// Used to observe the creation of permission prompt without responding.
+class PermissionRequestObserver : public PermissionBubbleManager::Observer {
+ public:
+  explicit PermissionRequestObserver(content::WebContents* web_contents)
+      : bubble_manager_(PermissionBubbleManager::FromWebContents(web_contents)),
+        request_shown_(false),
+        message_loop_runner_(new content::MessageLoopRunner) {
+    bubble_manager_->AddObserver(this);
+  }
+  ~PermissionRequestObserver() override {
+    // Safe to remove twice if it happens.
+    bubble_manager_->RemoveObserver(this);
+  }
+
+  void Wait() { message_loop_runner_->Run(); }
+
+  bool request_shown() const { return request_shown_; }
+
+ private:
+  // PermissionBubbleManager::Observer
+  void OnBubbleAdded() override {
+    request_shown_ = true;
+    bubble_manager_->RemoveObserver(this);
+    message_loop_runner_->Quit();
+  }
+
+  PermissionBubbleManager* bubble_manager_;
+  bool request_shown_;
+  scoped_refptr<content::MessageLoopRunner> message_loop_runner_;
+
+  DISALLOW_COPY_AND_ASSIGN(PermissionRequestObserver);
+};
+
+}  // namespace
+
+class WebRtcFromWebAccessibleResourceTest : public ExtensionApiTest {
+ public:
+  WebRtcFromWebAccessibleResourceTest() {}
+  ~WebRtcFromWebAccessibleResourceTest() override {}
+
+  // InProcessBrowserTest:
+  void SetUpCommandLine(base::CommandLine* command_line) override {
+    ExtensionApiTest::SetUpCommandLine(command_line);
+
+    // This test expects to run with fake devices and fake UI.
+    command_line->AppendSwitch(switches::kUseFakeDeviceForMediaStream);
+    command_line->AppendSwitch(switches::kUseFakeUIForMediaStream);
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(WebRtcFromWebAccessibleResourceTest);
+};
+
+// Flaky on Windows: http://crbug.com/301887

[Devlin, 2015/10/08 16:48:32]

Why are newly-added tests already flaky? :)

[jww, 2015/10/09 21:39:26]

Whoops, leftovers from when I first started trying to build off the
desktop_capture_apitest. Removing the flaky comment and macros.

+#if defined(OS_WIN)
+#define MAYBE_GetUserMediaInWebAccessibleResource \
+  DISABLED_GetUserMediaInWebAccessibleResource
+#else
+#define MAYBE_GetUserMediaInWebAccessibleResource \
+  GetUserMediaInWebAccessibleResource
+#endif
+IN_PROC_BROWSER_TEST_F(WebRtcFromWebAccessibleResourceTest,
+                       MAYBE_GetUserMediaInWebAccessibleResource) {
+  host_resolver()->AddRule("a.com", "127.0.0.1");
+  ASSERT_TRUE(StartEmbeddedTestServer());
+
+  content::WebContents* web_contents =
+      browser()->tab_strip_model()->GetActiveWebContents();
+  PermissionBubbleManager::FromWebContents(web_contents)
+      ->set_auto_response_for_test(PermissionBubbleManager::ACCEPT_ALL);
+  ASSERT_TRUE(RunExtensionTest("webrtc_from_web_accessible_resource"))

[Devlin, 2015/10/08 16:48:32]

For funzies, we should also add
PermissionBubbleManager::FromWebContents(web_contents)
    ->set_auto_response_for_test(PermissionBubbleManager::REJECT_ALL);
ASSERT_FALSE(RunExtensionTest("webrtc_from_web_accessible_resource"));

To make sure that if the user refuses, it doesn't get access.

(This probably required clearing the permissions for the web contents, or
navigating to a different page.)

[jww, 2015/10/09 21:39:26]

Good call. Added.

+      << message_;
+}
+
+}  // namespace extensions