Extracting placeholder information from Webkit to Blimp

content/public/browser/web_contents_delegate.h

Index: content/public/browser/web_contents_delegate.h
diff --git a/content/public/browser/web_contents_delegate.h b/content/public/browser/web_contents_delegate.h
index b7d17d294e0b56b8f18bba5df9fe567d5042fe78..59dec355d1162c4c359c5ff34b11c400da62ac33 100644
--- a/content/public/browser/web_contents_delegate.h
+++ b/content/public/browser/web_contents_delegate.h
@@ -22,6 +22,7 @@
 #include "third_party/WebKit/public/platform/WebDisplayMode.h"
 #include "third_party/WebKit/public/platform/WebDragOperation.h"
 #include "third_party/skia/include/core/SkColor.h"
+#include "ui/base/ime/text_input_type.h"
 #include "ui/base/window_open_disposition.h"
 #include "ui/gfx/geometry/rect_f.h"
 #include "ui/gfx/native_widget_types.h"
@@ -529,6 +530,13 @@ class CONTENT_EXPORT WebContentsDelegate {
       RenderWidgetHost* render_widget_host,
       const std::vector<uint8_t>& proto) {}
 
+  virtual void OnUpdateTextInputState(

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

All public methods here should have comments.

Also, it's a bigger problem here if this only works on Aura. 
WebContentsDelegate methods are generally expected to be platform agnostic,
unless it's clear otherwise (e.g., ifdef or clear explanation).  I'm not sure it
makes sense to only have this for one platform.

+      content::RenderWidgetHost* render_widget_host,
+      ui::TextInputType type,
+      const std::string& text,
+      const std::string& placeholder,
+      bool show_ime_if_needed) {}
+
   // Requests the app banner. This method is called from the DevTools.
   virtual void RequestAppBannerFromDevTools(content::WebContents* web_contents);