Bug 22698

Summary: In Chromium, pan scrolling should only be enabled on windows.
Product: WebKit Reporter: Tony Chang <tony>
Component: PlatformAssignee: Tony Chang <tony>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
[1/1] Only have pan scrolling on chromium win
eric: review-
[1/1] Only have pan scrolling on for WIN_OS eric: review+

Description Tony Chang 2008-12-05 15:59:20 PST 
this is how it works on other platforms (wx, qt, etc)
Comment 1 Tony Chang 2008-12-05 16:05:13 PST 
Created attachment 25798 [details]
[1/1] Only have pan scrolling on chromium win

 JavaScriptCore/ChangeLog      |    8 ++++++++
 JavaScriptCore/wtf/Platform.h |    2 +-
 2 files changed, 9 insertions(+), 1 deletions(-)
Comment 2 Eric Seidel (no email) 2008-12-05 17:35:15 PST 
Comment on attachment 25798 [details]
[1/1] Only have pan scrolling on chromium win

I think we should just turn in on by default for WIN_OS.  Yeah, that turns it on for qt, but I expect they want it anyway.
Comment 3 Tony Chang 2008-12-05 17:38:58 PST 
Created attachment 25803 [details]
[1/1] Only have pan scrolling on for WIN_OS

 JavaScriptCore/ChangeLog      |    8 ++++++++
 JavaScriptCore/wtf/Platform.h |    2 +-
 2 files changed, 9 insertions(+), 1 deletions(-)
Comment 4 Eric Seidel (no email) 2008-12-05 17:43:38 PST 
Comment on attachment 25803 [details]
[1/1] Only have pan scrolling on for WIN_OS

LGTM.
Comment 5 Eric Seidel (no email) 2008-12-08 15:43:56 PST 
	M	JavaScriptCore/ChangeLog
	M	JavaScriptCore/wtf/Platform.h
Committed r39106