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

Unified Diff: tools/testing/run_selenium.py

Issue 9252002: Make changes for Safari to run with webdriver on the bots. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: '' Created 8 years, 11 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
« no previous file with comments | « tools/testing/perf_testing/perf_README.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/run_selenium.py
===================================================================
--- tools/testing/run_selenium.py (revision 3378)
+++ tools/testing/run_selenium.py (working copy)
@@ -9,10 +9,12 @@
the result.
"""
+import os
import optparse
import platform
import selenium
from selenium.webdriver.support.ui import WebDriverWait
+import shutil
import socket
import sys
import time
@@ -108,6 +110,11 @@
elif browser == 'ie' and platform.system() == 'Windows':
browser = selenium.webdriver.Ie()
elif browser == 'safari' and platform.system() == 'Darwin':
+ # TODO(efortuna): Ensure our preferences (no pop-up blocking) file is the
+ # same (Safari auto-deletes when it has too many "crashes," or in our case,
+ # timeouts). Come up with a less hacky way to do this.
+ shutil.copy(os.path.dirname(__file__) + '/com.apple.Safari.plist',
+ '/Library/Preferences/com.apple.Safari.plist')
sel = selenium.selenium('localhost', 4444, "*safari", 'file://' + html_out)
try:
sel.start()
« no previous file with comments | « tools/testing/perf_testing/perf_README.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698