Cleanup webview app_shell_browsertests and add webview media permission tests in app_shell_browsert…

extensions/browser/guest_view/web_view/web_view_apitest.cc

 // Copyright 2014 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 "extensions/browser/guest_view/web_view/web_view_apitest.h"
+
 #include "base/path_service.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/test_utils.h"
 #include "extensions/browser/api/test/test_api.h"
 #include "extensions/browser/app_window/app_window.h"
 #include "extensions/browser/app_window/app_window_registry.h"
 #include "extensions/browser/extension_host.h"
 #include "extensions/browser/guest_view/guest_view_manager.h"
 #include "extensions/browser/guest_view/guest_view_manager_factory.h"
-#include "extensions/browser/guest_view/web_view/test_guest_view_manager.h"
+#include "extensions/browser/guest_view/test_guest_view_manager.h"
 #include "extensions/browser/process_manager.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/extension_paths.h"
 #include "extensions/shell/browser/shell_content_browser_client.h"
 #include "extensions/shell/browser/shell_extension_system.h"
 #include "extensions/shell/test/shell_test.h"
 #include "extensions/test/extension_test_message_listener.h"
 #include "net/base/filename_util.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
 #include "net/test/embedded_test_server/http_request.h"
@@ skipping 38 matching lines @@
     return scoped_ptr<net::test_server::HttpResponse>(new EmptyHttpResponse);
   }
 
   return scoped_ptr<net::test_server::HttpResponse>();
 }
 
 }  // namespace
 
 namespace extensions {
 
-// This class intercepts download request from the guest.
-class WebViewAPITest : public AppShellTest {
- protected:
-  void LaunchApp(const std::string& app_location) {
-    base::FilePath test_data_dir;
-    PathService::Get(DIR_TEST_DATA, &test_data_dir);
-    test_data_dir = test_data_dir.AppendASCII(app_location.c_str());
+void WebViewAPITest::LaunchApp(const std::string& app_location) {
+  base::FilePath test_data_dir;
+  PathService::Get(DIR_TEST_DATA, &test_data_dir);
+  test_data_dir = test_data_dir.AppendASCII(app_location.c_str());
 
-    test_config_.SetString(kTestDataDirectory,
-                           net::FilePathToFileURL(test_data_dir).spec());
+  test_config_.SetString(kTestDataDirectory,
+                         net::FilePathToFileURL(test_data_dir).spec());
 
-    embedded_test_server()->ServeFilesFromDirectory(test_data_dir);
+  embedded_test_server()->ServeFilesFromDirectory(test_data_dir);
 
-    ASSERT_TRUE(extension_system_->LoadApp(test_data_dir));
-    extension_system_->LaunchApp();
+  ASSERT_TRUE(extension_system_->LoadApp(test_data_dir));
+  extension_system_->LaunchApp();
 
-    ExtensionTestMessageListener launch_listener("LAUNCHED", false);
-    ASSERT_TRUE(launch_listener.WaitUntilSatisfied());
+  ExtensionTestMessageListener launch_listener("LAUNCHED", false);
+  ASSERT_TRUE(launch_listener.WaitUntilSatisfied());
 
-    const AppWindowRegistry::AppWindowList& app_window_list =
-        AppWindowRegistry::Get(browser_context_)->app_windows();
-    DCHECK(app_window_list.size() == 1);
-    embedder_web_contents_ = (*app_window_list.begin())->web_contents();
+  const AppWindowRegistry::AppWindowList& app_window_list =
+      AppWindowRegistry::Get(browser_context_)->app_windows();
+  DCHECK(app_window_list.size() == 1);
+  embedder_web_contents_ = (*app_window_list.begin())->web_contents();
+}
+
+void WebViewAPITest::RunTest(const std::string& test_name,
+                             const std::string& app_location) {
+  LaunchApp(app_location);
+
+  ExtensionTestMessageListener done_listener("TEST_PASSED", false);
+  done_listener.set_failure_message("TEST_FAILED");
+  if (!content::ExecuteScript(
+          embedder_web_contents_,
+          base::StringPrintf("runTest('%s')", test_name.c_str()))) {
+    LOG(ERROR) << "Unable to start test.";
+    return;
+  }
+  ASSERT_TRUE(done_listener.WaitUntilSatisfied());
+}
+
+void WebViewAPITest::SetUpOnMainThread() {
+  AppShellTest::SetUpOnMainThread();
+
+  TestGetConfigFunction::set_test_config_state(&test_config_);
+  base::FilePath test_data_dir;
+  test_config_.SetInteger(kTestWebSocketPort, 0);
+}
+
+void WebViewAPITest::StartTestServer() {
+  // For serving guest pages.
+  if (!embedded_test_server()->InitializeAndWaitUntilReady()) {
+    LOG(ERROR) << "Failed to start test server.";
+    return;
   }
 
-  void RunTest(const std::string& test_name, const std::string& app_location) {
-    LaunchApp(app_location);
+  test_config_.SetInteger(kTestServerPort, embedded_test_server()->port());
 
-    ExtensionTestMessageListener done_listener("TEST_PASSED", false);
-    done_listener.set_failure_message("TEST_FAILED");
-    if (!content::ExecuteScript(
-            embedder_web_contents_,
-            base::StringPrintf("runTest('%s')", test_name.c_str()))) {
-      LOG(ERROR) << "Unable to start test.";
-      return;
-    }
-    ASSERT_TRUE(done_listener.WaitUntilSatisfied());
+  embedded_test_server()->RegisterRequestHandler(
+      base::Bind(&RedirectResponseHandler,
+                 kRedirectResponsePath,
+                 embedded_test_server()->GetURL(kRedirectResponseFullPath)));
+
+  embedded_test_server()->RegisterRequestHandler(
+      base::Bind(&EmptyResponseHandler, kEmptyResponsePath));
+}
+
+void WebViewAPITest::StopTestServer() {
+  if (!embedded_test_server()->ShutdownAndWaitUntilComplete()) {
+    LOG(ERROR) << "Failed to shutdown test server.";
   }
+}
 
-  virtual void SetUpOnMainThread() override {
-    AppShellTest::SetUpOnMainThread();
+void WebViewAPITest::TearDownOnMainThread() {
+  TestGetConfigFunction::set_test_config_state(nullptr);
 
-    TestGetConfigFunction::set_test_config_state(&test_config_);
-    base::FilePath test_data_dir;
-    test_config_.SetInteger(kTestWebSocketPort, 0);
-  }
+  AppShellTest::TearDownOnMainThread();
+}
 
-  void StartTestServer() {
-    // For serving guest pages.
-    if (!embedded_test_server()->InitializeAndWaitUntilReady()) {
-      LOG(ERROR) << "Failed to start test server.";
-      return;
-    }
+TestGuestViewManager* WebViewAPITest::GetGuestViewManager() {
+  return static_cast<TestGuestViewManager*>(
+      TestGuestViewManager::FromBrowserContext(
+          ShellContentBrowserClient::Get()->GetBrowserContext()));
+}
 
-    test_config_.SetInteger(kTestServerPort, embedded_test_server()->port());
-
-    embedded_test_server()->RegisterRequestHandler(
-        base::Bind(&RedirectResponseHandler,
-                   kRedirectResponsePath,
-                   embedded_test_server()->GetURL(kRedirectResponseFullPath)));
-
-    embedded_test_server()->RegisterRequestHandler(
-        base::Bind(&EmptyResponseHandler, kEmptyResponsePath));
-  }
-
-  void StopTestServer() {
-    if (!embedded_test_server()->ShutdownAndWaitUntilComplete()) {
-      LOG(ERROR) << "Failed to shutdown test server.";
-    }
-  }
-
-  virtual void TearDownOnMainThread() override {
-    TestGetConfigFunction::set_test_config_state(nullptr);
-
-    AppShellTest::TearDownOnMainThread();
-  }
-
-  TestGuestViewManager* GetGuestViewManager() {
-    return static_cast<TestGuestViewManager*>(
-        TestGuestViewManager::FromBrowserContext(
-            ShellContentBrowserClient::Get()->GetBrowserContext()));
-  }
-
-  WebViewAPITest() { GuestViewManager::set_factory_for_testing(&factory_); }
-
- protected:
-  content::WebContents* embedder_web_contents_;
-  TestGuestViewManagerFactory factory_;
-  base::DictionaryValue test_config_;
-};
+WebViewAPITest::WebViewAPITest() {

[lazyboy, 2014/10/06 16:21:20]

Move constructor at the very beginning.

[lfg, 2014/10/06 16:53:18]

Done.

+  GuestViewManager::set_factory_for_testing(&factory_);
+}
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestAllowTransparencyAttribute) {
   RunTest("testAllowTransparencyAttribute", "web_view/apitest");
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestAPIMethodExistence) {
   RunTest("testAPIMethodExistence", "web_view/apitest");
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestAssignSrcAfterCrash) {
@@ skipping 240 matching lines @@
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestResizeWebviewResizesContent) {
   RunTest("testResizeWebviewResizesContent", "web_view/apitest");
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestTerminateAfterExit) {
   RunTest("testTerminateAfterExit", "web_view/apitest");
 }
 
 }  // namespace extensions