[Autofill]Enable mojo AutofillDriver for android_webview.

android_webview/browser/aw_content_browser_client.cc

 // Copyright (c) 2012 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 "android_webview/browser/aw_content_browser_client.h"
 
 #include <utility>
 
 #include "android_webview/browser/aw_browser_context.h"
 #include "android_webview/browser/aw_browser_main_parts.h"
@@ skipping 15 matching lines @@
 #include "android_webview/common/render_view_messages.h"
 #include "android_webview/common/url_constants.h"
 #include "android_webview/grit/aw_resources.h"
 #include "base/android/locale_utils.h"
 #include "base/base_paths_android.h"
 #include "base/command_line.h"
 #include "base/files/scoped_file.h"
 #include "base/json/json_reader.h"
 #include "base/memory/ptr_util.h"
 #include "base/path_service.h"
+#include "components/autofill/content/browser/content_autofill_driver_factory.h"
 #include "components/cdm/browser/cdm_message_filter_android.h"
 #include "components/crash/content/browser/crash_micro_dump_manager_android.h"
 #include "components/navigation_interception/intercept_navigation_delegate.h"
 #include "content/public/browser/browser_message_filter.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/child_process_security_policy.h"
 #include "content/public/browser/client_certificate_delegate.h"
 #include "content/public/browser/navigation_handle.h"
 #include "content/public/browser/navigation_throttle.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/service_names.h"
 #include "content/public/common/url_constants.h"
 #include "content/public/common/web_preferences.h"
 #include "device/geolocation/access_token_store.h"
 #include "device/geolocation/geolocation_delegate.h"
 #include "net/android/network_library.h"
 #include "net/ssl/ssl_cert_request_info.h"
 #include "net/ssl/ssl_info.h"
+#include "services/service_manager/public/cpp/interface_registry.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/base/resource/resource_bundle_android.h"
 #include "ui/resources/grit/ui_resources.h"
 
 #if defined(ENABLE_SPELLCHECK)
 #include "components/spellcheck/browser/spellcheck_message_filter_platform.h"
 #include "components/spellcheck/common/spellcheck_switches.h"
 #endif
 
 using content::BrowserThread;
@@ skipping 464 matching lines @@
     id = IDR_AW_BROWSER_MANIFEST_OVERLAY;
   if (id == -1)
     return nullptr;
 
   base::StringPiece manifest_contents =
       ui::ResourceBundle::GetSharedInstance().GetRawDataResourceForScale(
           id, ui::ScaleFactor::SCALE_FACTOR_NONE);
   return base::JSONReader::Read(manifest_contents);
 }
 
+void AwContentBrowserClient::RegisterRenderFrameMojoInterfaces(
+    service_manager::InterfaceRegistry* registry,
+    content::RenderFrameHost* render_frame_host) {
+  registry->AddInterface(
+      base::Bind(&autofill::ContentAutofillDriverFactory::BindAutofillDriver,

[vabr (Chromium), 2016/11/11 09:07:42]

When you bind this, does it mean that the browser-side will work the same in
WebView as it does in normal Chrome? In other words, would it be now left with
the renderer process to drop information it should not have access to?

[sgurun-gerrit only, 2016/11/18 23:51:28]

yes it works the same way as chrome. Specifically, when a form is seen, the
renderer sends a mojo message for forms seen:
https://cs.chromium.org/chromium/src/out/Debug/gen/components/autofill/conten...

which I think is the most important piece for webview.

[leonhsl(Using Gerrit), 2016/11/19 12:05:31]

Thanks sgurun@ a lot for the clarifications. As for the concern about renderer
process getting unnecessary information, I think:
  1, At least autofill mojo IPC is keeping the same logic with legacy chromium
IPC before, mojo is not making things worse. 
  2, It's true that chrome and webview have the same capability of sending
autofill mojo IPC messages, but, comparing the implementation of webview
autofill client with chrome autofill client, we can see that webview autofill
client is handling only those interested features, such as autocomplete, and is
ignoring others by empty or NOTIMPLEMENTED() functions, this means that actually
many mojo messages(except autocomplete) will never be triggered to be sent to
renderer process.

+                 render_frame_host));
+}
+
 }  // namespace android_webview