81146 2012-03-14 12:51:34 -0700 Don't cap the scroll position if layout happens when a FrameView's overhangAmount is non-zero 2012-03-16 11:40:12 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 andersca andersca bdakin dbeam rniwa oldest_to_newest 578672 0 andersca 2012-03-14 12:51:34 -0700 Don't cap the scroll position if layout happens when a FrameView's overhangAmount is non-zero 578679 1 131903 andersca 2012-03-14 13:02:39 -0700 Created attachment 131903 Patch 578689 2 andersca 2012-03-14 13:07:58 -0700 Committed r110737: <http://trac.webkit.org/changeset/110737> 579174 3 rniwa 2012-03-15 00:16:56 -0700 It seems like this patch caused some regression on Chromium's setting's page: http://code.google.com/p/chromium/issues/detail?id=118349 We'll probably file a new WebKit bug when we get a reduction. 579506 4 andersca 2012-03-15 10:46:04 -0700 (In reply to comment #3) > It seems like this patch caused some regression on Chromium's setting's page: > http://code.google.com/p/chromium/issues/detail?id=118349 > > We'll probably file a new WebKit bug when we get a reduction. I've seen problems when resizing the safari window as well. We might have to limit the scope of this fix. 580734 5 dbeam 2012-03-16 11:40:12 -0700 Hey Anders, we're seeing the same behavior in chromium - here's how it manifested - http://code.google.com/p/chromium/issues/detail?id=118349. 131903 2012-03-14 13:02:39 -0700 2012-03-14 13:05:32 -0700 Patch bug-81146-20120314130323.patch text/plain 2040 andersca U3VidmVyc2lvbiBSZXZpc2lvbjogMTEwNzI4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNGUyNDQwNTQ0MjQ0N2E2 NDRhNTc0OTllMWU0OThhZGNlOWNhZTk0Mi4uMzBhODVkYzg0MjRkYzA1Zjk0YjU1NjliZWE0Y2Qz YWVlZTEzMmRkNyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDEyLTAzLTE0ICBBbmRl cnMgQ2FybHNzb24gIDxhbmRlcnNjYUBhcHBsZS5jb20+CisKKyAgICAgICAgRG9uJ3QgY2FwIHRo ZSBzY3JvbGwgcG9zaXRpb24gaWYgbGF5b3V0IGhhcHBlbnMgd2hlbiBhIEZyYW1lVmlldydzIG92 ZXJoYW5nQW1vdW50IGlzIG5vbi16ZXJvCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn L3Nob3dfYnVnLmNnaT9pZD04MTE0NgorICAgICAgICA8cmRhcjovL3Byb2JsZW0vMTA4NTAwNzU+ CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgSWYgbGF5 b3V0IGhhcHBlbnMgd2hlbiBhIHBhZ2UgaGFzIGEgbm9uLXplcm8gb3ZlcmhhbmcgYW1vdW50LCB3 ZSBzaG91bGRuJ3QgY2FwIHRoZSBzY3JvbGwgcG9zaXRpb24KKyAgICAgICAgdG8gYmUgaW5zaWRl IG9mIHRoZSB2YWxpZCBtYXhpbXVtL21pbmltdW0gc2Nyb2xsIG9mZnNldHMgYmVjYXVzZSB0aGF0 IHdpbGwgY2F1c2UgdGhlIHBhZ2UgdG8ganVtcCBiYWNrLCB3aGljaAorICAgICAgICBjYW4gbG9v ayByZWFsbHkgYmFkIGlmIGxheW91dCBoYXBwZW5zIGluIHJlc3BvbnNlIHRvIHRoZSBwYWdlIGJl aW5nIHNjcm9sbGVkLgorCisgICAgICAgICogcGxhdGZvcm0vU2Nyb2xsVmlldy5jcHA6CisgICAg ICAgIChXZWJDb3JlOjpTY3JvbGxWaWV3Ojp1cGRhdGVTY3JvbGxiYXJzKToKKwogMjAxMi0wMy0x NCAgSmVyIE5vYmxlICA8amVyLm5vYmxlQGFwcGxlLmNvbT4KIAogICAgICAgICBXZWJQcm9jZXNz IHNwaW5zIGJlbmVhdGggW1FUTW92aWUgbW92aWVGaWxlVHlwZXM6XQpkaWZmIC0tZ2l0IGEvU291 cmNlL1dlYkNvcmUvcGxhdGZvcm0vU2Nyb2xsVmlldy5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0 Zm9ybS9TY3JvbGxWaWV3LmNwcAppbmRleCA2YWYyYWE2OTYwNGZhZTY4NzEzYzM5ZTcwZDZhYjE4 NDYxNGY4NDk3Li5lMTJjZDM1NGY4M2E3NmQ1YTZlMTE2ODhhYTUxNjg1Y2E5NmE3MzI2IDEwMDY0 NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TY3JvbGxWaWV3LmNwcAorKysgYi9Tb3Vy Y2UvV2ViQ29yZS9wbGF0Zm9ybS9TY3JvbGxWaWV3LmNwcApAQCAtNTkwLDcgKzU5MCwxMCBAQCB2 b2lkIFNjcm9sbFZpZXc6OnVwZGF0ZVNjcm9sbGJhcnMoY29uc3QgSW50U2l6ZSYgZGVzaXJlZE9m ZnNldCkKICAgICAgICAgICAgIGludmFsaWRhdGVTY3JvbGxDb3JuZXJSZWN0KG9sZFNjcm9sbENv cm5lclJlY3QpOwogICAgIH0KIAotICAgIEludFBvaW50IGFkanVzdGVkU2Nyb2xsUG9zaXRpb24g PSBhZGp1c3RTY3JvbGxQb3NpdGlvbldpdGhpblJhbmdlKEludFBvaW50KGRlc2lyZWRPZmZzZXQp KTsKKyAgICBJbnRQb2ludCBhZGp1c3RlZFNjcm9sbFBvc2l0aW9uID0gSW50UG9pbnQoZGVzaXJl ZE9mZnNldCk7CisgICAgaWYgKG92ZXJoYW5nQW1vdW50KCkuaXNaZXJvKCkpCisgICAgICAgIGFk anVzdGVkU2Nyb2xsUG9zaXRpb24gPSBhZGp1c3RTY3JvbGxQb3NpdGlvbldpdGhpblJhbmdlKGFk anVzdGVkU2Nyb2xsUG9zaXRpb24pOworCiAgICAgaWYgKGFkanVzdGVkU2Nyb2xsUG9zaXRpb24g IT0gc2Nyb2xsUG9zaXRpb24oKSB8fCBzY3JvbGxPcmlnaW5DaGFuZ2VkKCkpIHsKICAgICAgICAg U2Nyb2xsYWJsZUFyZWE6OnNjcm9sbFRvT2Zmc2V0V2l0aG91dEFuaW1hdGlvbihhZGp1c3RlZFNj cm9sbFBvc2l0aW9uICsgSW50U2l6ZShzY3JvbGxPcmlnaW4oKS54KCksIHNjcm9sbE9yaWdpbigp LnkoKSkpOwogICAgICAgICByZXNldFNjcm9sbE9yaWdpbkNoYW5nZWQoKTsK