130474 2014-03-19 14:15:57 -0700 REGRESSION(r156316): Scrollbar on a large document is broken 2014-03-19 14:53:47 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED http://www.whatwg.org/specs/web-apps/current-work/multipage/ InRadar, Regression P2 Normal --- 1 rniwa rniwa andersca bdakin bfulgham kling sam simon.fraser webkit-bug-importer oldest_to_newest 992248 0 rniwa 2014-03-19 14:15:57 -0700 Reproduction steps 1. Go to http://www.whatwg.org/specs/web-apps/current-work/multipage/ 2. Scroll all the way down to the bottom via wheel/trackpad 3. Scroll up so that the scollbar is above "Help" button via wheel/trackpad. 4. Try to drag the scrollbar. Expected result: Scrollbar moves to the direction I dragged Actual result: Scrollbar always moves downwards. 992249 1 webkit-bug-importer 2014-03-19 14:16:39 -0700 <rdar://problem/16371523> 992259 2 rniwa 2014-03-19 14:29:48 -0700 The problem appears that updateArrowPlacement is no longer exiting early in ToT: http://trac.webkit.org/changeset/156316/trunk/Source/WebCore/platform/mac/ScrollbarThemeMac.mm Since isScrollbarOverlayAPIAvailable() is always true, we should be returning there. 992265 3 227219 rniwa 2014-03-19 14:32:40 -0700 Created attachment 227219 Fixes the bug 992267 4 rniwa 2014-03-19 14:37:46 -0700 Committed r165918: <http://trac.webkit.org/changeset/165918> 227219 2014-03-19 14:32:40 -0700 2014-03-19 14:35:58 -0700 Fixes the bug bug-130474-20140319143221.patch text/plain 2533 rniwa SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE2NTkxNykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE5IEBACisyMDE0LTAzLTE5ICBSeW9zdWtl IE5pd2EgIDxybml3YUB3ZWJraXQub3JnPgorCisgICAgICAgIFJFR1JFU1NJT04ocjE1NjMxNik6 IFNjcm9sbGJhciBvbiBhIGxhcmdlIGRvY3VtZW50IGlzIGJyb2tlbgorICAgICAgICBodHRwczov L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTMwNDc0CisKKyAgICAgICAgUmV2aWV3 ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhlIGJ1ZyB3YXMgY2F1c2VkIGJ5IHVw ZGF0ZUFycm93UGxhY2VtZW50KCkgZG9pbmcgdGhlIHdvcmsgZXZlbiB0aG91Z2ggU2Nyb2xsYmFy IE92ZXJsYXkgQVBJIGlzIGF2YWlsYWJsZS4KKyAgICAgICAgUHJpb3IgdG8gcjE1NjMxNiwgdGhp cyBmdW5jdGlvbiBleGl0ZWQgZWFybHk6CisgICAgICAgIGh0dHA6Ly90cmFjLndlYmtpdC5vcmcv YnJvd3Nlci90cnVuay9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9tYWMvU2Nyb2xsYmFyVGhlbWVN YWMubW0/cmV2PTE1NjMxNSNMMTQ3CisKKyAgICAgICAgRml4ZWQgdGhlIGJ1ZyBieSBkZWxldGlu ZyB1cGRhdGVBcnJvd1BsYWNlbWVudCgpIGVudGlyZWx5LgorCisgICAgICAgICogcGxhdGZvcm0v bWFjL1Njcm9sbGJhclRoZW1lTWFjLm1tOgorICAgICAgICAoV2ViQ29yZTo6U2Nyb2xsYmFyVGhl bWVNYWM6OnByZWZlcmVuY2VzQ2hhbmdlZCk6CisKIDIwMTQtMDMtMTkgIEplciBOb2JsZSAgPGpl ci5ub2JsZUBhcHBsZS5jb20+CiAKICAgICAgICAgW01hY11bRU1FXSBRdWVyeSB3aGV0aGVyIGRl Y29kZXIgc3VwcG9ydCBleGlzdHMgYmVmb3JlIGNyZWF0aW5nIGEgbmV3IE1lZGlhS2V5cyBvYmpl Y3QuCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9tYWMvU2Nyb2xsYmFyVGhlbWVNYWMu bW0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vbWFjL1Njcm9sbGJhclRo ZW1lTWFjLm1tCShyZXZpc2lvbiAxNjU2MzQpCisrKyBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9t YWMvU2Nyb2xsYmFyVGhlbWVNYWMubW0JKHdvcmtpbmcgY29weSkKQEAgLTE0MSwyMSArMTQxLDYg QEAgc3RhdGljIGJvb2wgc3VwcG9ydHNFeHBhbmRlZFNjcm9sbGJhcnMoKQogICAgIHJldHVybiBn bG9iYWxTdXBwb3J0c0V4cGFuZGVkU2Nyb2xsYmFyczsKIH0KIAotc3RhdGljIHZvaWQgdXBkYXRl QXJyb3dQbGFjZW1lbnQoKQotewotICAgIE5TU3RyaW5nICpidXR0b25QbGFjZW1lbnQgPSBbW05T VXNlckRlZmF1bHRzIHN0YW5kYXJkVXNlckRlZmF1bHRzXSBvYmplY3RGb3JLZXk6QCJBcHBsZVNj cm9sbEJhclZhcmlhbnQiXTsKLSAgICBpZiAoW2J1dHRvblBsYWNlbWVudCBpc0VxdWFsVG9TdHJp bmc6QCJTaW5nbGUiXSkKLSAgICAgICAgZ0J1dHRvblBsYWNlbWVudCA9IFNjcm9sbGJhckJ1dHRv bnNTaW5nbGU7Ci0gICAgZWxzZSBpZiAoW2J1dHRvblBsYWNlbWVudCBpc0VxdWFsVG9TdHJpbmc6 QCJEb3VibGVNaW4iXSkKLSAgICAgICAgZ0J1dHRvblBsYWNlbWVudCA9IFNjcm9sbGJhckJ1dHRv bnNEb3VibGVTdGFydDsKLSAgICBlbHNlIGlmIChbYnV0dG9uUGxhY2VtZW50IGlzRXF1YWxUb1N0 cmluZzpAIkRvdWJsZUJvdGgiXSkKLSAgICAgICAgZ0J1dHRvblBsYWNlbWVudCA9IFNjcm9sbGJh ckJ1dHRvbnNEb3VibGVCb3RoOwotICAgIGVsc2UgewotCi0gICAgICAgIGdCdXR0b25QbGFjZW1l bnQgPSBTY3JvbGxiYXJCdXR0b25zRG91YmxlRW5kOwotICAgIH0KLX0KLQogc3RhdGljIE5TQ29u dHJvbFNpemUgc2Nyb2xsYmFyQ29udHJvbFNpemVUb05TQ29udHJvbFNpemUoU2Nyb2xsYmFyQ29u dHJvbFNpemUgY29udHJvbFNpemUpCiB7CiAgICAgc3dpdGNoIChjb250cm9sU2l6ZSkgewpAQCAt MjI2LDcgKzIxMSw2IEBAIHZvaWQgU2Nyb2xsYmFyVGhlbWVNYWM6OnByZWZlcmVuY2VzQ2hhbmcK IHsKICAgICBOU1VzZXJEZWZhdWx0cyAqZGVmYXVsdHMgPSBbTlNVc2VyRGVmYXVsdHMgc3RhbmRh cmRVc2VyRGVmYXVsdHNdOwogICAgIFtkZWZhdWx0cyBzeW5jaHJvbml6ZV07Ci0gICAgdXBkYXRl QXJyb3dQbGFjZW1lbnQoKTsKICAgICBnSW5pdGlhbEJ1dHRvbkRlbGF5ID0gW2RlZmF1bHRzIGZs b2F0Rm9yS2V5OkAiTlNTY3JvbGxlckJ1dHRvbkRlbGF5Il07CiAgICAgZ0F1dG9zY3JvbGxCdXR0 b25EZWxheSA9IFtkZWZhdWx0cyBmbG9hdEZvcktleTpAIk5TU2Nyb2xsZXJCdXR0b25QZXJpb2Qi XTsKICAgICBnSnVtcE9uVHJhY2tDbGljayA9IFtkZWZhdWx0cyBib29sRm9yS2V5OkAiQXBwbGVT Y3JvbGxlclBhZ2luZ0JlaGF2aW9yIl07Cg==