Move logic in UserImageScreen from UserImageScreenHandler

chrome/browser/chromeos/login/screens/user_image_screen.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_USER_IMAGE_SCREEN_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_USER_IMAGE_SCREEN_H_
 
 #include "base/compiler_specific.h"
+#include "base/memory/weak_ptr.h"
+#include "chrome/browser/chromeos/camera_detector.h"

[ygorshenin1, 2013/04/18 08:12:30]

Move to .cc

[Denis Kuznetsov (DE-MUC), 2013/04/18 08:48:46]

Done.

 #include "chrome/browser/chromeos/login/screens/user_image_screen_actor.h"
 #include "chrome/browser/chromeos/login/screens/wizard_screen.h"
 #include "chrome/browser/chromeos/login/user.h"
+#include "chrome/browser/image_decoder.h"
+#include "chrome/common/url_constants.h"

[ygorshenin1, 2013/04/18 08:12:30]

Why do we need this?

[Denis Kuznetsov (DE-MUC), 2013/04/18 08:48:46]

profile_picture_data_url_(chrome::kAboutBlankURL)

+#include "content/public/browser/browser_thread.h"

[ygorshenin1, 2013/04/18 08:12:30]

Move to .cc

[Denis Kuznetsov (DE-MUC), 2013/04/18 08:48:46]

Done.

 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
 namespace chromeos {
 
 class UserImageScreen: public WizardScreen,
                        public UserImageScreenActor::Delegate,
+                       public ImageDecoder::Delegate,
                        public content::NotificationObserver {
  public:
   UserImageScreen(ScreenObserver* screen_observer,
                   UserImageScreenActor* actor);
   virtual ~UserImageScreen();
 
   // Indicates whether profile picture is enabled for given user.
   void SetProfilePictureEnabled(bool support_profile_picture);
   // Sets |user_id| of user that would have picture updated.
   void SetUserID(const std::string& user_id);
 
   // WizardScreen implementation:
   virtual void PrepareToShow() OVERRIDE;
   virtual void Show() OVERRIDE;
   virtual void Hide() OVERRIDE;
   virtual std::string GetName() const OVERRIDE;
 
   // UserImageScreenActor::Delegate implementation:
-  virtual void OnPhotoTaken(const gfx::ImageSkia& image) OVERRIDE;
-  virtual void OnProfileImageSelected() OVERRIDE;
-  virtual void OnDefaultImageSelected(int index) OVERRIDE;
+  virtual void CheckCameraPresence() OVERRIDE;
+  virtual void OnPhotoTaken(const std::string& raw_data) OVERRIDE;
+  virtual void OnImageSelected(const std::string& image_url,
+                               const std::string& image_type) OVERRIDE;
+  virtual void OnImageAccepted() OVERRIDE;
   virtual void OnActorDestroyed(UserImageScreenActor* actor) OVERRIDE;
 
+  virtual bool profile_picture_absent() OVERRIDE;
+  virtual int selected_image() OVERRIDE;
+  virtual std::string profile_picture_data_url() OVERRIDE;
+
   // content::NotificationObserver implementation:
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
+  // Overriden from ImageDecoder::Delegate:
+  virtual void OnImageDecoded(const ImageDecoder* decoder,
+                              const SkBitmap& decoded_image) OVERRIDE;
+  virtual void OnDecodeImageFailed(const ImageDecoder* decoder) OVERRIDE;
+
  private:
   const User* GetUser();
 
+  // Called when the camera presence check has been completed.
+  void OnCameraPresenceCheckDone();
+
   content::NotificationRegistrar registrar_;
 
   UserImageScreenActor* actor_;
 
+  base::WeakPtrFactory<UserImageScreen> weak_factory_;
+
+  // Last ImageDecoder instance used to decode an image blob received by
+  // HandlePhotoTaken.
+  scoped_refptr<ImageDecoder> image_decoder_;
+
+  // Last user photo, if taken.
+  gfx::ImageSkia user_photo_;
+
+  // If |true|, decoded photo should be immediately accepeted (i.e., both
+  // HandleTakePhoto and HandleImageAccepted have already been called but we're
+  // still waiting for  photo image decoding to finish.
+  bool accept_photo_after_decoding_;
+
+  // Index of the selected user image.
+  int selected_image_;
+
   bool profile_picture_enabled_;
 
+  // Encoded profile picture.
+  std::string profile_picture_data_url_;
+
+  // True if user has no custom profile picture.
+  bool profile_picture_absent_;
+
   std::string user_id_;
 
   DISALLOW_COPY_AND_ASSIGN(UserImageScreen);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_USER_IMAGE_SCREEN_H_