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

chrome/browser/automation/automation_provider_observers_chromeos.cc

 // 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.
 
 #include "chrome/browser/automation/automation_provider_observers.h"
 
 #include "base/values.h"
 #include "chrome/browser/automation/automation_provider.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
 #include "chrome/browser/chromeos/login/authentication_notification_details.h"
 #include "chrome/browser/chromeos/login/enterprise_enrollment_screen_actor.h"
 #include "chrome/browser/chromeos/login/existing_user_controller.h"
 #include "chrome/browser/chromeos/login/screen_locker.h"
+#include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/chromeos/login/wizard_controller.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "content/common/notification_service.h"
 
 using chromeos::CrosLibrary;
 using chromeos::NetworkLibrary;
 
 NetworkManagerInitObserver::NetworkManagerInitObserver(
     AutomationProvider* automation)
     : automation_(automation->AsWeakPtr()) {}
@@ skipping 321 matching lines @@
   const chromeos::WifiNetworkVector& wifi_networks =
       network_library->wifi_networks();
   for (chromeos::WifiNetworkVector::const_iterator iter = wifi_networks.begin();
        iter != wifi_networks.end(); ++iter) {
     const chromeos::WifiNetwork* wifi = *iter;
     if (wifi->name() == ssid_)
       return wifi;
   }
   return NULL;
 }
+
+PhotoCaptureObserver::PhotoCaptureObserver(
+    AutomationProvider* automation,
+    IPC::Message* reply_message)
+    : automation_(automation->AsWeakPtr()),
+      reply_message_(reply_message) {
+}
+
+PhotoCaptureObserver::~PhotoCaptureObserver() {
+  // TODO(frankf): Currently, we do not destroy TakePhotoDialog
+  // or any of its children.
+}
+
+void PhotoCaptureObserver::OnCaptureSuccess(
+    chromeos::TakePhotoDialog* take_photo_dialog,
+    chromeos::TakePhotoView* take_photo_view) {
+  take_photo_view->ButtonPressed();
+}
+
+void PhotoCaptureObserver::OnCaptureFailure(
+    chromeos::TakePhotoDialog* take_photo_dialog,
+    chromeos::TakePhotoView* take_photo_view) {
+  AutomationJSONReply(automation_,
+                      reply_message_.release()).SendError("Capture failure");
+  delete this;
+}
+
+void PhotoCaptureObserver::OnCapturingStopped(
+    chromeos::TakePhotoDialog* take_photo_dialog,
+    chromeos::TakePhotoView* take_photo_view) {
+  take_photo_dialog->Accept();
+  const SkBitmap& photo = take_photo_view->GetImage();
+  chromeos::UserManager* user_manager = chromeos::UserManager::Get();
+  if(!user_manager) {
+    AutomationJSONReply(automation_,
+                        reply_message_.release()).SendError(
+                            "No user manager");
+  }
+
+  const chromeos::UserManager::User& user = user_manager->logged_in_user();

[Nirnimesh, 2011/07/30 10:32:31]

It's possible to reach here if user_manager == NULL. The AutomationJSONReply
above responds back to the client but does not return from this function.

You should 'delete this; return;' in all the error conditions.

[frankf, 2011/08/01 19:31:09]

Done.

+  if(user.email().empty()) {
+    AutomationJSONReply(automation_,
+                        reply_message_.release()).SendError(
+                            "User email is not set");
+  }
+
+  user_manager->SetLoggedInUserImage(photo);
+  user_manager->SaveUserImage(user.email(), photo);
+
+  AutomationJSONReply(automation_, reply_message_.release()).SendSuccess(NULL);
+  delete this;
+}