|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>|
|Severity:||Minor||CC:||abarth, aparna.nand, commit-queue, eric, webkit.review.bot|
Description Demian Odasso 2010-12-20 12:55:43 PST
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.
Comment 1 Aparna Nandyal 2011-01-25 19:33:41 PST
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.
Comment 2 Aparna Nandyal 2011-01-25 20:49:40 PST
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 3 Andreas Kling 2011-01-28 06:53:06 PST
Comment on attachment 80161 [details] Patch for context sensitive menu option r=me
Comment 4 WebKit Commit Bot 2011-01-28 08:33:10 PST
Comment on attachment 80161 [details] Patch for context sensitive menu option Clearing flags on attachment: 80161 Committed r76946: <http://trac.webkit.org/changeset/76946>
Comment 5 WebKit Commit Bot 2011-01-28 08:33:14 PST
All reviewed patches have been landed. Closing bug.