| Summary: | Page scroll popup menu "Scroll here" option not working when cliking above scroll slider/handler. | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Demian Odasso <argbat> | ||||||
| Component: | WebKit Qt | Assignee: | Nobody <webkit-unassigned> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Minor | CC: | abarth, aparna.nand, commit-queue, eric, webkit.review.bot | ||||||
| Priority: | P2 | ||||||||
| Version: | 420+ | ||||||||
| Hardware: | PC | ||||||||
| OS: | Windows XP | ||||||||
| Attachments: |
|
||||||||
Created attachment 80161 [details]
Patch for context sensitive menu option
Context sensitive menu was not working as desired since m_PressedPos was not getting set properly in this scenario when moveThumb is used. In all other scenarios where moveThumb is used m_PressedPos is being set.
Additional information about the bug: 1. The scroll bar context sensitive menu option "Scroll here" does not work as expected even when he thumb of he scrollbar is not at the end. It scrolls to any position. 2. Tested that this bug is reproducible with sample QWebView code, arora brower and QtTestBrowser and patch was tested on sample code, arora browser and QtTestBrowser Comment on attachment 80161 [details]
Patch for context sensitive menu option
r=me
Comment on attachment 80161 [details] Patch for context sensitive menu option Clearing flags on attachment: 80161 Committed r76946: <http://trac.webkit.org/changeset/76946> All reviewed patches have been landed. Closing bug. http://trac.webkit.org/changeset/76946 might have broken Leopard Intel Release (Tests) |
Created attachment 77030 [details] zip with a single project to reproduce this issue and a print screen of step 5. Steps to Reproduce: 1. Have a web page loaded into a qwebkit component, the web page must be long enougth so vertical scroll bar is shown. 2. Position the scroll at the bottom. 3. Now right click the scroll bar above the scroll slider/handler. 4. A popup menu will be shown. 5. Select "Scroll here" option. Actual Results: Notice that there is no action, the scroll bar doesnt get updated and the page doesnt scroll too. Expected Results: The scroll bar gets updated and the page scrolls too. Build 2010-12-20 on Windows XP SP3, Qt 4.7.1, VS Professional 2008. Attached a zip with a single project to reproduce this issue and a print screen of step 5. Regards.