Bug 99591

Summary: HitTestResult::scrollbar() is NULL when hit test point is over the main scrollbars
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: UI EventsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: allan.jensen, bdakin, darin, eflews.bot, elima, gyuyoung.kim, philn, xan.lopez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on: 114430    
Bug Blocks: 104369    
Attachments:
Description Flags
Patch none

Carlos Garcia Campos
Reported 2012-10-17 06:33:11 PDT
It works for control element scrollbars, but not for the main ones. Events are correctly propagated to the scrollbars but the HitTestResult is not updated accordingly.
Attachments
Patch (1.50 KB, patch)
2012-10-17 06:38 PDT, Carlos Garcia Campos 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2012-10-17 06:38:14 PDT
Created attachment 169171 [details] Patch
Eduardo Lima Mitev
Comment 2 2012-12-13 07:10:34 PST
adding myself to CC.
Darin Adler
Comment 3 2013-01-15 15:49:11 PST
Comment on attachment 169171 [details] Patch What’s the visible symptom of this?
Carlos Garcia Campos
Comment 4 2013-01-15 16:06:08 PST
The problem is that if you make a hit test for any point over a main scrollbar HitTestResutl::scrollbar() is NULL, so it's not possible to know whether you are clicking in a scrollbar or not, for example. We need that in Epiphany to ignore any action associated to mouse buttons when mouse is over a scrollbar and propagate the event to webkit.
Allan Sandfeld Jensen
Comment 5 2013-04-14 05:24:11 PDT
This is probably fixed by the patch for bug 114430
EFL EWS Bot
Comment 6 2013-04-23 17:40:45 PDT
Comment on attachment 169171 [details] Patch Attachment 169171 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/65358
Carlos Garcia Campos
Comment 7 2013-04-25 06:14:12 PDT
(In reply to comment #5) > This is probably fixed by the patch for bug 114430 Yes, this work now.
Carlos Garcia Campos
Comment 8 2013-04-25 06:15:01 PDT
*** This bug has been marked as a duplicate of bug 114430 ***
Note You need to log in before you can comment on or make changes to this bug.