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

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

Issue 539483002: Move all typecasting code to individual screens. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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 #include "chrome/browser/chromeos/login/screens/user_image_screen.h" 5 #include "chrome/browser/chromeos/login/screens/user_image_screen.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 50
51 // Time histogram suffix for profile image download. 51 // Time histogram suffix for profile image download.
52 const char kProfileDownloadReason[] = "OOBE"; 52 const char kProfileDownloadReason[] = "OOBE";
53 53
54 // Maximum amount of time to wait for the user image to sync. 54 // Maximum amount of time to wait for the user image to sync.
55 // The screen is shown iff sync failed or time limit exceeded. 55 // The screen is shown iff sync failed or time limit exceeded.
56 const int kSyncTimeoutSeconds = 10; 56 const int kSyncTimeoutSeconds = 10;
57 57
58 } // namespace 58 } // namespace
59 59
60 // static
61 UserImageScreen* UserImageScreen::Get(ScreenManager* manager) {
62 return static_cast<UserImageScreen*>(
63 manager->GetScreen(WizardController::kUserImageScreenName));
64 }
65
60 UserImageScreen::UserImageScreen(ScreenObserver* screen_observer, 66 UserImageScreen::UserImageScreen(ScreenObserver* screen_observer,
61 UserImageScreenActor* actor) 67 UserImageScreenActor* actor)
62 : WizardScreen(screen_observer), 68 : WizardScreen(screen_observer),
63 actor_(actor), 69 actor_(actor),
64 accept_photo_after_decoding_(false), 70 accept_photo_after_decoding_(false),
65 selected_image_(user_manager::User::USER_IMAGE_INVALID), 71 selected_image_(user_manager::User::USER_IMAGE_INVALID),
66 profile_picture_data_url_(url::kAboutBlankURL), 72 profile_picture_data_url_(url::kAboutBlankURL),
67 profile_picture_absent_(false), 73 profile_picture_absent_(false),
68 is_screen_ready_(false), 74 is_screen_ready_(false),
69 user_has_selected_image_(false) { 75 user_has_selected_image_(false) {
(...skipping 278 matching lines...) Expand 10 before | Expand all | Expand 10 after
348 354
349 void UserImageScreen::ExitScreen() { 355 void UserImageScreen::ExitScreen() {
350 policy_registrar_.reset(); 356 policy_registrar_.reset();
351 sync_timer_.reset(); 357 sync_timer_.reset();
352 if (UserImageSyncObserver* sync_observer = GetSyncObserver()) 358 if (UserImageSyncObserver* sync_observer = GetSyncObserver())
353 sync_observer->RemoveObserver(this); 359 sync_observer->RemoveObserver(this);
354 get_screen_observer()->OnExit(ScreenObserver::USER_IMAGE_SELECTED); 360 get_screen_observer()->OnExit(ScreenObserver::USER_IMAGE_SELECTED);
355 } 361 }
356 362
357 } // namespace chromeos 363 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698