[chromedriver] setting browser default download directory

chrome/test/chromedriver/chrome_launcher.cc

 // Copyright (c) 2013 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 "chrome/test/chromedriver/chrome_launcher.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/base64.h"
@@ skipping 110 matching lines @@
   switches.SetSwitch("remote-debugging-port", base::IntToString(port));
   switches.SetSwitch("test-type", "webdriver");
 
   for (std::set<std::string>::const_iterator iter =
            capabilities.exclude_switches.begin();
        iter != capabilities.exclude_switches.end();
        ++iter) {
     switches.RemoveSwitch(*iter);
   }
   switches.SetFromSwitches(capabilities.switches);
-
+  base::FilePath user_data_dir_path;
   if (!switches.HasSwitch("user-data-dir")) {
     command.AppendArg("data:,");
     if (!user_data_dir->CreateUniqueTempDir())
       return Status(kUnknownError, "cannot create temp dir for user data dir");
     switches.SetSwitch("user-data-dir", user_data_dir->path().value());
-    Status status = internal::PrepareUserDataDir(
-        user_data_dir->path(), capabilities.prefs.get(),
-        capabilities.local_state.get());
-    if (status.IsError())
+    user_data_dir_path = user_data_dir->path();
+  } else {
+    user_data_dir_path = base::FilePath(
+        switches.GetSwitchValue("user-data-dir"));
+  }
+
+  Status status = internal::PrepareUserDataDir(
+                         user_data_dir_path,
+                         capabilities.prefs.get(),
+                         capabilities.local_state.get());

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

please fix indenting for arguments - they should either be lined up to the
opening parenthesis or indented by 4 spaces

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

Done.

+  if (status.IsError())
       return status;

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

please fix indenting for "return status;" (it should only be indented by 4
spaces, not 6)

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

Done.

-  }
 
   if (!extension_dir->CreateUniqueTempDir()) {
     return Status(kUnknownError,
                   "cannot create temp dir for unpacking extensions");
   }
-  Status status = internal::ProcessExtensions(capabilities.extensions,
-                                              extension_dir->path(),
-                                              true,
-                                              &switches,
-                                              extension_bg_pages);
+  status = internal::ProcessExtensions(capabilities.extensions,
+                                       extension_dir->path(),
+                                       true,
+                                       &switches,
+                                       extension_bg_pages);
   if (status.IsError())
     return status;
   switches.AppendToCommandLine(&command);
   *prepared_command = command;
   return Status(kOk);
 }
 
 Status WaitForDevToolsAndCheckVersion(
     const NetAddress& address,
     URLRequestContextGetter* context_getter,
@@ skipping 602 matching lines @@
 
 Status PrepareUserDataDir(
     const base::FilePath& user_data_dir,
     const base::DictionaryValue* custom_prefs,
     const base::DictionaryValue* custom_local_state) {
   base::FilePath default_dir =
       user_data_dir.AppendASCII(chrome::kInitialProfile);
   if (!base::CreateDirectory(default_dir))
     return Status(kUnknownError, "cannot create default profile directory");
 
+  std::string preferences;
+  base::FilePath preferences_path =
+         default_dir.Append(chrome::kPreferencesFilename);

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

fix indenting (should be 6 spaces, not 9)

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

Done.

+
+  if (base::PathExists(preferences_path))
+    base::ReadFileToString(preferences_path, &preferences);
+  else
+    preferences = std::string(kPreferences);

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

why not just do "preferences = kPreferences;"?

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

Done.

+
   Status status =
-      WritePrefsFile(kPreferences,
+      WritePrefsFile(preferences,
                      custom_prefs,
                      default_dir.Append(chrome::kPreferencesFilename));
   if (status.IsError())
     return status;
 
-  status = WritePrefsFile(kLocalState,
+  std::string local_state;
+  base::FilePath local_state_path =
+        user_data_dir.Append(chrome::kLocalStateFilename);

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

fix indenting (should be 6 spaces, not 8)

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

Done.

+
+  if (base::PathExists(local_state_path))
+    base::ReadFileToString(local_state_path, &local_state);
+  else
+    local_state = std::string(kLocalState);

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

similar to above, why not just do "local_state = kLocalState;"?

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

Done.

+
+  status = WritePrefsFile(local_state,
                           custom_local_state,
                           user_data_dir.Append(chrome::kLocalStateFilename));
   if (status.IsError())
     return status;
 
   // Write empty "First Run" file, otherwise Chrome will wipe the default
   // profile that was written.
   if (base::WriteFile(
           user_data_dir.Append(chrome::kFirstRunSentinel), "", 0) != 0) {
     return Status(kUnknownError, "failed to write first run file");
   }
   return Status(kOk);
 }
 
 }  // namespace internal