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

Side by Side Diff: chrome/browser/ui/passwords/manage_passwords_ui_controller.h

Issue 257153003: We have a problem in the process on destroying WebContentsImpl because (Closed) Base URL: https://git.chromium.org/chromium/src.git@master
Patch Set: Rebased onto origin/master again, resolved conflicts Created 6 years, 7 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_
6 #define CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_ 6 #define CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_
7 7
8 #include "components/password_manager/core/browser/password_form_manager.h" 8 #include "components/password_manager/core/browser/password_form_manager.h"
9 #include "components/password_manager/core/browser/password_store.h" 9 #include "components/password_manager/core/browser/password_store.h"
10 #include "components/password_manager/core/browser/password_store_change.h" 10 #include "components/password_manager/core/browser/password_store_change.h"
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 108
109 // Called when a passwordform is autofilled, when a new passwordform is 109 // Called when a passwordform is autofilled, when a new passwordform is
110 // submitted, or when a navigation occurs to update the visibility of the 110 // submitted, or when a navigation occurs to update the visibility of the
111 // manage passwords icon and bubble. 111 // manage passwords icon and bubble.
112 void UpdateBubbleAndIconVisibility(); 112 void UpdateBubbleAndIconVisibility();
113 113
114 // content::WebContentsObserver: 114 // content::WebContentsObserver:
115 virtual void DidNavigateMainFrame( 115 virtual void DidNavigateMainFrame(
116 const content::LoadCommittedDetails& details, 116 const content::LoadCommittedDetails& details,
117 const content::FrameNavigateParams& params) OVERRIDE; 117 const content::FrameNavigateParams& params) OVERRIDE;
118 virtual void WebContentsDestroyed( 118 virtual void WebContentsDestroyed() OVERRIDE;
119 content::WebContents* web_contents) OVERRIDE;
120 119
121 // Set by OnPasswordSubmitted() when the user submits a form containing login 120 // Set by OnPasswordSubmitted() when the user submits a form containing login
122 // information. If the user responds to a subsequent "Do you want to save 121 // information. If the user responds to a subsequent "Do you want to save
123 // this password?" prompt, we ask this object to save or blacklist the 122 // this password?" prompt, we ask this object to save or blacklist the
124 // associated login information in Chrome's password store. 123 // associated login information in Chrome's password store.
125 scoped_ptr<password_manager::PasswordFormManager> form_manager_; 124 scoped_ptr<password_manager::PasswordFormManager> form_manager_;
126 125
127 // Stores whether autofill was blocked due to a user's decision to blacklist 126 // Stores whether autofill was blocked due to a user's decision to blacklist
128 // the current site ("Never save passwords for this site"). 127 // the current site ("Never save passwords for this site").
129 bool autofill_blocked_; 128 bool autofill_blocked_;
130 129
131 // The origin of the form we're currently dealing with; we'll use this to 130 // The origin of the form we're currently dealing with; we'll use this to
132 // determine which PasswordStore changes we should care about when updating 131 // determine which PasswordStore changes we should care about when updating
133 // |password_form_map_|. 132 // |password_form_map_|.
134 GURL origin_; 133 GURL origin_;
135 134
136 DISALLOW_COPY_AND_ASSIGN(ManagePasswordsUIController); 135 DISALLOW_COPY_AND_ASSIGN(ManagePasswordsUIController);
137 }; 136 };
138 137
139 #endif // CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_ 138 #endif // CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698