Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(401)

Side by Side Diff: chrome/browser/chromeos/login/screens/user_image_screen.h

Issue 931993002: Make image_decoder a Leaky LazyInstance (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add ImageDecoder->RemoveDelegate Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_USER_IMAGE_SCREEN_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_USER_IMAGE_SCREEN_H_
6 #define CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_USER_IMAGE_SCREEN_H_ 6 #define CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_USER_IMAGE_SCREEN_H_
7 7
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "chrome/browser/chromeos/camera_presence_notifier.h" 10 #include "chrome/browser/chromeos/camera_presence_notifier.h"
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 bool is_user_selection) override; 56 bool is_user_selection) override;
57 void OnImageAccepted() override; 57 void OnImageAccepted() override;
58 void OnViewDestroyed(UserImageView* view) override; 58 void OnViewDestroyed(UserImageView* view) override;
59 59
60 // content::NotificationObserver implementation: 60 // content::NotificationObserver implementation:
61 void Observe(int type, 61 void Observe(int type,
62 const content::NotificationSource& source, 62 const content::NotificationSource& source,
63 const content::NotificationDetails& details) override; 63 const content::NotificationDetails& details) override;
64 64
65 // ImageDecoder::Delegate implementation: 65 // ImageDecoder::Delegate implementation:
66 void OnImageDecoded(const ImageDecoder* decoder, 66 void OnImageDecoded(const SkBitmap& decoded_image) override;
67 const SkBitmap& decoded_image) override; 67 void OnDecodeImageFailed() override;
68 void OnDecodeImageFailed(const ImageDecoder* decoder) override;
69 68
70 // CameraPresenceNotifier::Observer implementation: 69 // CameraPresenceNotifier::Observer implementation:
71 void OnCameraPresenceCheckDone(bool is_camera_present) override; 70 void OnCameraPresenceCheckDone(bool is_camera_present) override;
72 71
73 // UserImageSyncObserver::Observer implementation: 72 // UserImageSyncObserver::Observer implementation:
74 void OnInitialSync(bool local_image_updated) override; 73 void OnInitialSync(bool local_image_updated) override;
75 74
76 bool user_selected_image() const { return user_has_selected_image_; } 75 bool user_selected_image() const { return user_has_selected_image_; }
77 76
78 private: 77 private:
(...skipping 22 matching lines...) Expand all
101 100
102 // Closes the screen. 101 // Closes the screen.
103 void ExitScreen(); 102 void ExitScreen();
104 103
105 content::NotificationRegistrar notification_registrar_; 104 content::NotificationRegistrar notification_registrar_;
106 105
107 scoped_ptr<policy::PolicyChangeRegistrar> policy_registrar_; 106 scoped_ptr<policy::PolicyChangeRegistrar> policy_registrar_;
108 107
109 UserImageView* view_; 108 UserImageView* view_;
110 109
111 // Last ImageDecoder instance used to decode an image blob received by
112 // HandlePhotoTaken.
113 scoped_refptr<ImageDecoder> image_decoder_;
114
115 // Last user photo, if taken. 110 // Last user photo, if taken.
116 gfx::ImageSkia user_photo_; 111 gfx::ImageSkia user_photo_;
117 112
118 // If |true|, decoded photo should be immediately accepeted (i.e., both 113 // If |true|, decoded photo should be immediately accepeted (i.e., both
119 // HandleTakePhoto and HandleImageAccepted have already been called but we're 114 // HandleTakePhoto and HandleImageAccepted have already been called but we're
120 // still waiting for photo image decoding to finish. 115 // still waiting for photo image decoding to finish.
121 bool accept_photo_after_decoding_; 116 bool accept_photo_after_decoding_;
122 117
123 // Index of the selected user image. 118 // Index of the selected user image.
124 int selected_image_; 119 int selected_image_;
125 120
126 // Timer used for waiting for user image sync. 121 // Timer used for waiting for user image sync.
127 scoped_ptr<base::Timer> sync_timer_; 122 scoped_ptr<base::Timer> sync_timer_;
128 123
129 // If screen ready to be shown. 124 // If screen ready to be shown.
130 bool is_screen_ready_; 125 bool is_screen_ready_;
131 126
132 // True if user has explicitly selected some image. 127 // True if user has explicitly selected some image.
133 bool user_has_selected_image_; 128 bool user_has_selected_image_;
134 129
135 DISALLOW_COPY_AND_ASSIGN(UserImageScreen); 130 DISALLOW_COPY_AND_ASSIGN(UserImageScreen);
136 }; 131 };
137 132
138 } // namespace chromeos 133 } // namespace chromeos
139 134
140 #endif // CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_USER_IMAGE_SCREEN_H_ 135 #endif // CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_USER_IMAGE_SCREEN_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698