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

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

Issue 2633413002: cros: Remove BaseScreen::Initialize method. (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « no previous file | chrome/browser/chromeos/login/screens/base_screen.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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_BASE_SCREEN_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_BASE_SCREEN_H_
6 #define CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_BASE_SCREEN_H_ 6 #define CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_BASE_SCREEN_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/gtest_prod_util.h" 10 #include "base/gtest_prod_util.h"
(...skipping 25 matching lines...) Expand all
36 // ---- Old implementation ---- 36 // ---- Old implementation ----
37 37
38 // Makes wizard screen visible. 38 // Makes wizard screen visible.
39 virtual void Show() = 0; 39 virtual void Show() = 0;
40 40
41 // Makes wizard screen invisible. 41 // Makes wizard screen invisible.
42 virtual void Hide() = 0; 42 virtual void Hide() = 0;
43 43
44 // ---- New Implementation ---- 44 // ---- New Implementation ----
45 45
46 // Called to perform initialization of the screen. UI is guaranteed to exist
47 // at this point. Screen can alter context, resulting context will be passed
48 // to JS. This method will be called once per instance of the Screen object.
49 virtual void Initialize(::login::ScreenContext* context);
50
51 // Called when screen appears. 46 // Called when screen appears.
52 virtual void OnShow(); 47 virtual void OnShow();
53 // Called when screen disappears, either because it finished it's work, or 48 // Called when screen disappears, either because it finished it's work, or
54 // because some other screen pops up. 49 // because some other screen pops up.
55 virtual void OnHide(); 50 virtual void OnHide();
56 51
57 // Called when we navigate from screen so that we will never return to it. 52 // Called when we navigate from screen so that we will never return to it.
58 // This is a last chance to call JS counterpart, this object will be deleted 53 // This is a last chance to call JS counterpart, this object will be deleted
59 // soon. 54 // soon.
60 virtual void OnClose(); 55 virtual void OnClose();
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 BaseScreenDelegate* base_screen_delegate_ = nullptr; 148 BaseScreenDelegate* base_screen_delegate_ = nullptr;
154 149
155 const OobeScreen screen_id_; 150 const OobeScreen screen_id_;
156 151
157 DISALLOW_COPY_AND_ASSIGN(BaseScreen); 152 DISALLOW_COPY_AND_ASSIGN(BaseScreen);
158 }; 153 };
159 154
160 } // namespace chromeos 155 } // namespace chromeos
161 156
162 #endif // CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_BASE_SCREEN_H_ 157 #endif // CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_BASE_SCREEN_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/chromeos/login/screens/base_screen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698