Move setWindowRect and windowRect calls from WebViewClient to WebWidgetClient.

third_party/WebKit/Source/core/page/ChromeClient.h

Index: third_party/WebKit/Source/core/page/ChromeClient.h
diff --git a/third_party/WebKit/Source/core/page/ChromeClient.h b/third_party/WebKit/Source/core/page/ChromeClient.h
index 762d7844a784f046c7bdad2895f671c6c6d7b628..2b2e42eeba31e6d23dd141c5e39777f7b6d6324b 100644
--- a/third_party/WebKit/Source/core/page/ChromeClient.h
+++ b/third_party/WebKit/Source/core/page/ChromeClient.h
@@ -85,8 +85,8 @@ public:
 
     // The specified rectangle is adjusted for the minimum window size and the
     // screen, then setWindowRect with the adjusted rectangle is called.
-    void setWindowRectWithAdjustment(const IntRect&);
-    virtual IntRect windowRect() = 0;
+    void setWindowRectWithAdjustment(const IntRect&, LocalFrame*);

[dcheng, 2016/08/15 18:22:44]

Let's make this a LocalFrame&.

[lfg, 2016/08/16 21:02:13]

I know this is blink, but this goes against the google style guide of not
passing non-const refs. Can you explain the reasoning for this?

[dcheng, 2016/08/16 21:03:46]

We have a strong convention of passing never-null pointers as mutable refs. Yes,
it is contrary to the Google style guide, but this is one of the Blink
exceptions in place to enable the overall style merge.

+    virtual IntRect rootWindowRect() = 0;
 
     virtual IntRect pageRect() = 0;
 
@@ -288,7 +288,7 @@ protected:
     ~ChromeClient() override { }
 
     virtual void showMouseOverURL(const HitTestResult&) = 0;
-    virtual void setWindowRect(const IntRect&) = 0;
+    virtual void setWindowRect(const IntRect&, LocalFrame*) = 0;

[dcheng, 2016/08/15 18:22:44]

Ditto

     virtual bool openBeforeUnloadConfirmPanelDelegate(LocalFrame*, bool isReload) = 0;
     virtual bool openJavaScriptAlertDelegate(LocalFrame*, const String&) = 0;
     virtual bool openJavaScriptConfirmDelegate(LocalFrame*, const String&) = 0;