Add support for touch based zoom gesture on Windows. This change also handles the WM_POINTERDOWN

Add support for touch based zoom gesture on Windows. This change also handles the WM_POINTERDOWN
and WM_POINTERUP messages to scroll the page if we touch the scrollbar.

BUG=none
TEST=Touch based zoom should work on Windows.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=108232

[jschuh, 2011-10-28 01:40:41]

lgtm with a minor nit

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_view_win.h (right):

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_win.h:52: 
Would be good to have a TODO here with a bug for rolling up the SDK once it's
released.

[cpu_(ooo_6.6-7.5), 2011-10-28 02:30:03]

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_view_win.cc (right):

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_win.cc:231: 
*zoom = distance < start ? zoom_out : zoom_in

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_win.cc:1582:
::CloseGestureInfoHandle(gi_handle);
dont call close here.

http://codereview.chromium.org/8399009/diff/4002/content/renderer/render_view...
File content/renderer/render_view_impl.cc (right):

http://codereview.chromium.org/8399009/diff/4002/content/renderer/render_view...
content/renderer/render_view_impl.cc:3562: }
is it ok to put zoomLevelChanged() in line 3561 and remove it from 3588?

[ananta, 2011-10-28 18:42:02]

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_view_win.cc (right):

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_win.cc:231: 
On 2011/10/28 02:30:03, cpu wrote:
> *zoom = distance < start ? zoom_out : zoom_in

Done.

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_win.cc:1582:
::CloseGestureInfoHandle(gi_handle);
On 2011/10/28 02:30:03, cpu wrote:
> dont call close here.

Done.

http://codereview.chromium.org/8399009/diff/4002/content/renderer/render_view...
File content/renderer/render_view_impl.cc (right):

http://codereview.chromium.org/8399009/diff/4002/content/renderer/render_view...
content/renderer/render_view_impl.cc:3562: }
On 2011/10/28 02:30:03, cpu wrote:
> is it ok to put zoomLevelChanged() in line 3561 and remove it from 3588?

We need to call zoomLevelChanged in the OnZoomFactor IPC handler anyway. I left
this as is.

[ananta, 2011-10-28 18:43:10]

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_view_win.h (right):

http://codereview.chromium.org/8399009/diff/4002/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_win.h:52: 
On 2011/10/28 01:40:41, Justin Schuh wrote:
> Would be good to have a TODO here with a bug for rolling up the SDK once it's
> released.

Done.

[jam, 2011-10-28 19:26:19]

rubber stamp lgtm

[cpu_(ooo_6.6-7.5), 2011-10-28 19:53:56]

lgtm

[ananta, 2011-11-01 01:30:59]

On 2011/10/28 19:53:56, cpu wrote:
> lgtm
I found that the zoom factor stuff was flaky. Turned out that the zoom changed
notification coming in from the renderer was always sending the level as 0.
Refactored the zoomLevelChanged function into a helper function which is now
called from both places.

PTAL.

Thanks
Ananta

[James Cook, 2011-11-02 02:29:21]

On 2011/11/01 01:30:59, ananta wrote:
> On 2011/10/28 19:53:56, cpu wrote:
> > lgtm
> I found that the zoom factor stuff was flaky. Turned out that the zoom changed
> notification coming in from the renderer was always sending the level as 0.
> Refactored the zoomLevelChanged function into a helper function which is now
> called from both places.
> 
> PTAL.
> 
> Thanks
> Ananta

This change broke the linux touch build ("function" in render_view_impl.cc
should probably be "zoom").  I reverted it.