Always Close the Autofill UI through the same path

chrome/browser/ui/autofill/autofill_popup_controller_impl.h

Index: chrome/browser/ui/autofill/autofill_popup_controller_impl.h
diff --git a/chrome/browser/ui/autofill/autofill_popup_controller_impl.h b/chrome/browser/ui/autofill/autofill_popup_controller_impl.h
index e16ac31ba6708b831db9ae77f0e8adfbba06cf39..29df5d5a88ab0d22549aeb0b35cdc08b893bf39c 100644
--- a/chrome/browser/ui/autofill/autofill_popup_controller_impl.h
+++ b/chrome/browser/ui/autofill/autofill_popup_controller_impl.h
@@ -47,8 +47,8 @@ class AutofillPopupControllerImpl : public AutofillPopupController,
             const std::vector<int>& identifiers);
 
   // Hides the popup and destroys the controller. This also invalidates
-  // |delegate_|. Virtual for testing.
-  virtual void Hide();
+  // |delegate_|.
+  virtual void Hide() OVERRIDE;

[Ilya Sherman, 2013/02/22 00:29:04]

nit: Move this to be grouped with the other "AutofillPopupController
implementation." overrides.

[csharp, 2013/02/22 15:41:31]

This needs to be public though, the other functions are protected.

[Ilya Sherman, 2013/02/23 00:02:36]

Ah, awkward.  Dunno if Evan has a preference on how to treat methods that are
meant to be callable both by the controller's delegate and by the view.

 
   // KeyboardListener implementation.
   virtual bool HandleKeyPressEvent(