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

Unified Diff: chrome/browser/prerender/prerender_tab_helper.h

Issue 19705013: [password autofill] Remove references to PasswordForm from RenderViewImpl (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 7 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/prerender/prerender_tab_helper.h
diff --git a/chrome/browser/prerender/prerender_tab_helper.h b/chrome/browser/prerender/prerender_tab_helper.h
index 1e889ceb854a95d6171a29407ac3c6dee62cac25..51e8b398b2c689cf5278f3a987b49c54ed22b998 100644
--- a/chrome/browser/prerender/prerender_tab_helper.h
+++ b/chrome/browser/prerender/prerender_tab_helper.h
@@ -8,6 +8,7 @@
#include "base/memory/scoped_ptr.h"
#include "base/memory/weak_ptr.h"
#include "base/time/time.h"
+#include "chrome/browser/password_manager/password_manager.h"
#include "content/public/browser/web_contents_observer.h"
#include "content/public/browser/web_contents_user_data.h"
#include "url/gurl.h"
@@ -24,7 +25,8 @@ class PrerenderManager;
// to compare PLT's with prerendering enabled and disabled.
class PrerenderTabHelper
: public content::WebContentsObserver,
- public content::WebContentsUserData<PrerenderTabHelper> {
+ public content::WebContentsUserData<PrerenderTabHelper>,
+ public PasswordManager::SubmissionObserver {
public:
enum Event {
EVENT_LOGGED_IN_TABLE_REQUESTED = 0,
@@ -37,6 +39,10 @@ class PrerenderTabHelper
EVENT_MAX_VALUE
};
+ static void CreateForWebContentsWithPasswordManager(
+ content::WebContents* web_contents,
+ PasswordManager* password_manager);
+
virtual ~PrerenderTabHelper();
// content::WebContentsObserver implementation.
@@ -59,15 +65,16 @@ class PrerenderTabHelper
const GURL& validated_url,
content::PageTransition transition_type,
content::RenderViewHost* render_view_host) OVERRIDE;
- virtual void DidNavigateAnyFrame(
- const content::LoadCommittedDetails& details,
- const content::FrameNavigateParams& params) OVERRIDE;
+
+ // PasswordManager::SubmissionObserver
+ virtual void PasswordSubmitted(const content::PasswordForm& form) OVERRIDE;
// Called when this prerendered WebContents has just been swapped in.
void PrerenderSwappedIn();
private:
- explicit PrerenderTabHelper(content::WebContents* web_contents);
+ PrerenderTabHelper(content::WebContents* web_contents,
+ PasswordManager* password_manager);
friend class content::WebContentsUserData<PrerenderTabHelper>;
void RecordEvent(Event event) const;

Powered by Google App Engine
This is Rietveld 408576698