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

Side by Side Diff: chrome/browser/ui/views/login_view.h

Issue 869453002: Define class names for views class in c/b/ui/views (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: inline class name strings Created 5 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
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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_UI_VIEWS_LOGIN_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_LOGIN_VIEW_H_
6 #define CHROME_BROWSER_UI_VIEWS_LOGIN_VIEW_H_ 6 #define CHROME_BROWSER_UI_VIEWS_LOGIN_VIEW_H_
7 7
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "components/password_manager/core/browser/login_model.h" 9 #include "components/password_manager/core/browser/login_model.h"
10 #include "ui/views/view.h" 10 #include "ui/views/view.h"
11 11
12 namespace views { 12 namespace views {
13 class Label; 13 class Label;
14 class Textfield; 14 class Textfield;
15 } 15 }
16 16
17 // This class is responsible for displaying the contents of a login window 17 // This class is responsible for displaying the contents of a login window
18 // for HTTP/FTP authentication. 18 // for HTTP/FTP authentication.
19 class LoginView : public views::View, 19 class LoginView : public views::View,
20 public password_manager::LoginModelObserver { 20 public password_manager::LoginModelObserver {
21 public: 21 public:
22 // |model| is observed for the entire lifetime of the LoginView. 22 // |model| is observed for the entire lifetime of the LoginView.
23 // Therefore |model| should not be destroyed before the LoginView object. 23 // Therefore |model| should not be destroyed before the LoginView object.
24 LoginView(const base::string16& explanation, 24 LoginView(const base::string16& explanation,
25 password_manager::LoginModel* model); 25 password_manager::LoginModel* model);
26 ~LoginView() override; 26 ~LoginView() override;
27 27
28 // View:
Peter Kasting 2015/01/23 07:28:10 Nit: views::View:
oshima 2015/01/24 01:23:07 Done.
29 const char* GetClassName() const override;
30
28 // Access the data in the username/password text fields. 31 // Access the data in the username/password text fields.
29 const base::string16& GetUsername() const; 32 const base::string16& GetUsername() const;
30 const base::string16& GetPassword() const; 33 const base::string16& GetPassword() const;
31 34
32 // LoginModelObserver implementation. 35 // LoginModelObserver implementation.
33 void OnAutofillDataAvailable(const base::string16& username, 36 void OnAutofillDataAvailable(const base::string16& username,
34 const base::string16& password) override; 37 const base::string16& password) override;
35 void OnLoginModelDestroying() override; 38 void OnLoginModelDestroying() override;
36 39
37 // Used by LoginHandlerWin to set the initial focus. 40 // Used by LoginHandlerWin to set the initial focus.
(...skipping 12 matching lines...) Expand all
50 views::Label* message_label_; 53 views::Label* message_label_;
51 54
52 // If not null, points to a model we need to notify of our own destruction 55 // If not null, points to a model we need to notify of our own destruction
53 // so it doesn't try and access this when its too late. 56 // so it doesn't try and access this when its too late.
54 password_manager::LoginModel* login_model_; 57 password_manager::LoginModel* login_model_;
55 58
56 DISALLOW_COPY_AND_ASSIGN(LoginView); 59 DISALLOW_COPY_AND_ASSIGN(LoginView);
57 }; 60 };
58 61
59 #endif // CHROME_BROWSER_UI_VIEWS_LOGIN_VIEW_H_ 62 #endif // CHROME_BROWSER_UI_VIEWS_LOGIN_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698