[android] Autofill popup behavior fixes.

chrome/browser/ui/autofill/autofill_popup_controller_interactive_uitest.cc

Index: chrome/browser/ui/autofill/autofill_popup_controller_interactive_uitest.cc
diff --git a/chrome/browser/ui/autofill/autofill_popup_controller_interactive_uitest.cc b/chrome/browser/ui/autofill/autofill_popup_controller_interactive_uitest.cc
index 977122aeb6bb19fc7745337e878d55afeabf1b5a..c96d865c04b49fa062921fefacc3dce2d7a5a62f 100644
--- a/chrome/browser/ui/autofill/autofill_popup_controller_interactive_uitest.cc
+++ b/chrome/browser/ui/autofill/autofill_popup_controller_interactive_uitest.cc
@@ -96,23 +96,20 @@ class AutofillPopupControllerBrowserTest
   scoped_ptr<TestAutofillExternalDelegate> autofill_external_delegate_;
 };
 
-// Autofill UI isn't currently hidden on window move on Mac.
-// http://crbug.com/180566
-#if !defined(OS_MACOSX)
 IN_PROC_BROWSER_TEST_F(AutofillPopupControllerBrowserTest,
-                       HidePopupOnWindowConfiguration) {
+                       HidePopupOnWindowResize) {
   GenerateTestAutofillPopup(autofill_external_delegate_.get());
 
   EXPECT_FALSE(autofill_external_delegate_->popup_hidden());
 
   // Resize the window, which should cause the popup to hide.
-  gfx::Rect new_bounds = browser()->window()->GetBounds() - gfx::Vector2d(1, 1);
+  gfx::Rect new_bounds = browser()->window()->GetBounds();

[Evan Stade, 2015/01/26 20:21:54]

seems like you should leave this test as is (perhaps change expectations to
reflect new expected behavior) and add another test for the resize behavior

[please use gerrit instead, 2015/01/26 23:49:03]

Changed expectations and added another test. The two tests are now called
DoNotHidePopupOnWindowMove and HidePopupOnWindowResize.

+  new_bounds.Inset(1, 1);
   browser()->window()->SetBounds(new_bounds);
 
   autofill_external_delegate_->WaitForPopupHidden();
   EXPECT_TRUE(autofill_external_delegate_->popup_hidden());
 }
-#endif // !defined(OS_MACOSX)
 
 // This test checks that the browser doesn't crash if the delegate is deleted
 // before the popup is hidden.