Add a hook for capturing a user profile photo and saving it to file and local state.

chrome/test/pyautolib/pyauto.py

 #!/usr/bin/python
 
 # Copyright (c) 2011 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.
 
 """PyAuto: Python Interface to Chromium's Automation Proxy.
 
 PyAuto uses swig to expose Automation Proxy interfaces to Python.
 For complete documentation on the functionality available,
@@ skipping 3589 matching lines @@
     Raises:
       pyauto_errors.JSONInterfaceError if the automation call returns an error.
     """
     cmd_dict = { 'command': 'SetMute' }
     cmd_dict = {
         'command': 'SetMute',
         'mute': mute,
     }
     return self._GetResultFromJSONRequest(cmd_dict, windex=-1)
 
+  def CaptureProfilePhoto(self):
+    """Captures user profile photo and saves it to file.
+

[dennis_jeffrey, 2011/07/29 20:10:06]

Maybe mention here where the expected file is created and what its name is.

[frankf, 2011/07/29 21:50:40]

I've added more info here. The exact path to file is in local state.

On 2011/07/29 20:10:06, dennis_jeffrey wrote:

+    Raises:
+      pyauto_errors.JSONInterfaceError if the automation call returns an error.
+    """
+    cmd_dict = { 'command': 'CaptureProfilePhoto' }
+    return self._GetResultFromJSONRequest(cmd_dict)

[dennis_jeffrey, 2011/07/29 20:10:06]

Is something important actually returned here?  The sample test doesn't seem to
check for a return value, and there's not "Returns:" section in the docstring
above to describe what is returned.

[frankf, 2011/07/29 21:50:40]

No there is no return value. However, all the functions have this statement
regardless of whether they're returning anything. 
On 2011/07/29 20:10:06, dennis_jeffrey wrote:

[dennis_jeffrey, 2011/07/29 23:11:46]

I think it's kind of inconsistent in this file.  For example, "SetReleaseTrack"
above doesn't have the return.  I think it's ok as is - just seems unnecessary
to have the return.

+
   ## ChromeOS section -- end
 
 
 class _RemoteProxy():
   """Class for PyAuto remote method calls.
 
   Use this class along with RemoteHost.testRemoteHost to establish a PyAuto
   connection with another machine and make remote PyAuto calls. The RemoteProxy
   mimics a PyAuto object, so all json-style PyAuto calls can be made on it.
 
@@ skipping 498 matching lines @@
     successful = result.wasSuccessful()
     if not successful:
       pyauto_tests_file = os.path.join(self.TestsDir(), self._tests_filename)
       print >>sys.stderr, 'Tests can be disabled by editing %s. ' \
                           'Ref: %s' % (pyauto_tests_file, _PYAUTO_DOC_URL)
     sys.exit(not successful)
 
 
 if __name__ == '__main__':
   Main()