Remove RenderViewObserver::Resized (try 2)

components/autofill/content/common/autofill_messages.h

Index: components/autofill/content/common/autofill_messages.h
diff --git a/components/autofill/content/common/autofill_messages.h b/components/autofill/content/common/autofill_messages.h
index bc2e5afbd9eed2a91820140f2e533efdb5ede16d..94a2748c2fab8f2b1b0cb079949446ea8d53e78e 100644
--- a/components/autofill/content/common/autofill_messages.h
+++ b/components/autofill/content/common/autofill_messages.h
@@ -155,6 +155,9 @@ IPC_MESSAGE_ROUTED1(AutofillMsg_FillFieldWithValue,
 IPC_MESSAGE_ROUTED1(AutofillMsg_PreviewFieldWithValue,
                     base::string16 /* value */)
 
+// The popup has been hidden.
+IPC_MESSAGE_ROUTED0(AutofillMsg_PopupHidden);

[Avi (use Gerrit), 2015/01/16 20:01:45]

Was hidden, or should be hidden?

[Evan Stade, 2015/01/16 20:29:14]

Was hidden. "Should be hidden" would be "HidePopup".

Not sure if this comment was a request to change the message name, but if so,
it's an established pattern that past participle indicates notification of an
action that already happened. See AccountCreationFormsDetected,
FirstUserGestureObserved, FormSeen, etc.

[Avi (use Gerrit), 2015/01/16 20:35:12]

I got that. I misunderstood the code flow, and thought that this was a request
to the renderer to do the hiding rather than a notification that it had been
done.

+
 // Sets the currently selected node's value to be the given data list value.
 IPC_MESSAGE_ROUTED1(AutofillMsg_AcceptDataListSuggestion,
                     base::string16 /* accepted data list value */)