Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(41)

Issue 133053004: Ensure that trackpad and trackpoint scrolling works on Windows desktop Aura with legacy trackpad dr… (Closed)

Created:
6 years, 11 months ago by ananta
Modified:
6 years, 11 months ago
Reviewers:
cpu_(ooo_6.6-7.5), sky
CC:
chromium-reviews, joi+watch-content_chromium.org, yusukes+watch_chromium.org, yukishiino+watch_chromium.org, tfarina, jam, penghuang+watch_chromium.org, sievers+watch_chromium.org, jbauman+watch_chromium.org, nona+watch_chromium.org, darin-cc_chromium.org, kalyank, piman+watch_chromium.org, danakj+watch_chromium.org, James Su, ben+views_chromium.org, miu+watch_chromium.org
Visibility:
Public.

Description

Ensure that trackpad and trackpoint scrolling works on Windows desktop Aura with legacy trackpad drivers. Legacy trackpad drivers like the older synaptics drivers have bugs where in they don't generate the WM_MOUSEWHEEL messages for applications unless there is an entry for the application with the class name of the window in their config file. If an entry does not exist here, these drivers expect the window to have the WS_VSCROLL/WS_HSCROLL style to generate the legacy WM_VSCROLL/WM_HSCROLL messages. This patch adds these styles to top level windows for Windows Aura. To ensure that Windows does not draw these scrollbars, we hide them when the window size changes and readd them in a posted task which works correctly in our testing. Changes in this patch are as below:- 1. Add handlers to the HWNDMessageHandler class for the WM_VSCROLL/WM_HSCROLL messages. 2. Forward the scroll events to the HWNDMessageHandlerDelegate which then forwards it off to its delegate. 3. Miscellaneous fixes in the events code to allow the WM_VSCROLL/WM_HSCROLL messages to go through. BUG=320849 R=cpu@chromium.org, sky@chromium.org, cpu, sky Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=244295

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Total comments: 10

Patch Set 4 : #

Total comments: 2

Patch Set 5 : #

Total comments: 2

Patch Set 6 : #

Total comments: 2

Patch Set 7 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+111 lines, -9 lines) Patch
M content/browser/renderer_host/render_widget_host_view_aura.cc View 1 2 3 4 5 6 1 chunk +4 lines, -0 lines 0 comments Download
M ui/events/win/events_win.cc View 1 2 3 4 5 6 4 chunks +22 lines, -7 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_root_window_host_win.h View 1 2 3 4 5 6 1 chunk +1 line, -0 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_root_window_host_win.cc View 1 2 3 4 5 6 1 chunk +5 lines, -0 lines 0 comments Download
M ui/views/widget/native_widget_win.h View 1 2 3 4 5 6 1 chunk +1 line, -0 lines 0 comments Download
M ui/views/widget/native_widget_win.cc View 1 2 3 4 5 6 2 chunks +5 lines, -1 line 0 comments Download
M ui/views/win/hwnd_message_handler.h View 1 2 3 4 5 6 3 chunks +8 lines, -0 lines 0 comments Download
M ui/views/win/hwnd_message_handler.cc View 1 2 3 4 5 6 5 chunks +61 lines, -1 line 0 comments Download
M ui/views/win/hwnd_message_handler_delegate.h View 1 2 3 4 5 6 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 14 (0 generated)
ananta
6 years, 11 months ago (2014-01-10 07:40:48 UTC) #1
sky
https://codereview.chromium.org/133053004/diff/420001/content/browser/renderer_host/render_widget_host_view_aura.cc File content/browser/renderer_host/render_widget_host_view_aura.cc (right): https://codereview.chromium.org/133053004/diff/420001/content/browser/renderer_host/render_widget_host_view_aura.cc#newcode2876 content/browser/renderer_host/render_widget_host_view_aura.cc:2876: #if !defined(OS_WIN) Should this be an in metro mode ...
6 years, 11 months ago (2014-01-10 16:43:20 UTC) #2
ananta
https://codereview.chromium.org/133053004/diff/420001/content/browser/renderer_host/render_widget_host_view_aura.cc File content/browser/renderer_host/render_widget_host_view_aura.cc (right): https://codereview.chromium.org/133053004/diff/420001/content/browser/renderer_host/render_widget_host_view_aura.cc#newcode2876 content/browser/renderer_host/render_widget_host_view_aura.cc:2876: #if !defined(OS_WIN) On 2014/01/10 16:43:21, sky wrote: > Should ...
6 years, 11 months ago (2014-01-10 18:59:20 UTC) #3
sky
https://codereview.chromium.org/133053004/diff/650001/ui/views/win/hwnd_message_handler.cc File ui/views/win/hwnd_message_handler.cc (right): https://codereview.chromium.org/133053004/diff/650001/ui/views/win/hwnd_message_handler.cc#newcode302 ui/views/win/hwnd_message_handler.cc:302: bool IsTopLevelWindow(HWND window) { Is this still needed? Can't ...
6 years, 11 months ago (2014-01-10 19:18:41 UTC) #4
ananta
https://codereview.chromium.org/133053004/diff/650001/ui/views/win/hwnd_message_handler.cc File ui/views/win/hwnd_message_handler.cc (right): https://codereview.chromium.org/133053004/diff/650001/ui/views/win/hwnd_message_handler.cc#newcode302 ui/views/win/hwnd_message_handler.cc:302: bool IsTopLevelWindow(HWND window) { On 2014/01/10 19:18:42, sky wrote: ...
6 years, 11 months ago (2014-01-10 20:05:36 UTC) #5
sky
LGTM
6 years, 11 months ago (2014-01-10 20:46:17 UTC) #6
cpu_(ooo_6.6-7.5)
lgtm
6 years, 11 months ago (2014-01-10 20:58:55 UTC) #7
cpu_(ooo_6.6-7.5)
https://codereview.chromium.org/133053004/diff/770002/ui/views/win/hwnd_message_handler.cc File ui/views/win/hwnd_message_handler.cc (right): https://codereview.chromium.org/133053004/diff/770002/ui/views/win/hwnd_message_handler.cc#newcode2075 ui/views/win/hwnd_message_handler.cc:2075: long current_style = ::GetWindowLong(hwnd(), GWL_STYLE); current_style not used also ...
6 years, 11 months ago (2014-01-10 21:10:18 UTC) #8
ananta
Hi Scott I moved the setting of the WS_HSCROLL/WS_VSCROLL styles back from the CalculateWindowStylesFromInitParams to ...
6 years, 11 months ago (2014-01-11 00:20:28 UTC) #9
ananta
On 2014/01/11 00:20:28, ananta wrote: > Hi Scott > > I moved the setting of ...
6 years, 11 months ago (2014-01-11 01:32:58 UTC) #10
cpu_(ooo_6.6-7.5)
lgtm https://codereview.chromium.org/133053004/diff/980001/ui/views/win/hwnd_message_handler.cc File ui/views/win/hwnd_message_handler.cc (right): https://codereview.chromium.org/133053004/diff/980001/ui/views/win/hwnd_message_handler.cc#newcode303 ui/views/win/hwnd_message_handler.cc:303: long style = GetWindowLong(window, GWL_STYLE); use :: For ...
6 years, 11 months ago (2014-01-11 01:34:29 UTC) #11
ananta
https://codereview.chromium.org/133053004/diff/980001/ui/views/win/hwnd_message_handler.cc File ui/views/win/hwnd_message_handler.cc (right): https://codereview.chromium.org/133053004/diff/980001/ui/views/win/hwnd_message_handler.cc#newcode303 ui/views/win/hwnd_message_handler.cc:303: long style = GetWindowLong(window, GWL_STYLE); On 2014/01/11 01:34:30, cpu ...
6 years, 11 months ago (2014-01-11 01:36:20 UTC) #12
ananta
Committed patchset #7 manually as r244295 (presubmit successful).
6 years, 11 months ago (2014-01-11 01:37:59 UTC) #13
sky
6 years, 11 months ago (2014-01-13 14:11:51 UTC) #14
Message was sent while issue was closed.
Ok, LGTM

Powered by Google App Engine
This is Rietveld 408576698