Add unit test for WebRTCInternalsMessageHandler.

content/browser/media/webrtc/webrtc_internals_message_handler_unittest.cc

+// Copyright (c) 2016 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 "content/browser/media/webrtc/webrtc_internals_message_handler.h"
+
+#include <memory>
+#include <string>
+
+#include "base/run_loop.h"
+#include "content/browser/child_process_security_policy_impl.h"
+#include "content/browser/media/webrtc/webrtc_internals.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/common/url_constants.h"
+#include "content/public/test/test_web_ui.h"
+#include "content/test/test_web_contents.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace content {
+
+namespace {
+
+static const char kConstraints[] = "c";
+static const char kRtcConfiguration[] = "r";
+static const char kUrl[] = "u";
+
+class WebRTCInternalsMessageHandlerForTest
+    : public WebRTCInternalsMessageHandler {
+ public:
+  WebRTCInternalsMessageHandlerForTest(WebRTCInternals* webrtc_internals,
+                                       WebUI* web_UI)
+      : WebRTCInternalsMessageHandler(webrtc_internals) {
+    set_web_ui(web_UI);
+  }
+
+  ~WebRTCInternalsMessageHandlerForTest() override {}
+};
+class WebRTCInternalsForTest : public NON_EXPORTED_BASE(WebRTCInternals) {

[tommi (sloooow) - chröme, 2016/07/25 10:16:06]

nit: empty line above this one

[Max Morin, 2016/07/25 10:41:18]

Done.

+ public:
+  WebRTCInternalsForTest() : WebRTCInternals(0, false) {}
+  ~WebRTCInternalsForTest() override {}
+};
+
+}  // namespace
+
+class WebRtcInternalsMessageHandlerTest : public RenderViewHostTestHarness {
+ public:
+  const GURL kWebRTCURL =

[tommi (sloooow) - chröme, 2016/07/25 10:16:06]

either make this static or rename to webrtc_url_

[Max Morin, 2016/07/25 10:41:18]

Done.

+      GURL(std::string("chrome://") + kChromeUIWebRTCInternalsHost);
+  const GURL kExampleURL = GURL("http://www.example.com/");

[tommi (sloooow) - chröme, 2016/07/25 10:16:06]

same here.

Also, for initializing these variables, please just use the constructor instead
of the assignment operator

[Max Morin, 2016/07/25 10:41:18]

Done.

+  WebRtcInternalsMessageHandlerTest() {}
+
+ protected:
+  void SetUp() override {
+    RenderViewHostTestHarness::SetUp();
+    web_UI.reset(new TestWebUI());
+    web_UI->set_web_contents(web_contents());
+  }
+
+  void TearDown() override {
+    web_UI->set_web_contents(nullptr);
+    web_UI.reset();
+    RenderViewHostTestHarness::TearDown();
+  }
+
+  std::unique_ptr<TestWebUI> web_UI;

[tommi (sloooow) - chröme, 2016/07/25 10:16:06]

web_ui_

[Max Morin, 2016/07/25 10:41:18]

Done.

+};
+
+TEST_F(WebRtcInternalsMessageHandlerTest, DontRunJSBeforeNavigationCommitted) {
+  WebRTCInternalsForTest webrtc_internals;
+  WebRTCInternalsMessageHandlerForTest observer(&webrtc_internals,
+                                                web_UI.get());
+
+  NavigateAndCommit(kExampleURL);
+  webrtc_internals.OnAddPeerConnection(0, 1, 2, kUrl, kRtcConfiguration,
+                                       kConstraints);
+  base::RunLoop().RunUntilIdle();
+
+  static_cast<TestWebContents*>(web_contents())->StartNavigation(kWebRTCURL);
+  // We still shouldn't run JS, since navigation to webrtc-internals isn't
+  // finished.
+  webrtc_internals.OnRemovePeerConnection(1, 2);
+  base::RunLoop().RunUntilIdle();
+
+  webrtc_internals.RemoveObserver(&observer);
+}
+
+}  // namespace content