[chromedriver] setting browser default download directory

chrome/test/chromedriver/client/chromedriver.py

Index: chrome/test/chromedriver/client/chromedriver.py
diff --git a/chrome/test/chromedriver/client/chromedriver.py b/chrome/test/chromedriver/client/chromedriver.py
index fb1aa2b27fded702a22419ca0273bc43a79de2d2..0d74f24054d9489c99e50914dd9e640b923930e4 100644
--- a/chrome/test/chromedriver/client/chromedriver.py
+++ b/chrome/test/chromedriver/client/chromedriver.py
@@ -66,7 +66,7 @@ class ChromeDriver(object):
                chrome_extensions=None, chrome_log_path=None,
                debugger_address=None, browser_log_level=None,
                performance_log_level=None, mobile_emulation=None,
-               experimental_options=None):
+               experimental_options=None, download_dir=None):
     self._executor = command_executor.CommandExecutor(server_url)
 
     options = {}
@@ -115,6 +115,13 @@ class ChromeDriver(object):
       assert performance_log_level in log_levels
       logging_prefs['performance'] = performance_log_level
 
+    download_prefs = {}
+    if (download_dir):
+      default_dir = {}
+      default_dir['default_directory'] = download_dir
+      download_prefs['download'] =  default_dir
+      options['prefs'] =  download_prefs

[samuong, 2014/10/17 23:30:35]

Just in case prefs already exists in some form (e.g. via experimental_options)
maybe we should be careful about not clobbering it. How about we change this to
the following?

if download_dir:
  if 'prefs' not in options:
    options['prefs'] = {}
  if 'download' not in options['prefs']:
    options['prefs']['download'] = {}
  options['prefs']['download']['default_directory'] = download_dir

[andrewcheng, 2014/10/18 00:45:46]

Done.

+
     params = {
       'desiredCapabilities': {
         'chromeOptions': options,