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

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

nit: add space after the '//', and add a period at the end of the sentence in
the next line below

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

Done.

+  // 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");

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

line too long; 80 chars max

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

Done.

+  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();
+  DCHECK(user_manager);

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

instead of a DCHECK, maybe we could send an informative error reply message
instead (same at line 391 below), since DCHECKs are only used in Debug builds.

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

Done.

+
+  const chromeos::UserManager::User& user = user_manager->logged_in_user();
+  DCHECK(!user.email().empty());
+
+  user_manager->SetLoggedInUserImage(photo);
+  user_manager->SaveUserImage(user.email(), photo);
+
+  AutomationJSONReply(automation_, reply_message_.release()).SendSuccess(NULL);

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

Rather than returning success here, shouldn't we wait to make sure that the
OnCaptureSuccess event occurs first?  Or are we guaranteed that it occurs before
we get to this point?

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

We reach this point only when we execute PhotoCaptureObserver::OnCaptureSuccess.


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

+  delete this;
+}