OOPIF IME: Renderer Side Changes

content/renderer/render_view_impl.h

Index: content/renderer/render_view_impl.h
diff --git a/content/renderer/render_view_impl.h b/content/renderer/render_view_impl.h
index 65c98cb366aa78b77b8ee25ab316d700c4672976..9828d1ce83c67b26ab325c83ffc09d1ea75cbc44 100644
--- a/content/renderer/render_view_impl.h
+++ b/content/renderer/render_view_impl.h
@@ -240,6 +240,10 @@ class CONTENT_EXPORT RenderViewImpl
   // Plugin-related functions --------------------------------------------------
 
 #if defined(ENABLE_PLUGINS)
+  // TODO(ekaramad): This method is only used by TextInputClientObserver.
+  // Ideally, TextInputClientObserver should use RenderFrame/RenderWidget to
+  // obtain the plugin. Come back to this later when implementing IME for MAC

[Charlie Reis, 2016/07/12 00:00:12]

nit: s/MAC/Mac/

[EhsanK, 2016/07/12 15:13:24]

Done.

+  // to see if we can remove this API (https://crbug.com/578168).
   PepperPluginInstanceImpl* GetFocusedPepperPlugin();
 #endif  // ENABLE_PLUGINS
 
@@ -445,22 +449,7 @@ class CONTENT_EXPORT RenderViewImpl
   void OnResize(const ResizeParams& params) override;
   void OnSetFocus(bool enable) override;
   GURL GetURLForGraphicsContext3D() override;
-  void OnImeSetComposition(
-      const base::string16& text,
-      const std::vector<blink::WebCompositionUnderline>& underlines,
-      const gfx::Range& replacement_range,
-      int selection_start,
-      int selection_end) override;
-  void OnImeConfirmComposition(const base::string16& text,
-                               const gfx::Range& replacement_range,
-                               bool keep_selection) override;
   void OnOrientationChange() override;
-  ui::TextInputType GetTextInputType() override;
-  void GetSelectionBounds(gfx::Rect* start, gfx::Rect* end) override;
-  void GetCompositionCharacterBounds(
-      std::vector<gfx::Rect>* character_bounds_in_window) override;
-  void GetCompositionRange(gfx::Range* range) override;
-  bool CanComposeInline() override;
   void DidCommitCompositorFrame() override;
   void DidCompletePageScaleAnimation() override;
   void OnDeviceScaleFactorChanged() override;