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

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

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150214

[dennis_jeffrey, 2012-08-03 18:10:42]

2 nits and a question

http://codereview.chromium.org/10828135/diff/2001/chrome/test/webdriver/test/...
File chrome/test/webdriver/test/chromedriver_test.py (right):

http://codereview.chromium.org/10828135/diff/2001/chrome/test/webdriver/test/...
chrome/test/webdriver/test/chromedriver_test.py:26: data = file(path,
'r').read()
any reason for using file() instead of open()?  Do we need to close the file?

http://codereview.chromium.org/10828135/diff/2001/chrome/test/webdriver/test/...
chrome/test/webdriver/test/chromedriver_test.py:34: """A file serving HTTPS
server that serves on its own thread."""
nit: serving/server/serves --> /server/serves:

"An HTTPS server that serves files on its own thread."

http://codereview.chromium.org/10828135/diff/2001/chrome/test/webdriver/test/...
chrome/test/webdriver/test/chromedriver_test.py:61: """Shutsdown the server
synchronously."""
'Shutsdown' --> 'Shuts down'

[kkania, 2012-08-06 21:50:27]

http://codereview.chromium.org/10828135/diff/2001/chrome/test/webdriver/test/...
File chrome/test/webdriver/test/chromedriver_test.py (right):

http://codereview.chromium.org/10828135/diff/2001/chrome/test/webdriver/test/...
chrome/test/webdriver/test/chromedriver_test.py:26: data = file(path,
'r').read()
On 2012/08/03 18:10:42, dennis_jeffrey wrote:
> any reason for using file() instead of open()?  Do we need to close the file?

whoops, switched to open. file is being removed in python 3; the file will be
closed when python garbage collects, which is good enough i think (testserver
does the same thing)

http://codereview.chromium.org/10828135/diff/2001/chrome/test/webdriver/test/...
chrome/test/webdriver/test/chromedriver_test.py:34: """A file serving HTTPS
server that serves on its own thread."""
On 2012/08/03 18:10:42, dennis_jeffrey wrote:
> nit: serving/server/serves --> /server/serves:
> 
> "An HTTPS server that serves files on its own thread."

Done.

http://codereview.chromium.org/10828135/diff/2001/chrome/test/webdriver/test/...
chrome/test/webdriver/test/chromedriver_test.py:61: """Shutsdown the server
synchronously."""
On 2012/08/03 18:10:42, dennis_jeffrey wrote:
> 'Shutsdown' --> 'Shuts down'

Done.

[dennis_jeffrey, 2012-08-06 22:20:53]

LGTM

[commit-bot: I haz the power, 2012-08-06 22:42:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kkania@chromium.org/10828135/2002

[commit-bot: I haz the power, 2012-08-06 22:42:46]

Failed to apply patch for chrome/test/webdriver/test/chromedriver_tests.py:
While running patch -p1 --forward --force;
patching file chrome/test/webdriver/test/chromedriver_tests.py
Hunk #2 FAILED at 1137.
1 out of 2 hunks FAILED -- saving rejects to file
chrome/test/webdriver/test/chromedriver_tests.py.rej

[commit-bot: I haz the power, 2012-08-06 23:59:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kkania@chromium.org/10828135/9002

[commit-bot: I haz the power, 2012-08-07 01:17:20]

Change committed as 150214