91348 2012-07-15 12:12:10 -0700 REGRESSION (tiled drawing): Page’s scroll bars flash with each character you type in a textarea (affects Wikipedia and YouTube) 2012-08-04 22:56:49 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED data:text/html,%3Cbody%20style=%22width:%20200%25;%20height:%20200%25;%22%3E%3Ctextarea%3E InRadar, Regression P1 Normal --- 91363 1 mitz mitz cdumez d-r eric mifenton sw0524.lee tkent webkit.review.bot oldest_to_newest 670006 0 mitz 2012-07-15 12:12:10 -0700 <rdar://problem/11875795> When typing inside a textarea, the page’s main scroll bars flash with each character typed. To reproduce: 1. Navigate to the URL 2.Click inside the textarea and type With each character typed in the textarea, the page’s horizontal and vertical scrollbars flash. 670008 1 152464 mitz 2012-07-15 12:16:28 -0700 Created attachment 152464 Return early from ScrollableArea::scrollPositionChanged() if the scroll position doesn’t change 670014 2 mitz 2012-07-15 13:16:04 -0700 Fixed in <http://trac.webkit.org/r122681>. 670078 3 cdumez 2012-07-15 22:16:28 -0700 After this patch the following test started failing on EFL port: css3/flexbox/flexbox-overflow-auto.html http://build.webkit.org/results/EFL%20Linux%2064-bit%20Release/r122681%20(2660)/css3/flexbox/flexbox-overflow-auto-diffs.html The reason for the failure seems to be that some of the scrollbars are no longer positioned at the extremities. 670085 4 cdumez 2012-07-15 22:34:35 -0700 I filed Bug 91359 for the regression on EFL port. 670086 5 mitz 2012-07-15 22:35:34 -0700 (In reply to comment #4) > I filed Bug 91359 for the regression on EFL port. Thank you. I can reproduce this in OS X as well if I disable overlay scroll bars. 670137 6 d-r 2012-07-16 00:09:56 -0700 This causes the EFL and GTK Debug Bots to fail due to an assertion in ScrollableArea::scrollPosition() being hit, cmp. https://bugs.webkit.org/show_bug.cgi?id=91359#c4 670144 7 webkit.review.bot 2012-07-16 00:14:30 -0700 Re-opened since this is blocked by 91363 687562 8 156551 mitz 2012-08-04 22:32:15 -0700 Created attachment 156551 Call notifyContentAreaScrolled() from scrollPositionChanged() only if the scroll position changes 687564 9 mitz 2012-08-04 22:56:49 -0700 Fixed in <http://trac.webkit.org/r124714>. 152464 2012-07-15 12:16:28 -0700 2012-08-04 22:32:07 -0700 Return early from ScrollableArea::scrollPositionChanged() if the scroll position doesn’t change 91348_r1.diff text/plain 1622 mitz SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEyMjY4MCkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDEyLTA3LTE1ICBEYW4gQmVy bnN0ZWluICA8bWl0ekBhcHBsZS5jb20+CisKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzExODc1 Nzk1PiBSRUdSRVNTSU9OICh0aWxlZCBkcmF3aW5nKTogUGFnZeKAmXMgc2Nyb2xsIGJhcnMgZmxh c2ggd2l0aCBlYWNoIGNoYXJhY3RlciB5b3UgdHlwZSBpbiBhIHRleHRhcmVhIChhZmZlY3RzIFdp a2lwZWRpYSBhbmQgWW91VHViZSkKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv d19idWcuY2dpP2lkPTkxMzQ4CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku CisKKyAgICAgICAgKiBwbGF0Zm9ybS9TY3JvbGxhYmxlQXJlYS5jcHA6CisgICAgICAgIChXZWJD b3JlOjpTY3JvbGxhYmxlQXJlYTo6c2Nyb2xsUG9zaXRpb25DaGFuZ2VkKTogQWRkZWQgYW4gZWFy bHkgcmV0dXJuIGlmIHRoZSBzY3JvbGwgcG9zaXRpb24KKyAgICAgICAgZGlkIG5vdCwgaW4gZmFj dCwgY2hhbmdlLiBUaGlzIGF2b2lkcyB0aGUgY2FsbCB0byBTY3JvbGxBbmltYXRvcjo6bm90aWZ5 Q29udGVudEFyZWFTY3JvbGxlZCwKKyAgICAgICAgd2hpY2ggaXMgd2hhdCBjYXVzZXMgdGhlIHNj cm9sbCBiYXJzIHRvIGZsYXNoLgorCiAyMDEyLTA3LTE0ICBFcmljIENhcmxzb24gIDxlcmljLmNh cmxzb25AYXBwbGUuY29tPgogCiAgICAgICAgIEVuYWJsZSBBVkNGIGhhcmR3YXJlIHZpZGVvIGRl Y29kaW5nCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TY3JvbGxhYmxlQXJlYS5jcHAK PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vU2Nyb2xsYWJsZUFyZWEuY3Bw CShyZXZpc2lvbiAxMjI2ODApCisrKyBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TY3JvbGxhYmxl QXJlYS5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTE0Miw5ICsxNDIsMTMgQEAgdm9pZCBTY3JvbGxh YmxlQXJlYTo6bm90aWZ5U2Nyb2xsUG9zaXRpbwogCiB2b2lkIFNjcm9sbGFibGVBcmVhOjpzY3Jv bGxQb3NpdGlvbkNoYW5nZWQoY29uc3QgSW50UG9pbnQmIHBvc2l0aW9uKQogeworICAgIEludFBv aW50IG9sZFBvc2l0aW9uID0gc2Nyb2xsUG9zaXRpb24oKTsKICAgICAvLyBUZWxsIHRoZSBkZXJp dmVkIGNsYXNzIHRvIHNjcm9sbCBpdHMgY29udGVudHMuCiAgICAgc2V0U2Nyb2xsT2Zmc2V0KHBv c2l0aW9uKTsKIAorICAgIGlmIChzY3JvbGxQb3NpdGlvbigpID09IG9sZFBvc2l0aW9uKQorICAg ICAgICByZXR1cm47CisKICAgICBTY3JvbGxiYXIqIHZlcnRpY2FsU2Nyb2xsYmFyID0gdGhpcy0+ dmVydGljYWxTY3JvbGxiYXIoKTsKIAogICAgIC8vIFRlbGwgdGhlIHNjcm9sbGJhcnMgdG8gdXBk YXRlIHRoZWlyIHRodW1iIHBvc3Rpb25zLgo= 156551 2012-08-04 22:32:15 -0700 2012-08-04 22:53:38 -0700 Call notifyContentAreaScrolled() from scrollPositionChanged() only if the scroll position changes bug-91348-20120804223154.patch text/plain 3209 mitz SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEyNDcxMykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDEyLTA4LTA0ICBEYW4gQmVy bnN0ZWluICA8bWl0ekBhcHBsZS5jb20+CisKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzExODc1 Nzk1PiBSRUdSRVNTSU9OICh0aWxlZCBkcmF3aW5nKTogUGFnZeKAmXMgc2Nyb2xsIGJhcnMgZmxh c2ggd2l0aCBlYWNoIGNoYXJhY3RlciB5b3UgdHlwZSBpbiBhIHRleHRhcmVhIChhZmZlY3RzIFdp a2lwZWRpYSBhbmQgWW91VHViZSkKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv d19idWcuY2dpP2lkPTkxMzQ4CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku CisKKyAgICAgICAgKiBwbGF0Zm9ybS9TY3JvbGxhYmxlQXJlYS5jcHA6CisgICAgICAgIChXZWJD b3JlOjpTY3JvbGxhYmxlQXJlYTo6c2Nyb2xsUG9zaXRpb25DaGFuZ2VkKTogQ2hhbmdlZCB0byBj YWxsIG5vdGlmeUNvbnRlbnRBcmVhU2Nyb2xsZWQoKQorICAgICAgICBvbmx5IGlmIHRoZSBzY3Jv bGwgcG9zaXRpb24gYWZ0ZXIgdGhlIGNoYW5nZSBkaWZmZXJzIGZyb20gd2hhdCBpdCB3YXMgYmVm b3JlIHRoZSBjaGFuZ2UuCisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlckxpc3RCb3guY3BwOgor ICAgICAgICAoV2ViQ29yZTo6UmVuZGVyTGlzdEJveDo6c2Nyb2xsUG9zaXRpb24pOiBBZGRlZCBh biBvdmVycmlkZSBvZiB0aGlzIFNjcm9sbGFibGVBcmVhIGZ1bmN0aW9uLgorICAgICAgICAqIHJl bmRlcmluZy9SZW5kZXJMaXN0Qm94Lmg6CisKIDIwMTItMDgtMDQgIEFtaSBGaXNjaG1hbiAgPGZp c2NobWFuQGNocm9taXVtLm9yZz4KIAogICAgICAgICBIVE1MTWVkaWFFbGVtZW50IG1heSBmaXJl IHRoZSBzZWVrZWQgZXZlbnQgYmVmb3JlIGN1cnJlbnRUaW1lIHJlYWNoZXMgdGhlIHNlZWsgdGlt ZQpJbmRleDogU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vU2Nyb2xsYWJsZUFyZWEuY3BwCj09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL1Njcm9sbGFibGVBcmVhLmNwcAkocmV2 aXNpb24gMTI0NzA1KQorKysgU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vU2Nyb2xsYWJsZUFyZWEu Y3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xNDIsNiArMTQyLDcgQEAgdm9pZCBTY3JvbGxhYmxlQXJl YTo6bm90aWZ5U2Nyb2xsUG9zaXRpbwogCiB2b2lkIFNjcm9sbGFibGVBcmVhOjpzY3JvbGxQb3Np dGlvbkNoYW5nZWQoY29uc3QgSW50UG9pbnQmIHBvc2l0aW9uKQogeworICAgIEludFBvaW50IG9s ZFBvc2l0aW9uID0gc2Nyb2xsUG9zaXRpb24oKTsKICAgICAvLyBUZWxsIHRoZSBkZXJpdmVkIGNs YXNzIHRvIHNjcm9sbCBpdHMgY29udGVudHMuCiAgICAgc2V0U2Nyb2xsT2Zmc2V0KHBvc2l0aW9u KTsKIApAQCAtMTY4LDcgKzE2OSw4IEBAIHZvaWQgU2Nyb2xsYWJsZUFyZWE6OnNjcm9sbFBvc2l0 aW9uQ2hhbmcKICAgICAgICAgICAgIHZlcnRpY2FsU2Nyb2xsYmFyLT5pbnZhbGlkYXRlKCk7CiAg ICAgfQogCi0gICAgc2Nyb2xsQW5pbWF0b3IoKS0+bm90aWZ5Q29udGVudEFyZWFTY3JvbGxlZCgp OworICAgIGlmIChzY3JvbGxQb3NpdGlvbigpICE9IG9sZFBvc2l0aW9uKQorICAgICAgICBzY3Jv bGxBbmltYXRvcigpLT5ub3RpZnlDb250ZW50QXJlYVNjcm9sbGVkKCk7CiB9CiAKIGJvb2wgU2Ny b2xsYWJsZUFyZWE6OmhhbmRsZVdoZWVsRXZlbnQoY29uc3QgUGxhdGZvcm1XaGVlbEV2ZW50JiB3 aGVlbEV2ZW50KQpJbmRleDogU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckxpc3RCb3gu Y3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMaXN0Qm94 LmNwcAkocmV2aXNpb24gMTI0NzA1KQorKysgU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRl ckxpc3RCb3guY3BwCSh3b3JraW5nIGNvcHkpCkBAIC02MjcsNiArNjI3LDExIEBAIHZvaWQgUmVu ZGVyTGlzdEJveDo6c2V0U2Nyb2xsT2Zmc2V0KGNvbnMKICAgICBzY3JvbGxUbyhvZmZzZXQueSgp KTsKIH0KIAorSW50UG9pbnQgUmVuZGVyTGlzdEJveDo6c2Nyb2xsUG9zaXRpb24oKSBjb25zdAor eworICAgIHJldHVybiBJbnRQb2ludCgwLCBtX2luZGV4T2Zmc2V0KTsKK30KKwogdm9pZCBSZW5k ZXJMaXN0Qm94OjpzY3JvbGxUbyhpbnQgbmV3T2Zmc2V0KQogewogICAgIGlmIChuZXdPZmZzZXQg PT0gbV9pbmRleE9mZnNldCkKSW5kZXg6IFNvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJM aXN0Qm94LmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckxp c3RCb3guaAkocmV2aXNpb24gMTI0NzA1KQorKysgU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1Jl bmRlckxpc3RCb3guaAkod29ya2luZyBjb3B5KQpAQCAtMTAwLDYgKzEwMCw3IEBAIHByaXZhdGU6 CiAgICAgdmlydHVhbCBpbnQgc2Nyb2xsU2l6ZShTY3JvbGxiYXJPcmllbnRhdGlvbikgY29uc3Q7 CiAgICAgdmlydHVhbCBpbnQgc2Nyb2xsUG9zaXRpb24oU2Nyb2xsYmFyKikgY29uc3Q7CiAgICAg dmlydHVhbCB2b2lkIHNldFNjcm9sbE9mZnNldChjb25zdCBJbnRQb2ludCYpOworICAgIHZpcnR1 YWwgSW50UG9pbnQgc2Nyb2xsUG9zaXRpb24oKSBjb25zdCBPVkVSUklERTsKICAgICB2aXJ0dWFs IHZvaWQgaW52YWxpZGF0ZVNjcm9sbGJhclJlY3QoU2Nyb2xsYmFyKiwgY29uc3QgSW50UmVjdCYp OwogICAgIHZpcnR1YWwgYm9vbCBpc0FjdGl2ZSgpIGNvbnN0OwogICAgIHZpcnR1YWwgYm9vbCBp c1Njcm9sbENvcm5lclZpc2libGUoKSBjb25zdCB7IHJldHVybiBmYWxzZTsgfSAvLyBXZSBkb24n dCBzdXBwb3J0IHJlc2l6ZSBvbiBsaXN0IGJveGVzIHlldC4gSWYgd2UgZGlkIHRoZXNlIHdvdWxk IGhhdmUgdG8gY2hhbmdlLgo=