76133 2012-01-11 17:37:04 -0800 ScrollAnimatorMac::mouseEnteredScrollbar() and mouseExitedScrollbar() should only do stuff for legacy scrollbars 2012-01-12 11:07:11 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 bdakin bdakin bdakin oldest_to_newest 534454 0 bdakin 2012-01-11 17:37:04 -0800 The AppKit methods that we call into in ScrollAnimatorMac::mouseEnteredScrollbar() and mouseExitedScrollbar() are really unique to legacy scrollbar concepts. We should avoid sending those notifications when the scrollbars are overlay. Patch forthcoming. <rdar://problem/10603290> 534466 1 122145 bdakin 2012-01-11 18:01:44 -0800 Created attachment 122145 Patch 534470 2 122145 bdakin 2012-01-11 18:04:54 -0800 Comment on attachment 122145 Patch Hmm, this doesn't seem to be entirely sufficient. 534964 3 bdakin 2012-01-12 10:55:08 -0800 (In reply to comment #2) > (From update of attachment 122145 [details]) > Hmm, this doesn't seem to be entirely sufficient. No, it is. I was just encountering a different bug. 534978 4 bdakin 2012-01-12 11:07:11 -0800 Thanks Sam! Committed change with http://trac.webkit.org/changeset/104837 122145 2012-01-11 18:01:44 -0800 2012-01-12 11:03:04 -0800 Patch for-review.txt text/plain 2096 bdakin SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEwNDc3NykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDEyLTAxLTExICBCZXRoIERh a2luICA8YmRha2luQGFwcGxlLmNvbT4KKworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y Zy9zaG93X2J1Zy5jZ2k/aWQ9NzYxMzMKKyAgICAgICAgU2Nyb2xsQW5pbWF0b3JNYWM6Om1vdXNl RW50ZXJlZFNjcm9sbGJhcigpIGFuZCBtb3VzZUV4aXRlZFNjcm9sbGJhcigpIHNob3VsZCAKKyAg ICAgICAgb25seSBkbyBzdHVmZiBmb3IgbGVnYWN5IHNjcm9sbGJhcnMKKyAgICAgICAgLWFuZCBj b3JyZXNwb25kaW5nLQorICAgICAgICA8cmRhcjovL3Byb2JsZW0vMTA2MDMyOTA+CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9tYWMv U2Nyb2xsQW5pbWF0b3JNYWMubW06CisgICAgICAgIChXZWJDb3JlOjpTY3JvbGxBbmltYXRvck1h Yzo6bW91c2VFbnRlcmVkU2Nyb2xsYmFyKToKKyAgICAgICAgKFdlYkNvcmU6OlNjcm9sbEFuaW1h dG9yTWFjOjptb3VzZUV4aXRlZFNjcm9sbGJhcik6CisKIDIwMTItMDEtMTEgIEFkYW0gQmFydGgg IDxhYmFydGhAd2Via2l0Lm9yZz4KIAogICAgICAgICBpZnJhbWUgc2FuZGJveCBkb2Vzbid0IGJs b2NrIGF1dG9mb2N1cyAoSUVUQyBhdXRvbWF0aWMtZmVhdHVyZS1ibG9jay1hdXRvZm9jdXMtZm9y bS1jb250cm9sKQpJbmRleDogU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vbWFjL1Njcm9sbEFuaW1h dG9yTWFjLm1tCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL21hYy9TY3Jv bGxBbmltYXRvck1hYy5tbQkocmV2aXNpb24gMTA0NzcxKQorKysgU291cmNlL1dlYkNvcmUvcGxh dGZvcm0vbWFjL1Njcm9sbEFuaW1hdG9yTWFjLm1tCSh3b3JraW5nIGNvcHkpCkBAIC03NDIsOCAr NzQyLDEzIEBAIHZvaWQgU2Nyb2xsQW5pbWF0b3JNYWM6Om1vdXNlTW92ZWRJbkNvbnQKIAogdm9p ZCBTY3JvbGxBbmltYXRvck1hYzo6bW91c2VFbnRlcmVkU2Nyb2xsYmFyKFNjcm9sbGJhciogc2Ny b2xsYmFyKSBjb25zdAogeworICAgIC8vIEF0IHRoaXMgdGltZSwgb25seSBsZWdhY3kgc2Nyb2xs YmFycyBuZWVkcyB0byBzZW5kIG5vdGlmaWNhdGlvbnMgaGVyZS4KKyAgICBpZiAocmVjb21tZW5k ZWRTY3JvbGxlclN0eWxlKCkgIT0gTlNTY3JvbGxlclN0eWxlTGVnYWN5KQorICAgICAgICByZXR1 cm47CisKICAgICBpZiAoIXNjcm9sbGFibGVBcmVhKCktPmlzT25BY3RpdmVQYWdlKCkpCiAgICAg ICAgIHJldHVybjsKKwogICAgIGlmIChpc1Njcm9sbGJhck92ZXJsYXlBUElBdmFpbGFibGUoKSkg ewogICAgICAgICBpZiAoIXN1cHBvcnRzVUlTdGF0ZVRyYW5zaXRpb25Qcm9ncmVzcygpKQogICAg ICAgICAgICAgcmV0dXJuOwpAQCAtNzU0LDggKzc1OSwxMyBAQCB2b2lkIFNjcm9sbEFuaW1hdG9y TWFjOjptb3VzZUVudGVyZWRTY3JvCiAKIHZvaWQgU2Nyb2xsQW5pbWF0b3JNYWM6Om1vdXNlRXhp dGVkU2Nyb2xsYmFyKFNjcm9sbGJhciogc2Nyb2xsYmFyKSBjb25zdAogeworICAgIC8vIEF0IHRo aXMgdGltZSwgb25seSBsZWdhY3kgc2Nyb2xsYmFycyBuZWVkcyB0byBzZW5kIG5vdGlmaWNhdGlv bnMgaGVyZS4KKyAgICBpZiAocmVjb21tZW5kZWRTY3JvbGxlclN0eWxlKCkgIT0gTlNTY3JvbGxl clN0eWxlTGVnYWN5KQorICAgICAgICByZXR1cm47CisKICAgICBpZiAoIXNjcm9sbGFibGVBcmVh KCktPmlzT25BY3RpdmVQYWdlKCkpCiAgICAgICAgIHJldHVybjsKKwogICAgIGlmIChpc1Njcm9s bGJhck92ZXJsYXlBUElBdmFpbGFibGUoKSkgewogICAgICAgICBpZiAoIXN1cHBvcnRzVUlTdGF0 ZVRyYW5zaXRpb25Qcm9ncmVzcygpKQogICAgICAgICAgICAgcmV0dXJuOwo=