Bug 39918 - REGRESSION(58615): Scroll events are sent twice per keypress for ports that don't have a platformWidget scrollbar
Summary: REGRESSION(58615): Scroll events are sent twice per keypress for ports that d...
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Nico Weber
URL:
Keywords:
Depends on: 40102
Blocks:
  Show dependency treegraph
 
Reported: 2010-05-28 17:31 PDT by Nico Weber
Modified: 2011-04-19 05:15 PDT (History)
10 users (show)

See Also:


Attachments
Patch (6.06 KB, patch)
2010-05-28 17:33 PDT, Nico Weber
no flags Details | Formatted Diff | Diff
Patch (6.06 KB, patch)
2010-05-28 18:11 PDT, Nico Weber
no flags Details | Formatted Diff | Diff
Patch (6.08 KB, patch)
2010-06-02 11:39 PDT, Nico Weber
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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