Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(308)

Unified Diff: chrome/test/media_router/media_router_one_ua_integration_browsertest.cc

Issue 2820433003: [Presentation API] Add browser tests for 1-UA mode (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/media_router/media_router_one_ua_integration_browsertest.cc
diff --git a/chrome/test/media_router/media_router_one_ua_integration_browsertest.cc b/chrome/test/media_router/media_router_one_ua_integration_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..aeec2682740f088b1d07b87ae5498f413d595342
--- /dev/null
+++ b/chrome/test/media_router/media_router_one_ua_integration_browsertest.cc
@@ -0,0 +1,96 @@
+// Copyright 2017 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 <memory>
+
+#include "base/files/file_util.h"
+#include "base/path_service.h"
+#include "chrome/test/media_router/media_router_integration_browsertest.h"
+#include "content/public/test/test_utils.h"
+#include "net/base/filename_util.h"
+#include "net/test/embedded_test_server/embedded_test_server.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using content::WebContents;
+
+namespace media_router {
+
+class MediaRouterOneUAIntegrationBrowserTest
+ : public MediaRouterIntegrationBrowserTest {
+ public:
+ void SetUpOnMainThread() override {
+ MediaRouterIntegrationBrowserTest::SetUpOnMainThread();
+
+ base::FilePath base_dir;
+ CHECK(PathService::Get(base::DIR_MODULE, &base_dir));
+ base::FilePath resource_dir = base_dir.Append(
+ FILE_PATH_LITERAL("media_router/browser_test_resources/"));
imcheng 2017/04/17 19:06:44 Please add a comment on why we need this extra set
zhaobin 2017/04/17 21:24:07 Yes.
+ embedded_test_server()->ServeFilesFromDirectory(resource_dir);
+ ASSERT_TRUE(embedded_test_server()->Start());
+ }
+
+ GURL GetTestPageUrl(const base::FilePath& full_path) override {
+ GURL url = embedded_test_server()->GetURL("/basic_test.html");
+ return GURL(url.spec() + "?__oneUA__=true");
+ }
+};
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUAIntegrationBrowserTest, MANUAL_Basic) {
+ RunBasicTest();
+}
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUAIntegrationBrowserTest,
+ MANUAL_SendAndOnMessage) {
+ RunSendMessageTest("foo");
+}
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUAIntegrationBrowserTest,
+ MANUAL_ReceiverCloseConnection) {
+ RunSendMessageTest("close");
+}
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUAIntegrationBrowserTest,
+ MANUAL_Fail_SendMessage) {
+ RunFailToSendMessageTest();
+}
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUAIntegrationBrowserTest,
+ MANUAL_ReconnectSession) {
+ RunReconnectSessionTest();
+}
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUAIntegrationBrowserTest,
+ MANUAL_ReconnectSessionSameTab) {
+ RunReconnectSessionSameTabTest();
+}
+
+class MediaRouterOneUANoReceiverIntegrationBrowserTest
+ : public MediaRouterIntegrationBrowserTest {
+ public:
+ GURL GetTestPageUrl(const base::FilePath& full_path) override {
+ GURL url = MediaRouterIntegrationBrowserTest::GetTestPageUrl(full_path);
+ return GURL(url.spec() + "?__oneUANoReceiver__=true");
+ }
+};
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUANoReceiverIntegrationBrowserTest,
+ MANUAL_Basic) {
+ RunBasicTest();
+}
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUANoReceiverIntegrationBrowserTest,
+ MANUAL_Fail_SendMessage) {
+ RunFailToSendMessageTest();
+}
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUANoReceiverIntegrationBrowserTest,
+ MANUAL_ReconnectSession) {
+ RunReconnectSessionTest();
+}
+
+IN_PROC_BROWSER_TEST_F(MediaRouterOneUANoReceiverIntegrationBrowserTest,
+ MANUAL_ReconnectSessionSameTab) {
+ RunReconnectSessionSameTabTest();
+}
+
+} // namespace media_router

Powered by Google App Engine
This is Rietveld 408576698