Return the WebViewFrameWidget in RenderWidget::webwidget() if there

third_party/WebKit/Source/web/WebViewFrameWidget.h

Index: third_party/WebKit/Source/web/WebViewFrameWidget.h
diff --git a/third_party/WebKit/Source/web/WebViewFrameWidget.h b/third_party/WebKit/Source/web/WebViewFrameWidget.h
index 4da770b6e6d0d148e6670e121171b100eabae123..3f2f33a0da79d66b1876ef113c697f9ddbd63f65 100644
--- a/third_party/WebKit/Source/web/WebViewFrameWidget.h
+++ b/third_party/WebKit/Source/web/WebViewFrameWidget.h
@@ -87,6 +87,8 @@ public:
     void didChangeWindowResizerRect() override;
     WebColor backgroundColor() const override;
     WebPagePopup* pagePopup() const override;
+    bool getCompositionCharacterBounds(WebVector<WebRect>& bounds) override;
+    void applyReplacementRange(int start, int length) override;

[dcheng, 2016/09/08 19:42:16]

How come these lines are in this patch?

[lfg, 2016/09/08 19:58:24]

When they were added, they were added only to WebViewImpl, without being in
WebViewFrameWidget. Since with this patch we are now returning a
WebViewFrameWidget, these calls became a no-op unless we forward them.

BTW, this is exactly what this change tries to prevent.

     void updateTopControlsState(WebTopControlsState constraints, WebTopControlsState current, bool animate) override;
     void setVisibilityState(WebPageVisibilityState) override;
     bool isTransparent() const override;