Pipe "is pinned to left/right", "has horizontal/vertical scrollbar",

Pipe "is pinned to left/right", "has horizontal/vertical scrollbar",
"number of wheel handlers" to browser process.

We want to implement two-finger-swipe for history like safari does on
os x lion. The way this works is:
* If a page has no horizontal scrollbars, or the page is all the way
  to the left, scrolling to the left will go back in history instead.
* Same for right/forward.

Hence, this data needs to be available in the browser process.

This is modelled after the WebKit2 functions
WKPageIsPinnedToLeftSide(), WKPageIsPinnedToRightSide(),
WKPageHasHorizontalScrollbar(), and
webPageProxy::willHandleHorizontalScrollEvents().

Depends on https://bugs.webkit.org/show_bug.cgi?id=66094

BUG=90228
TEST=No observable effect yet


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96670

[jam, 2011-08-12 17:32:25]

http://codereview.chromium.org/7635011/diff/1001/content/renderer/render_view.h
File content/renderer/render_view.h (right):

http://codereview.chromium.org/7635011/diff/1001/content/renderer/render_view...
content/renderer/render_view.h:562: virtual void
numberOfWheelEventHandlersChanged(unsigned numHandlers);
nit: num_handlers

http://codereview.chromium.org/7635011/diff/1001/content/renderer/render_view...
content/renderer/render_view.h:563: virtual void
didChangeContentsSize(WebKit::WebFrame*, const WebKit::WebSize&);
chrome style is to always name the parameters in the headers

http://codereview.chromium.org/7635011/diff/1001/content/renderer/render_view...
content/renderer/render_view.h:738: void updateScrollState(WebKit::WebFrame*
frame);
chrome style is UpdateScrollState

[Nico, 2011-08-12 17:37:49]

http://codereview.chromium.org/7635011/diff/1001/content/renderer/render_view.h
File content/renderer/render_view.h (right):

http://codereview.chromium.org/7635011/diff/1001/content/renderer/render_view...
content/renderer/render_view.h:562: virtual void
numberOfWheelEventHandlersChanged(unsigned numHandlers);
On 2011/08/12 17:32:26, John Abd-El-Malek wrote:
> nit: num_handlers

Done.

http://codereview.chromium.org/7635011/diff/1001/content/renderer/render_view...
content/renderer/render_view.h:563: virtual void
didChangeContentsSize(WebKit::WebFrame*, const WebKit::WebSize&);
On 2011/08/12 17:32:26, John Abd-El-Malek wrote:
> chrome style is to always name the parameters in the headers

Done.

http://codereview.chromium.org/7635011/diff/1001/content/renderer/render_view...
content/renderer/render_view.h:738: void updateScrollState(WebKit::WebFrame*
frame);
On 2011/08/12 17:32:26, John Abd-El-Malek wrote:
> chrome style is UpdateScrollState

Done.

[jam, 2011-08-12 21:03:09]

lgtm

[commit-bot: I haz the power, 2011-08-13 03:44:34]

Change committed as 96670