Fix to allow the proxy authentication dialog to appear when hosting gaia in webview.

chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.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 "chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.h"
 
 #include "chrome/browser/extensions/chrome_extension_web_contents_observer.h"
 #include "chrome/browser/favicon/favicon_helper.h"
+#include "chrome/browser/password_manager/chrome_password_manager_client.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/renderer_context_menu/render_view_context_menu.h"
+#include "chrome/browser/ui/autofill/chrome_autofill_client.h"
+#include "chrome/browser/ui/passwords/manage_passwords_ui_controller.h"
 #include "chrome/browser/ui/pdf/chrome_pdf_web_contents_helper_client.h"
+#include "chrome/browser/ui/tab_dialogs.h"
 #include "chrome/common/chrome_version_info.h"
+#include "components/autofill/core/browser/autofill_manager.h"
 #include "components/browsing_data/storage_partition_http_cache_data_remover.h"
 #include "components/pdf/browser/pdf_web_contents_helper.h"
 #include "components/renderer_context_menu/context_menu_delegate.h"
 #include "components/web_cache/browser/web_cache_manager.h"
 #include "content/public/browser/render_process_host.h"
 #include "extensions/browser/api/web_request/web_request_api.h"
 #include "extensions/browser/guest_view/guest_view_event.h"
 #include "extensions/browser/guest_view/web_view/web_view_constants.h"
 
 #if defined(ENABLE_PRINTING)
@@ skipping 71 matching lines @@
   printing::PrintViewManager::CreateForWebContents(contents);
   printing::PrintPreviewMessageHandler::CreateForWebContents(contents);
 #else
   printing::PrintViewManagerBasic::CreateForWebContents(contents);
 #endif  // defined(ENABLE_PRINT_PREVIEW)
 #endif  // defined(ENABLE_PRINTING)
   pdf::PDFWebContentsHelper::CreateForWebContentsWithClient(
       contents,
       scoped_ptr<pdf::PDFWebContentsHelperClient>(
           new ChromePDFWebContentsHelperClient()));
+
+  // Autofill is currently only enabled for webview in WebUI.
+  if (!web_view_guest_->owner_web_contents()->GetWebUI())

[Fady Samuel, 2015/04/23 03:03:29]

I just realized this isn't a good way to determine you're in WebUI. This is not
nullptr in Chrome Apps too. See IsOwnedByExtension in this file:

https://codereview.chromium.org/1066563006/patch/120001/130016

[paulmeyer, 2015/04/23 23:35:27]

Okay, after talking to vabr@, I'm going to restrict these changes to just the
ChromeOS OOBE login case.

+    return;
+
+  autofill::ChromeAutofillClient::CreateForWebContents(contents);
+  autofill::ChromeAutofillClient* auto_fill_client =
+      autofill::ChromeAutofillClient::FromWebContents(contents);
+  ChromePasswordManagerClient::CreateForWebContentsWithAutofillClient(
+      contents, auto_fill_client);
+  ManagePasswordsUIController::CreateForWebContents(contents);
+  TabDialogs::CreateForWebContents(contents);
 }
 
 void ChromeWebViewGuestDelegate::OnDidCommitProvisionalLoadForFrame(
     bool is_main_frame) {
   if (is_main_frame)
     chromevox_injected_ = false;
 }
 
 void ChromeWebViewGuestDelegate::OnDidInitialize() {
 #if defined(OS_CHROMEOS)
@@ skipping 78 matching lines @@
       chromeos::ACCESSIBILITY_TOGGLE_SPOKEN_FEEDBACK) {
     if (details.enabled)
       InjectChromeVoxIfNeeded(guest_web_contents()->GetRenderViewHost());
     else
       chromevox_injected_ = false;
   }
 }
 #endif
 
 }  // namespace extensions