Autofill popup dismissed when page loses focus

chrome/browser/web_contents_view_win.cc

 // Copyright (c) 2006-2008 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/web_contents_view_win.h"
 
 #include <windows.h>
 
 #include "chrome/browser/bookmarks/bookmark_drag_data.h"
 #include "chrome/browser/browser.h"
@@ skipping 330 matching lines @@
   new_view->CreateView();
 
   // TODO(brettw) it seems bogus that we have to call this function on the
   // newly created object and give it one of its own member variables.
   new_view->CreateViewForWidget(new_contents->render_view_host());
   return new_contents;
 }
 
 RenderWidgetHostView* WebContentsViewWin::CreateNewWidgetInternal(
     int route_id,
-    bool focus_on_show) {
+    bool activatable) {
   // Create the widget and its associated view.
   // TODO(brettw) can widget creation be cross-platform?
   RenderWidgetHost* widget_host =
       new RenderWidgetHost(web_contents_->process(), route_id);
   RenderWidgetHostViewWin* widget_view =
       new RenderWidgetHostViewWin(widget_host);
 
   // We set the parent HWDN explicitly as pop-up HWNDs are parented and owned by
   // the first non-child HWND of the HWND that was specified to the CreateWindow
   // call.
   // TODO(brettw) this should not need to get the current RVHView from the
   // WebContents. We should have it somewhere ourselves.
   widget_view->set_parent_hwnd(
       web_contents_->render_widget_host_view()->GetPluginHWND());
   widget_view->set_close_on_deactivate(true);
-  widget_view->set_focus_on_show(focus_on_show);
+  widget_view->set_activatable(activatable);
 
   return widget_view;
 }
 
 void WebContentsViewWin::ShowCreatedWindowInternal(
     WebContents* new_web_contents,
     WindowOpenDisposition disposition,
     const gfx::Rect& initial_pos,
     bool user_gesture) {
   if (!new_web_contents->render_widget_host_view() ||
@@ skipping 24 matching lines @@
   }
 
   // This logic should be implemented by RenderWidgetHostHWND (as mentioned
   // above) in the ::Init function, which should take a parent and some initial
   // bounds.
   widget_host_view_win->Create(GetContainerHWND(), NULL, NULL,
                                WS_POPUP, WS_EX_TOOLWINDOW);
   widget_host_view_win->MoveWindow(initial_pos.x(), initial_pos.y(),
                                    initial_pos.width(), initial_pos.height(),
                                    TRUE);
-  widget_host_view_win->ShowWindow(widget_host_view_win->focus_on_show() ?
-                                   SW_SHOW : SW_SHOWNOACTIVATE);
+  widget_host_view_win->ShowWindow(widget_host_view_win->activatable() ?
+                                   SW_SHOW : SW_SHOWNA);
   widget_host->Init();
 }
 
 void WebContentsViewWin::OnHScroll(int scroll_type, short position,
                                    HWND scrollbar) {
   ScrollCommon(WM_HSCROLL, scroll_type, position, scrollbar);
 }
 
 void WebContentsViewWin::OnMouseLeave() {
   // Let our delegate know that the mouse moved (useful for resetting status
@@ skipping 213 matching lines @@
   }
   return false;
 }
 
 void WebContentsViewWin::WheelZoom(int distance) {
   if (web_contents_->delegate()) {
     bool zoom_in = distance > 0;
     web_contents_->delegate()->ContentsZoomChange(zoom_in);
   }
 }