Bug 39918

Summary: REGRESSION(58615): Scroll events are sent twice per keypress for ports that don't have a platformWidget scrollbar
Product: WebKit Reporter: Nico Weber <thakis>
Component: New BugsAssignee: Nico Weber <thakis>
Status: CLOSED FIXED    
Severity: Normal CC: abarth, commit-queue, eric, hausmann, jamesr, ossy, simon.fraser, thakis, webkit-ews, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Bug Depends on: 40102    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Description Nico Weber 2010-05-28 17:31:49 PDT
Scroll events are sent twice per keypress for ports that don't have a platformWidget scrollbar
Comment 1 Nico Weber 2010-05-28 17:33:59 PDT
Created attachment 57397 [details]
Patch
Comment 2 Nico Weber 2010-05-28 17:53:23 PDT
+Simon, who merged the regressing patch into qtwebkit-2.0.
Comment 3 James Robinson 2010-05-28 17:59:14 PDT
This patch looks right, I missed this side effect in http://trac.webkit.org/changeset/58615.
Comment 4 Early Warning System Bot 2010-05-28 18:04:52 PDT
Attachment 57397 [details] did not build on qt:
Build output: http://webkit-commit-queue.appspot.com/results/2632055
Comment 5 Nico Weber 2010-05-28 18:11:38 PDT
Created attachment 57399 [details]
Patch
Comment 6 Nico Weber 2010-05-28 19:02:20 PDT
This is http://crbug.com/43958 btw.
Comment 7 Simon Fraser (smfr) 2010-06-02 11:32:16 PDT
Comment on attachment 57399 [details]
Patch

> +    void scrollPositionChanged();
> +    virtual void repaintFixedElementsPostScroll();

We don't use the term "postScroll" anywhere else, but we do use "afterScrolling", so I suggest you change the method name to use that.
Comment 8 Nico Weber 2010-06-02 11:39:09 PDT
Created attachment 57673 [details]
Patch
Comment 9 Nico Weber 2010-06-02 11:40:57 PDT
Comment on attachment 57673 [details]
Patch

Earlier patch got r+ from smfr as can be seen here: https://bugs.webkit.org/show_activity.cgi?id=39918 Looks like webkit-patch clobbered that information.
Comment 10 WebKit Commit Bot 2010-06-02 19:10:36 PDT
Comment on attachment 57673 [details]
Patch

Clearing flags on attachment: 57673

Committed r60591: <http://trac.webkit.org/changeset/60591>
Comment 11 WebKit Commit Bot 2010-06-02 19:10:43 PDT
All reviewed patches have been landed.  Closing bug.
Comment 12 WebKit Review Bot 2010-06-02 19:38:30 PDT
http://trac.webkit.org/changeset/60591 might have broken Qt Linux Release
Comment 13 Simon Hausmann 2010-06-04 00:33:16 PDT
Revision r60591 cherry-picked into qtwebkit-2.0 with commit 7d41434425ce9c42e0fbb3640b13ee0669c9b6c7
Comment 14 Csaba Osztrogonác 2010-06-05 01:42:07 PDT
(In reply to comment #13)
> Revision r60591 cherry-picked into qtwebkit-2.0 with commit 7d41434425ce9c42e0fbb3640b13ee0669c9b6c7

editing/input/page-up-down-scrolls.html fails with QtWebKit,
and is on Skipped list, see https://bugs.webkit.org/show_bug.cgi?id=40102