[Password manager tests] Enable "Offer to save password" via webriver.

components/test/data/password_manager/automated_tests/environment.py

 # Copyright 2014 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.
 
 """The testing Environment class."""
 
 import logging
 import shutil
 import sys
 import time
@@ skipping 171 matching lines @@
     negation = ""
     if clear_passwords:
       negation = "!"
     script += (
         "if (%sdocument.querySelector('#delete-passwords-checkbox').checked)"
         "  document.querySelector('#delete-passwords-checkbox').click();"
         % negation)
     script += "document.querySelector('#clear-browser-data-commit').click();"
     self.driver.execute_script(script)
     time.sleep(2)
+    # Every time we do something we cache let's enable password saving.

[vabr (Chromium), 2015/02/02 09:25:29]

possible typo:
we cache -> to the cache
?

[melandory, 2015/02/02 09:35:53]

Done.

+    # TODO(melandory): We should check why it's off on a first place.

[vabr (Chromium), 2015/02/02 09:25:29]

nit: on the first place -> in the first place
Also, could you please mention the role of chromedriver? For example:
"While saving passwords is enabled by default normally, once Chrome is run via
chromedriver, the setting is off."

[melandory, 2015/02/02 09:35:53]

Done.

+    # TODO(melandory): Investigate, maybe there is no need to enable it that
+    # often

[vabr (Chromium), 2015/02/02 09:25:29]

nit: Missing a full stop at the end of the sentence.

[melandory, 2015/02/02 09:35:53]

Done.

+    self.EnablePasswordsSaving()
+
+  def EnablePasswordsSaving(self):
+    logging.info("\nEnablePasswordSaving\n")
+    self.driver.get("chrome://settings")
+    self.driver.switch_to_frame("settings")
+    script = "document.getElementById('advanced-settings-expander').click();"
+    script += (
+        "if (!document.querySelector('#password-manager-enabled').checked)"
+        "{ document.querySelector('#password-manager-enabled').click();}")
+    self.driver.execute_script(script)
+    time.sleep(2)
 
   def OpenTabAndGoToInternals(self, url):
     """If there is no |self.website_window|, opens a new tab and navigates to
     |url| in the new tab. Navigates to the passwords internals page in the
     first tab. Raises an exception otherwise.
 
     Args:
       url: Url to go to in the new tab.
 
     Raises:
@@ skipping 196 matching lines @@
         websitetest.PromptTest()
       except Exception as e:
         successful = False
       self.tests_results.append(TestResult(websitetest.name, "prompt",
           successful, e.message))
 
   def Quit(self):
     """Closes the tests."""
     # Close the webdriver.
     self.driver.quit()