99591 2012-10-17 06:33:11 -0700 HitTestResult::scrollbar() is NULL when hit test point is over the main scrollbars 2013-04-25 06:15:01 -0700 1 1 1 Unclassified WebKit UI Events 528+ (Nightly build) PC Linux RESOLVED DUPLICATE 114430 P2 Normal --- 114430 104369 1 cgarcia webkit-unassigned allan.jensen bdakin darin eflews.bot elima gyuyoung.kim philn xan.lopez oldest_to_newest 744205 0 cgarcia 2012-10-17 06:33:11 -0700 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. 744208 1 169171 cgarcia 2012-10-17 06:38:14 -0700 Created attachment 169171 Patch 790582 2 elima 2012-12-13 07:10:34 -0800 adding myself to CC. 807771 3 169171 darin 2013-01-15 15:49:11 -0800 Comment on attachment 169171 Patch What’s the visible symptom of this? 807795 4 cgarcia 2013-01-15 16:06:08 -0800 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. 875318 5 allan.jensen 2013-04-14 05:24:11 -0700 This is probably fixed by the patch for bug 114430 880199 6 169171 eflews.bot 2013-04-23 17:40:45 -0700 Comment on attachment 169171 Patch Attachment 169171 did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/65358 881218 7 cgarcia 2013-04-25 06:14:12 -0700 (In reply to comment #5) > This is probably fixed by the patch for bug 114430 Yes, this work now. 881219 8 cgarcia 2013-04-25 06:15:01 -0700 *** This bug has been marked as a duplicate of bug 114430 *** 169171 2012-10-17 06:38:14 -0700 2013-04-25 06:14:30 -0700 Patch wcore-hit-test-result-scrollbar.diff text/plain 1534 cgarcia ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCAxNGU0MDM3Li5hOTY2NjJiIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTkg QEAKKzIwMTItMTAtMTcgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t PgorCisgICAgICAgIEhpdFRlc3RSZXN1bHQ6OnNjcm9sbGJhcigpIGlzIE5VTEwgd2hlbiBoaXQg dGVzdCBwb2ludCBpcyBvdmVyIHRoZSBtYWluIHNjcm9sbGJhcnMKKyAgICAgICAgaHR0cHM6Ly9i dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTk5NTkxCisKKyAgICAgICAgUmV2aWV3ZWQg YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgSXQgd29ya3MgZm9yIGNvbnRyb2wgZWxlbWVu dCBzY3JvbGxiYXJzLCBidXQgbm90IGZvciB0aGUgbWFpbgorICAgICAgICBvbmVzLiBFdmVudHMg YXJlIGNvcnJlY3RseSBwcm9wYWdhdGVkIHRvIHRoZSBzY3JvbGxiYXJzIGJ1dCB0aGUKKyAgICAg ICAgSGl0VGVzdFJlc3VsdCBpcyBub3QgdXBkYXRlZCBhY2NvcmRpbmdseS4KKworICAgICAgICAq IHBhZ2UvRXZlbnRIYW5kbGVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkV2ZW50SGFuZGxlcjo6 aGFuZGxlTW91c2VNb3ZlRXZlbnQpOiBVcGRhdGUgdGhlCisgICAgICAgIEhpdFRlc3RSZXN1bHQg dG8gc2V0IHRoZSBzY3JvbGxiYXIgaWYgdGhlIHBvaW50IGlzIGluIHRoZSBtYWluCisgICAgICAg IHNjcm9sbGJhcnMuCisKIDIwMTItMTAtMTUgIFl1cnkgU2VtaWtoYXRza3kgIDx5dXJ5c0BjaHJv bWl1bS5vcmc+CiAKICAgICAgICAgV2ViIEluc3BlY3RvcjogY2FuJ3QgY2xpY2sgZGVsZXRlIGJ1 dHRvbiBpZiB3YXRjaCBleHByZXNzaW9uIGlzIHZlcnkgbG9uZwpkaWZmIC0tZ2l0IGEvU291cmNl L1dlYkNvcmUvcGFnZS9FdmVudEhhbmRsZXIuY3BwIGIvU291cmNlL1dlYkNvcmUvcGFnZS9FdmVu dEhhbmRsZXIuY3BwCmluZGV4IDVlZjBjNjEuLmRhNjk5ODIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X ZWJDb3JlL3BhZ2UvRXZlbnRIYW5kbGVyLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wYWdlL0V2 ZW50SGFuZGxlci5jcHAKQEAgLTE3OTYsNiArMTc5Niw4IEBAIGJvb2wgRXZlbnRIYW5kbGVyOjpo YW5kbGVNb3VzZU1vdmVFdmVudChjb25zdCBQbGF0Zm9ybU1vdXNlRXZlbnQmIG1vdXNlRXZlbnQs IEhpCiAKICAgICAgICAgaWYgKCFzY3JvbGxiYXIpCiAgICAgICAgICAgICBzY3JvbGxiYXIgPSBt ZXYuc2Nyb2xsYmFyKCk7CisgICAgICAgIGVsc2UgaWYgKGhvdmVyZWROb2RlICYmICFtZXYuc2Ny b2xsYmFyKCkpCisgICAgICAgICAgICBob3ZlcmVkTm9kZS0+c2V0U2Nyb2xsYmFyKHNjcm9sbGJh cik7CiAKICAgICAgICAgdXBkYXRlTGFzdFNjcm9sbGJhclVuZGVyTW91c2Uoc2Nyb2xsYmFyLCAh bV9tb3VzZVByZXNzZWQpOwogICAgICAgICBpZiAob25seVVwZGF0ZVNjcm9sbGJhcnMpCg==