Allow popups, mouselock and https media-stream content by default.

Add a simple HTTPS server for testing getUserMedia. Popups used to be allowed through
--disable-popup-blocking, but this has a bug where the resulting popup cannot be resized
smaller. Instead, allow popups by modifying the prefs.
BUG=chromedriver:118,chromedriver:126
TEST=none

Review URL: https://chromiumcodereview.appspot.com/10828135

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150214 0039d316-1c4b-4281-b951-d872f2087c98

chrome/test/webdriver/test/cert.pem

+-----BEGIN RSA PRIVATE KEY-----
+MIICXQIBAAKBgQDQj2tPWPUgbuI4H3/3dnttqVbndwU33BdRCd67DFM44GRrsjDS
+H4bY/EbFyX9D52d/iy6ZaAmDePcCz5k/fgP3DMujykYGqgNiV2ywxTlMj7NlN2C7
+SRt68fQMZr5iI7rypdxuaZt9lSMD3ENBffYtuLTyZd9a3JPJe1TaIab5GwIDAQAB
+AoGANHXu8z2YIzlhE+bwhGm8MGBpKL3qhRuKjeriqMA36tWezOw8lY4ymEAU+Ulv
+BsCdaxqydQoTYou57m4TyUHEcxq9pq3H0zB0qL709DdHi/t4zbV9XIoAzC5v0/hG
+9+Ca29TwC02FCw+qLkNrtwCpwOcQmc+bPxqvFu1iMiahURECQQD2I/Hi2413CMZz
+TBjl8fMiVO9GhA2J0sc8Qi+YcgJakaLD9xcbaiLkTzPZDlA389C1b6Ia+poAr4YA
+Ve0FFbxpAkEA2OobayyHE/QtPEqoy6NLR57jirmVBNmSWWd4lAyL5UIHIYVttJZg
+8CLvbzaU/iDGwR+wKsM664rKPHEmtlyo4wJBAMeSqYO5ZOCJGu9NWjrHjM3fdAsG
+8zs2zhiLya+fcU0iHIksBW5TBmt71Jw/wMc9R5J1K0kYvFml98653O5si1ECQBCk
+RV4/mE1rmlzZzYFyEcB47DQkcM5ictvxGEsje0gnfKyRtAz6zI0f4QbDRUMJ+LWw
+XK+rMsYHa+SfOb0b9skCQQCLdeonsIpFDv/Uv+flHISy0WA+AFkLXrRkBKh6G/OD
+dMHaNevkJgUnpceVEnkrdenp5CcEoFTI17pd+nBgDm/B
+-----END RSA PRIVATE KEY-----
+-----BEGIN CERTIFICATE-----
+MIIEZDCCAkygAwIBAgIBATANBgkqhkiG9w0BAQUFADBgMRAwDgYDVQQDEwdUZXN0
+IENBMQswCQYDVQQGEwJVUzETMBEGA1UECBMKQ2FsaWZvcm5pYTEWMBQGA1UEBxMN
+TW91bnRhaW4gVmlldzESMBAGA1UEChMJQ2VydCBUZXN0MB4XDTA4MDcyODIyMzIy
+OFoXDTEzMDcyNzIyMzIyOFowSjELMAkGA1UEBhMCVVMxEzARBgNVBAgTCkNhbGlm
+b3JuaWExEjAQBgNVBAoTCUNlcnQgVGVzdDESMBAGA1UEAxMJMTI3LjAuMC4xMIGf
+MA0GCSqGSIb3DQEBAQUAA4GNADCBiQKBgQDQj2tPWPUgbuI4H3/3dnttqVbndwU3
+3BdRCd67DFM44GRrsjDSH4bY/EbFyX9D52d/iy6ZaAmDePcCz5k/fgP3DMujykYG
+qgNiV2ywxTlMj7NlN2C7SRt68fQMZr5iI7rypdxuaZt9lSMD3ENBffYtuLTyZd9a
+3JPJe1TaIab5GwIDAQABo4HCMIG/MAkGA1UdEwQCMAAwHQYDVR0OBBYEFCYLBv5K
+x5sLNVlpLh5FwTwhdDl7MIGSBgNVHSMEgYowgYeAFF3Of5nj1BlBMU/Gz7El9Vqv
+45cxoWSkYjBgMRAwDgYDVQQDEwdUZXN0IENBMQswCQYDVQQGEwJVUzETMBEGA1UE
+CBMKQ2FsaWZvcm5pYTEWMBQGA1UEBxMNTW91bnRhaW4gVmlldzESMBAGA1UEChMJ
+Q2VydCBUZXN0ggkA1FGT1D/e2U4wDQYJKoZIhvcNAQEFBQADggIBAEtkVmLObUgk
+b2cIA2S+QDtifq1UgVfBbytvR2lFmnADOR55mo0gHQG3HHqq4g034LmoVXDHhUk8
+Gb6aFiv4QubmVhLXcUelTRXwiNvGzkW7pC6Jrq105hdPjzXMKTcmiLaopm5Fqfc7
+hj5Cn1Sjspc8pdeQjrbeMdvca7KlFrGP8YkwCU2xOOX9PiN9G0966BWfjnr/fZZp
++OQVuUFHdiAZwthEMuDpAAXHqYXIsermgdOpgJaA53cf8NqBV2QGhtFgtsJCRoiu
+7DKqhyRWBGyz19VIH2b7y+6qvQVxuHk19kKRM0nftw/yNcJnm7gtttespMUPsOMa
+a2SD1G0hm0TND6vxaBhgR3cVqpl/qIpAdFi00Tm7hTyYE7I43zPW03t+/DpCt3Um
+EMRZsQ90co5q+bcx/vQ7YAtwUh30uMb0wpibeyCwDp8cqNmSiRkEuc/FjTYes5t8
+5gR//WX1l0+qjrjusO9NmoLnq2Yk6UcioX+z+q6Z/dudGfqhLfeWD2Q0LWYA242C
+d7km5Y3KAt1PJdVsof/aiVhVdddY/OIEKTRQhWEdDbosy2eh16BCKXT2FFvhNDg1
+AYFvn6I8nj9IldMJiIc3DdhacEAEzRMeRgPdzAa1griKUGknxsyTyRii8ru0WS6w
+DCNrlDOVXdzYGEZooBI76BDVY0W0akjV
+-----END CERTIFICATE-----

chrome/test/webdriver/test/chromedriver_test.py

@@ -4,7 +4,10 @@
 
 """Factory that creates ChromeDriver instances."""
 
+import BaseHTTPServer
 import os
+import ssl
+import threading
 import unittest
 
 from chromedriver_factory import ChromeDriverFactory
@@ -12,6 +15,54 @@ from chromedriver_launcher import ChromeDriverLauncher
 import test_paths
 
 
+class _FileRequestHandler(BaseHTTPServer.BaseHTTPRequestHandler):
+  """Sends back file resources relative to the server's |root_dir|."""
+
+  def do_GET(self):
+    if self.path.endswith('favicon.ico'):
+      self.send_error(404)
+      return
+    path = os.path.join(self.server.root_dir, *self.path.split('/'))
+    data = open(path, 'r').read()
+    self.send_response(200)
+    self.send_header('Content-Length', len(data))
+    self.end_headers()
+    self.wfile.write(data)
+
+
+class _HttpsServer(object):
+  """An HTTPS server that serves files on its own thread."""
+
+  def __init__(self, server_cert_and_key_path, root_dir):
+    """Starts the HTTPS server on its own thread on an ephemeral port.
+
+    After this function returns, it is safe to assume the server is ready
+    to receive requests.
+
+    Args:
+      server_cert_and_key_path: path to a PEM file containing the cert and key.
+      root_dir: root path to serve files from.
+    """
+    self._server = BaseHTTPServer.HTTPServer(
+        ('127.0.0.1', 0), _FileRequestHandler)
+    self._server.root_dir = root_dir
+    self._server.socket = ssl.wrap_socket(
+        self._server.socket, certfile=server_cert_and_key_path,
+        server_side=True)
+
+    self._thread = threading.Thread(target=self._server.serve_forever)
+    self._thread.start()
+
+  def GetUrl(self):
+    """Returns the base URL of the server."""
+    return 'https://127.0.0.1:%s' % self._server.server_port
+
+  def Shutdown(self):
+    """Shuts down the server synchronously."""
+    self._server.shutdown()
+    self._thread.join()
+
+
 class ChromeDriverTest(unittest.TestCase):
   """Fixture for tests that need to instantiate ChromeDriver(s)."""
 
@@ -27,18 +78,26 @@ class ChromeDriverTest(unittest.TestCase):
     ChromeDriverTest._driver_path = driver_path
     ChromeDriverTest._chrome_path = chrome_path
     ChromeDriverTest._server = ChromeDriverLauncher(driver_path).Launch()
-    ChromeDriverTest._webserver = ChromeDriverLauncher(
+    ChromeDriverTest._http_server = ChromeDriverLauncher(
         driver_path, test_paths.TEST_DATA_PATH).Launch()
+    ChromeDriverTest._https_server = _HttpsServer(
+        test_paths.PEM_CERT_AND_KEY, test_paths.TEST_DATA_PATH)
 
   @staticmethod
   def GlobalTearDown():
     ChromeDriverTest._server.Kill()
-    ChromeDriverTest._webserver.Kill()
+    ChromeDriverTest._http_server.Kill()
+    ChromeDriverTest._https_server.Shutdown()
 
   @staticmethod
   def GetTestDataUrl():
-    """Returns the base url for serving files from the ChromeDriver test dir."""
-    return ChromeDriverTest._webserver.GetUrl()
+    """Returns the base http url for serving files from the test dir."""
+    return ChromeDriverTest._http_server.GetUrl()
+
+  @staticmethod
+  def GetHttpsTestDataUrl():
+    """Returns the base https url for serving files from the test dir."""
+    return ChromeDriverTest._https_server.GetUrl()
 
   @staticmethod
   def GetDriverPath():

chrome/test/webdriver/test/chromedriver_tests.py

@@ -288,28 +288,19 @@ class DesiredCapabilitiesTest(ChromeDriverTest):
     self.assertTrue('ExtTest2' in extension_names)
     driver.quit()
 
-  def testUseWebsiteTestingDefaults(self):
-    """Test that chromedriver initializes options for website testing."""
-    driver = self.GetNewDriver()
-    driver.get(self.GetTestDataUrl() + '/content_setting_test.html')
-    driver.set_script_timeout(10)
-    # Will timeout if infobar appears.
-    driver.execute_async_script('waitForGeo(arguments[0])')
-
   def testPrefs(self):
     """Test that chromedriver can set user preferences."""
     driver = self.GetNewDriver({
       'chrome.noWebsiteTestingDefaults': True,
       'chrome.prefs': {
         'profile.default_content_settings': {
-          'geolocation': 1
+          'popups': 1
         },
       }
     })
-    driver.get(self.GetTestDataUrl() + '/content_setting_test.html')
-    driver.set_script_timeout(10)
-    # Will timeout if infobar appears.
-    driver.execute_async_script('waitForGeo(arguments[0])')
+    driver.get(self.GetTestDataUrl() + '/empty.html')
+    driver.execute_script('window.open("about:blank")')
+    self.assertEquals(2, len(driver.window_handles))
 
 
 class DetachProcessTest(ChromeDriverTest):
@@ -1147,6 +1138,7 @@ class ExtensionTest(ChromeDriverTest):
     ext.click_page_action()
     self._testExtensionView(driver, ext.get_popup_handle(), ext)
 
+
 class BadJSTest(ChromeDriverTest):
   """Tests that ensure sites with hacky JS don't break ChromeDriver."""
 
@@ -1155,3 +1147,46 @@ class BadJSTest(ChromeDriverTest):
     driver.get(self.GetTestDataUrl() + '/bad_native_funcs.html')
     # This will throw an exception if any native funcs are used.
     driver.find_element_by_tag_name('body').find_elements_by_tag_name('div')
+
+
+class ContentSettingsTest(ChromeDriverTest):
+  """Tests that various types of content are allowed by default."""
+
+  def testPopups(self):
+    driver = self.GetNewDriver()
+    driver.get(self.GetTestDataUrl() + '/empty.html')
+    driver.execute_script('window.open("about:blank")')
+    self.assertEquals(2, len(driver.window_handles))
+
+  def testPopupsCanBeResized(self):
+    """Regression test for chromedriver issue 126."""
+    driver = self.GetNewDriver()
+    driver.get(self.GetTestDataUrl() + '/empty.html')
+    driver.execute_script(
+        'window.open("empty.html", "popup", "width=500,height=500")')
+    driver.switch_to_window(driver.window_handles[1])
+    size = driver.get_window_size()
+    bigger_size = dict(map(lambda x: (x, size[x] + 100), size))
+    smaller_size = dict(map(lambda x: (x, size[x] - 100), size))
+    driver.set_window_size(bigger_size['width'], bigger_size['height'])
+    self.assertEquals(bigger_size, driver.get_window_size())
+    driver.set_window_size(smaller_size['width'], smaller_size['height'])
+    self.assertEquals(smaller_size, driver.get_window_size())
+
+  def testGeolocation(self):
+    driver = self.GetNewDriver()
+    driver.get(self.GetTestDataUrl() + '/empty.html')
+    driver.set_script_timeout(10)
+    # Will timeout if infobar appears.
+    driver.execute_async_script(
+        'navigator.geolocation.getCurrentPosition(arguments[0], arguments[0]);')
+
+  def testMediaStream(self):
+    driver = self.GetNewDriver()
+    # Allowing camera/mic access by default only works for https sites.
+    driver.get(self.GetHttpsTestDataUrl() + '/empty.html')
+    driver.set_script_timeout(10)
+    # Will timeout if infobar appears.
+    driver.execute_async_script(
+        'navigator.webkitGetUserMedia({audio:true, video:true},' +
+        '                             arguments[0], arguments[0]);')

chrome/test/webdriver/test/content_setting_test.html

-<script>
-var g_callback = null;
-var g_geoDone = false;
-function waitForGeo(doneCallback) {
-  if (g_geoDone) {
-    doneCallback();
-  } else {
-    g_callback = doneCallback;
-  }
-}
-
-function geoDone() {
-  g_geoDone = true;
-  if (g_callback)
-    g_callback();
-}
-
-// Don't care whether geo succeeds or fails, just whether it gets past the
-// infobar.
-navigator.geolocation.getCurrentPosition(geoDone, geoDone);
-</script>

chrome/test/webdriver/test/empty.html

+<html>
+</html>

chrome/test/webdriver/test/test_paths.py

-# Copyright (c) 2011 The Chromium Authors. All rights reserved.
+# Copyright (c) 2012 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.
 
@@ -26,8 +26,9 @@ def _SetupPaths():
   global SRC_PATH
   SRC_PATH = J(start_dir, os.pardir, os.pardir, os.pardir, os.pardir)
 
-  global TEST_DATA_PATH
+  global TEST_DATA_PATH, PEM_CERT_AND_KEY
   TEST_DATA_PATH = start_dir
+  PEM_CERT_AND_KEY = J(start_dir, 'cert.pem')
 
   global CHROME_TEST_DATA_PATH
   CHROME_TEST_DATA_PATH = J(SRC_PATH, 'chrome', 'test', 'data')

chrome/test/webdriver/webdriver_automation.cc

@@ -337,8 +337,7 @@ namespace webdriver {
 Automation::BrowserOptions::BrowserOptions()
     : command(CommandLine::NO_PROGRAM),
       detach_process(false),
-      ignore_certificate_errors(false),
-      disable_popup_blocking(false) {}
+      ignore_certificate_errors(false) {}
 
 Automation::BrowserOptions::~BrowserOptions() {}
 
@@ -367,8 +366,6 @@ void Automation::Init(
     command.AppendSwitch(switches::kAutomationReinitializeOnChannelError);
   if (options.ignore_certificate_errors)
     command.AppendSwitch(switches::kIgnoreCertificateErrors);
-  if (options.disable_popup_blocking)
-    command.AppendSwitch(switches::kDisablePopupBlocking);
   if (options.user_data_dir.empty())
     command.AppendSwitchASCII(switches::kHomePage, chrome::kAboutBlankURL);
 

chrome/test/webdriver/webdriver_automation.h

@@ -68,9 +68,6 @@ class Automation {
 
     // True if the browser should ignore certificate related errors.
     bool ignore_certificate_errors;
-
-    // True if the browser should disable popup blocking.
-    bool disable_popup_blocking;
   };
 
   explicit Automation(const Logger& logger);

chrome/test/webdriver/webdriver_session.cc

@@ -111,7 +111,6 @@ Error* Session::Init(const DictionaryValue* capabilities_dict) {
   browser_options.user_data_dir = capabilities_.profile;
   if (!capabilities_.no_website_testing_defaults) {
     browser_options.ignore_certificate_errors = true;
-    browser_options.disable_popup_blocking = true;
   }
   RunSessionTask(base::Bind(
       &Session::InitOnSessionThread,
@@ -1900,11 +1899,28 @@ Error* Session::InitForWebsiteTesting() {
   if (error)
     return error;
 
-  // Allow certain content by default.
+  // Allow content by default.
+  // Media-stream cannot be enabled by default; we must specify
+  // particular host patterns and devices.
+  DictionaryValue* devices = new DictionaryValue();
+  devices->SetString("audio", "Default");
+  devices->SetString("video", "Default");
+  DictionaryValue* content_settings = new DictionaryValue();
+  content_settings->Set("media-stream", devices);
+  DictionaryValue* pattern_pairs = new DictionaryValue();
+  pattern_pairs->Set("https://*,*", content_settings);
+  error = SetPreference(
+      "profile.content_settings.pattern_pairs",
+      true /* is_user_pref */,
+      pattern_pairs);
+  if (error)
+    return error;
   const int kAllowContent = 1;
   DictionaryValue* default_content_settings = new DictionaryValue();
   default_content_settings->SetInteger("geolocation", kAllowContent);
+  default_content_settings->SetInteger("mouselock", kAllowContent);
   default_content_settings->SetInteger("notifications", kAllowContent);
+  default_content_settings->SetInteger("popups", kAllowContent);
   return SetPreference(
       "profile.default_content_settings",
       true /* is_user_pref */,