Refactor Autofill for out of process iframes (OOPIF).

components/password_manager/content/browser/content_password_manager_driver_factory.cc

Index: components/password_manager/content/browser/content_password_manager_driver_factory.cc
diff --git a/components/password_manager/content/browser/content_password_manager_driver_factory.cc b/components/password_manager/content/browser/content_password_manager_driver_factory.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e4c965790a53e5b6e7aaa26dc2d942e1131ffcfd
--- /dev/null
+++ b/components/password_manager/content/browser/content_password_manager_driver_factory.cc
@@ -0,0 +1,95 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "components/password_manager/content/browser/content_password_manager_driver_factory.h"
+
+#include "components/autofill/content/browser/content_autofill_driver.h"
+#include "components/autofill/content/browser/content_autofill_driver_factory.h"
+#include "components/autofill/content/common/autofill_messages.h"
+#include "components/autofill/core/common/form_data.h"
+#include "components/autofill/core/common/password_form.h"
+#include "components/password_manager/content/browser/content_password_manager_driver.h"
+#include "components/password_manager/core/browser/password_manager_client.h"
+#include "content/public/browser/browser_context.h"
+#include "content/public/browser/navigation_details.h"
+#include "content/public/browser/navigation_entry.h"
+#include "content/public/browser/render_frame_host.h"
+#include "content/public/browser/render_view_host.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/common/ssl_status.h"
+#include "ipc/ipc_message_macros.h"
+#include "net/cert/cert_status_flags.h"
+
+namespace password_manager {
+
+namespace {
+
+const char kContentPasswordManagerDriverFactoryWebContentsUserDataKey[] =
+    "web_contents_password_manager_driver_factory";
+
+}  // namespace
+
+ContentPasswordManagerDriverFactory::ContentPasswordManagerDriverFactory(
+    content::WebContents* web_contents,
+    PasswordManagerClient* password_client,
+    autofill::AutofillClient* autofill_client)
+    : content::WebContentsObserver(web_contents),
+      password_client_(password_client),
+      autofill_client_(autofill_client) {
+  web_contents->SetUserData(
+      kContentPasswordManagerDriverFactoryWebContentsUserDataKey, this);
+  if (web_contents->GetMainFrame())
+    RenderFrameCreated(web_contents->GetMainFrame());
+}
+
+ContentPasswordManagerDriverFactory::~ContentPasswordManagerDriverFactory() {
+}
+
+// static
+ContentPasswordManagerDriverFactory*
+ContentPasswordManagerDriverFactory::FromWebContents(
+    content::WebContents* contents) {
+  return static_cast<ContentPasswordManagerDriverFactory*>(
+      contents->GetUserData(
+          kContentPasswordManagerDriverFactoryWebContentsUserDataKey));
+}
+
+ContentPasswordManagerDriver*
+ContentPasswordManagerDriverFactory::GetDriverForFrame(
+    content::RenderFrameHost* render_frame_host) {
+  return frame_driver_map_[render_frame_host];

[vabr (Chromium), 2014/11/10 14:34:40]

As in the case of ContentAutofillDriverFactory, please be careful not to create
new entries in frame_driver_map_ through the use of the operator[]. Except for
line 66 where you indeed want to create a new entry, please prefer find() to
operator[] in this file.

[Evan Stade, 2014/11/14 23:25:50]

like with ContentAutofillDriverFactory, lookup should never fail; if it does,
there's a bug, and we should crash. This is equivalent to DCHECKing the return
value of find().

+}
+
+void ContentPasswordManagerDriverFactory::RenderFrameCreated(
+    content::RenderFrameHost* render_frame_host) {
+  frame_driver_map_[render_frame_host] = new ContentPasswordManagerDriver(
+      render_frame_host, password_client_, autofill_client_);
+}
+
+void ContentPasswordManagerDriverFactory::RenderFrameDeleted(
+    content::RenderFrameHost* render_frame_host) {
+  ContentPasswordManagerDriver* driver = frame_driver_map_[render_frame_host];
+  DCHECK(driver);
+  delete driver;
+  frame_driver_map_.erase(render_frame_host);
+}
+
+bool ContentPasswordManagerDriverFactory::OnMessageReceived(
+    const IPC::Message& message,
+    content::RenderFrameHost* render_frame_host) {
+  ContentPasswordManagerDriver* driver = frame_driver_map_[render_frame_host];
+  DCHECK(driver);
+  return driver->HandleMessage(message);
+}
+
+void ContentPasswordManagerDriverFactory::DidNavigateAnyFrame(
+    content::RenderFrameHost* render_frame_host,
+    const content::LoadCommittedDetails& details,
+    const content::FrameNavigateParams& params) {
+  ContentPasswordManagerDriver* driver = frame_driver_map_[render_frame_host];
+  DCHECK(driver);
+  driver->DidNavigateFrame(details, params);
+}
+
+}  // namespace password_manager