Port perf.py's _LoginToGoogleAccount to chrome_remote_control

tools/chrome_remote_control/chrome_remote_control/google_credentials_backend.py

Index: tools/chrome_remote_control/chrome_remote_control/google_credentials_backend.py
diff --git a/tools/chrome_remote_control/chrome_remote_control/google_credentials_backend.py b/tools/chrome_remote_control/chrome_remote_control/google_credentials_backend.py
new file mode 100644
index 0000000000000000000000000000000000000000..daea914eb6ac498ab74d408e7bb875f81c2e49e0
--- /dev/null
+++ b/tools/chrome_remote_control/chrome_remote_control/google_credentials_backend.py
@@ -0,0 +1,86 @@
+# Copyright (c) 2012 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.
+
+import sys
+import time
+import logging
+import os
+import chrome_remote_control
+from credentials_backend import CredentialsBackend
+
+class GoogleCredentialsBackend(CredentialsBackend):
+  label = 'google'
+
+  def __init__(self, tab):
+    self._tab = tab
+    self._logged_in = False
+
+  def LoginNeeded(self):
+    if not self._logged_in:
+      self._logged_in = self._LoginToGoogleAccount()
+
+  def LoginNoLongerNeeded(self):

[dtu, 2012/09/24 21:35:10]

This is a placeholder for the future, right? What are some potential scenarios
in which this method will be needed/used?

[hartmanng, 2012/09/26 22:06:15]

I added this in response to http://codereview.chromium.org/10914237/#msg2 - it's
my understanding that we may just leave this as a no-op. Probably best if @nduca
answers this though.

+    pass
+
+  def GoogleAccountsLogin(self, username, password, url=None):
+    """Log into Google Accounts.
+
+    Attempts to login to Google by entering the username/password into the
+    google login page and click submit button.
+
+    Args:
+      username: users login input.
+      password: users login password input.
+      url: an alternative url for login page, if None, original one will be
+      used.
+    """
+    url = url or 'https://accounts.google.com/'
+
+    try:
+      logging.info('Loading %s...', url)
+      self._tab.page.Navigate(url)
+      time.sleep(2)  # Wait for unpredictable redirects.

[tonyg, 2012/09/24 20:41:01]

Looks brittle. Is there anything better to WaitFor rather than sleeping?

[hartmanng, 2012/09/26 22:06:15]

Done.

+      self._tab.WaitForDocumentReadyStateToBeInteractiveOrBetter()
+      logging.info('Loaded page: %s', url)
+
+      email_id = 'document.getElementById("Email").value = "%s"; ' % username
+      password = 'document.getElementById("Passwd").value = "%s"; ' % password
+      self._tab.runtime.Execute(email_id)
+      self._tab.runtime.Execute(password)
+
+      self.SubmitFormAndWait('gaia_loginform')
+
+      return True
+    except chrome_remote_control.TimeoutException:
+      logging.warning('Timed out while loading: %s', url)
+      print 'Timed out while loading: ', url
+    return False
+
+
+  def _LoginToGoogleAccount(self):
+    """Logs in to a test Google account.
+
+    Login with user-defined credentials if they exist.
+    Else fail.
+
+    Raises:
+      RuntimeError: if could not get credential information.
+    """
+    config = CredentialsBackend._GetConfig(
+        {'google_username': None,
+         'google_password': None,
+         'google_account_url': 'https://accounts.google.com/'})
+    google_account_url = config.get('google_account_url')
+    username = config.get('google_username')
+    password = config.get('google_password')
+    if username and password:
+      logging.info(
+          'Using google account credential from %s',
+          self._config_file, 'credentials.json')
+    else:
+      message = 'No credentials could be found. ' \
+                'Please specify credential information in %s.' \
+                % self._config_file
+      raise RuntimeError(message)
+    return self.GoogleAccountsLogin(username, password, url=google_account_url)