[Autofill] Componentize AutofillCCInfoBarDelegate.

components/autofill/content/browser/content_autofill_driver.cc

 // 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/autofill/content/browser/content_autofill_driver.h"
 
 #include "base/command_line.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "components/autofill/content/common/autofill_messages.h"
 #include "components/autofill/core/browser/autofill_client.h"
 #include "components/autofill/core/browser/autofill_external_delegate.h"
 #include "components/autofill/core/browser/autofill_manager.h"
 #include "components/autofill/core/browser/form_structure.h"
 #include "components/autofill/core/common/autofill_switches.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/site_instance.h"
+#include "content/public/browser/web_contents.h"
 #include "ipc/ipc_message_macros.h"
 
 namespace autofill {
 
 ContentAutofillDriver::ContentAutofillDriver(
     content::RenderFrameHost* render_frame_host,
     AutofillClient* client,
     const std::string& app_locale,
     AutofillManager::AutofillDownloadManagerState enable_download_manager)
     : render_frame_host_(render_frame_host),
@@ skipping 106 matching lines @@
 }
 
 void ContentAutofillDriver::RendererShouldPreviewFieldWithValue(
     const base::string16& value) {
   if (!RendererIsAvailable())
     return;
   render_frame_host_->Send(new AutofillMsg_PreviewFieldWithValue(
       render_frame_host_->GetRoutingID(), value));
 }
 
+void ContentAutofillDriver::LinkClicked(const GURL& url,
+                                        WindowOpenDisposition disposition) {
+  content::WebContents* web_contents =
+      content::WebContents::FromRenderFrameHost(render_frame_host_);
+  web_contents->OpenURL(content::OpenURLParams(
+      url, content::Referrer(), disposition, ui::PAGE_TRANSITION_LINK, false));
+}

[Ilya Sherman, 2015/01/06 02:54:59]

When you started working on this CL, there was one driver per WebContents,
rather than one driver per frame.  At this point, it might be easier to instead
provide LinkClicked as a method on the client.

[Pritam Nikam, 2015/01/06 12:47:42]

Done.

Abstracting the call to LinkClicked throuh the AutofillClient.

+
 bool ContentAutofillDriver::HandleMessage(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(ContentAutofillDriver, message)
   IPC_MESSAGE_FORWARD(AutofillHostMsg_FirstUserGestureObserved, client_,
                       AutofillClient::OnFirstUserGestureObserved)
   IPC_MESSAGE_FORWARD(AutofillHostMsg_FormsSeen,
                       autofill_manager_.get(),
                       AutofillManager::OnFormsSeen)
   IPC_MESSAGE_FORWARD(AutofillHostMsg_FormSubmitted,
                       autofill_manager_.get(),
@@ skipping 40 matching lines @@
     autofill_manager_->Reset();
 }
 
 void ContentAutofillDriver::SetAutofillManager(
     scoped_ptr<AutofillManager> manager) {
   autofill_manager_ = manager.Pass();
   autofill_manager_->SetExternalDelegate(&autofill_external_delegate_);
 }
 
 }  // namespace autofill