[autofill] Ingore mouse move on Windows for the first 50ms.

chrome/browser/ui/views/autofill/autofill_popup_base_view.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/ui/views/autofill/autofill_popup_base_view.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/message_loop/message_loop.h"
 #include "chrome/browser/ui/autofill/popup_constants.h"
@@ skipping 48 matching lines @@
     // deletion.
     views::Widget* widget = new views::Widget;
     views::Widget::InitParams params(views::Widget::InitParams::TYPE_POPUP);
     params.delegate = this;
     params.parent = container_view();
     widget->Init(params);
     widget->SetContentsView(this);
 
     // No animation for popup appearance (too distracting).
     widget->SetVisibilityAnimationTransition(views::Widget::ANIMATE_HIDE);
+
+    show_time_ = base::Time::Now();
   }
 
   SetBorder(views::Border::CreateSolidBorder(kPopupBorderThickness,
                                              kBorderColor));
 
   DoUpdateBoundsAndRedrawPopup();
   GetWidget()->Show();
 
   // Showing the widget can change native focus (which would result in an
   // immediate hiding of the popup). Only start observing after shown.
@@ skipping 57 matching lines @@
   // via AcceleratorPressed, so we need to let that run first.
   base::MessageLoop::current()->PostTask(
       FROM_HERE,
       base::Bind(&AutofillPopupBaseView::ClearSelection,
                  weak_ptr_factory_.GetWeakPtr()));
 }
 
 void AutofillPopupBaseView::OnMouseMoved(const ui::MouseEvent& event) {
   // A synthesized mouse move will be sent when the popup is first shown.
   // Don't preview a suggestion if the mouse happens to be hovering there.
+#if defined(OS_WIN)
+  static const int64 kMouseMoveIngoreMs = 50;
+  if ((base::Time::Now() - show_time_).InMilliseconds() <= kMouseMoveIngoreMs)

[Evan Stade, 2015/03/19 20:16:30]

if (base::Time::Now() < show_time_ + TimeDelta::FromMilliseconds(50))

but I feel like we should be using some timestamp on |event| rather than |now|
(there can be a big difference if Chrome is bogged down)

[please use gerrit instead, 2015/03/20 17:36:43]

Done.

+    return;
+#else
   if (event.flags() & ui::EF_IS_SYNTHESIZED)
     return;
+#endif
 
   if (HitTestPoint(event.location()))
     SetSelection(event.location());
   else
     ClearSelection();
 }
 
 bool AutofillPopupBaseView::OnMousePressed(const ui::MouseEvent& event) {
   return event.GetClickCount() == 1;
 }
@@ skipping 68 matching lines @@
 void AutofillPopupBaseView::HideController() {
   if (delegate_)
     delegate_->Hide();
 }
 
 gfx::NativeView AutofillPopupBaseView::container_view() {
   return delegate_->container_view();
 }
 
 }  // namespace autofill