Extracting placeholder information from Webkit to Blimp

content/browser/renderer_host/render_widget_host_view_aura.cc

Index: content/browser/renderer_host/render_widget_host_view_aura.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_aura.cc b/content/browser/renderer_host/render_widget_host_view_aura.cc
index fe65bc1d0651a379c827f89cdeadb5937170cf36..73cc0bc3a5655009c8966c0313ac7b7c269466ec 100644
--- a/content/browser/renderer_host/render_widget_host_view_aura.cc
+++ b/content/browser/renderer_host/render_widget_host_view_aura.cc
@@ -2993,6 +2993,10 @@ void RenderWidgetHostViewAura::OnUpdateTextInputStateCalled(
   if (state && state->show_ime_if_needed)
     GetInputMethod()->ShowImeIfNeeded();
 
+  if (state && host_->delegate())

[nyquist, 2016/10/05 04:20:56]

Nit: I think this needs curlies. It's multi-line.

[Charlie Reis, 2016/10/05 21:19:33]

Yes, that's right.

+    host_->delegate()->OnUpdateTextInputState(host_, state->type, state->value,
+                                              state->placeholder,
+                                              state->show_ime_if_needed);

[Charlie Reis, 2016/10/05 21:19:33]

Why is this only done in Aura?

If it's not needed on other platforms, we should more clearly document that in
the delegate's declaration, so that other people don't implement it and assume
it works everywhere.

 
   if (state && state->type != ui::TEXT_INPUT_TYPE_NONE) {
     // Start monitoring the composition information if the focused node is