54532 2011-02-15 22:53:42 -0800 On platforms without platformWidgets, going Back to a scrolled page draws at the top-left before drawing at the restored scroll position 2011-02-15 23:59:03 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) All OS X 10.6 RESOLVED FIXED InRadar P2 Normal --- 1 sullivan sullivan koivisto mitz oldest_to_newest 351894 0 sullivan 2011-02-15 22:53:42 -0800 To reproduce (on Windows or using WebKit2 on Mac, e.g.): 1. Visit http://www.apple.com in a window that’s narrow enough that a horizontal scroller is needed. 2. Scroll to the right side of the page. 3. Click one of the links at the top (e.g. “iPad”). 4. Click Back button http://www.apple.com draws at the top-left, then redraws with the correct scroll position. This does not happen in shipping Safari on Mac (due to the use of platformWidgets for the scrollers, as it turns out). 351898 1 82594 sullivan 2011-02-15 23:02:15 -0800 Created attachment 82594 Patch to not send didFirstLayout for a nested layout. 351899 2 mitz 2011-02-15 23:04:19 -0800 <rdar://problem/8998838> 351901 3 82594 koivisto 2011-02-15 23:24:34 -0800 Comment on attachment 82594 Patch to not send didFirstLayout for a nested layout. r=me 351905 4 mrowe 2011-02-15 23:40:32 -0800 Committed r78687. http://trac.webkit.org/changeset/78687 82594 2011-02-15 23:02:15 -0800 2011-02-15 23:24:34 -0800 Patch to not send didFirstLayout for a nested layout. scrollBack_patch.txt text/plain 2150 sullivan SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDc4Njg1KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTkgQEAKKzIwMTEtMDItMTUgIEpvaG4gU3Vs bGl2YW4gIDxzdWxsaXZhbkBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp P2lkPTU0NTMyCisgICAgICAgIE9uIHBsYXRmb3JtcyB3aXRob3V0IHBsYXRmb3JtV2lkZ2V0cywg Z29pbmcgQmFjayB0byBhIHNjcm9sbGVkIHBhZ2UgCisgICAgICAgIGRyYXdzIGF0IHRoZSB0b3At bGVmdCBiZWZvcmUgZHJhd2luZyBhdCB0aGUgcmVzdG9yZWQgc2Nyb2xsIHBvc2l0aW9uCisgICAg ICAgIAorICAgICAgICA8cmRhcjovL3Byb2JsZW0vODk5ODgzOD4KKworICAgICAgICAqIHBhZ2Uv RnJhbWVWaWV3LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkZyYW1lVmlldzo6cGVyZm9ybVBvc3RM YXlvdXRUYXNrcyk6CisgICAgICAgIERvbid0IHNlbmQgdGhlIGRpZEZpcnN0TGF5b3V0IGFuZCBk aWRGaXJzdFZpc3VhbGx5Tm9uRW1wdHlMYXlvdXQgY2FsbGJhY2tzCisgICAgICAgIGZvciBuZXN0 ZWQgbGF5b3V0cy4gRG9pbmcgc28gd2FzIGNhdXNpbmcgU2Nyb2xsVmlldzo6c2V0U2Nyb2xsUG9z aXRpb24oKQorICAgICAgICB0byBzaG9ydC1jaXJjdWl0LgorCiAyMDExLTAyLTE1ICBBbWkgRmlz Y2htYW4gIDxmaXNjaG1hbkBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGF2 aWQgTGV2aW4uCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wYWdlL0ZyYW1lVmlldy5jcHAKPT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PQotLS0gU291cmNlL1dlYkNvcmUvcGFnZS9GcmFtZVZpZXcuY3BwCShyZXZpc2lvbiA3ODY3 NCkKKysrIFNvdXJjZS9XZWJDb3JlL3BhZ2UvRnJhbWVWaWV3LmNwcAkod29ya2luZyBjb3B5KQpA QCAtMTg2MywxNCArMTg2MywxNiBAQCB2b2lkIEZyYW1lVmlldzo6cGVyZm9ybVBvc3RMYXlvdXRU YXNrcygpCiAgICAgbV9mcmFtZS0+c2VsZWN0aW9uKCktPnNldENhcmV0UmVjdE5lZWRzVXBkYXRl KCk7CiAgICAgbV9mcmFtZS0+c2VsZWN0aW9uKCktPnVwZGF0ZUFwcGVhcmFuY2UoKTsKIAotICAg IGlmIChtX2ZpcnN0TGF5b3V0Q2FsbGJhY2tQZW5kaW5nKSB7Ci0gICAgICAgIG1fZmlyc3RMYXlv dXRDYWxsYmFja1BlbmRpbmcgPSBmYWxzZTsKLSAgICAgICAgbV9mcmFtZS0+bG9hZGVyKCktPmRp ZEZpcnN0TGF5b3V0KCk7Ci0gICAgfQotCi0gICAgaWYgKG1faXNWaXN1YWxseU5vbkVtcHR5ICYm IG1fZmlyc3RWaXN1YWxseU5vbkVtcHR5TGF5b3V0Q2FsbGJhY2tQZW5kaW5nKSB7Ci0gICAgICAg IG1fZmlyc3RWaXN1YWxseU5vbkVtcHR5TGF5b3V0Q2FsbGJhY2tQZW5kaW5nID0gZmFsc2U7Ci0g ICAgICAgIG1fZnJhbWUtPmxvYWRlcigpLT5kaWRGaXJzdFZpc3VhbGx5Tm9uRW1wdHlMYXlvdXQo KTsKKyAgICBpZiAobV9uZXN0ZWRMYXlvdXRDb3VudCA8PSAxKSB7CisgICAgICAgIGlmIChtX2Zp cnN0TGF5b3V0Q2FsbGJhY2tQZW5kaW5nKSB7CisgICAgICAgICAgICBtX2ZpcnN0TGF5b3V0Q2Fs bGJhY2tQZW5kaW5nID0gZmFsc2U7CisgICAgICAgICAgICBtX2ZyYW1lLT5sb2FkZXIoKS0+ZGlk Rmlyc3RMYXlvdXQoKTsKKyAgICAgICAgfQorICAgICAgICAKKyAgICAgICAgaWYgKG1faXNWaXN1 YWxseU5vbkVtcHR5ICYmIG1fZmlyc3RWaXN1YWxseU5vbkVtcHR5TGF5b3V0Q2FsbGJhY2tQZW5k aW5nKSB7CisgICAgICAgICAgICBtX2ZpcnN0VmlzdWFsbHlOb25FbXB0eUxheW91dENhbGxiYWNr UGVuZGluZyA9IGZhbHNlOworICAgICAgICAgICAgbV9mcmFtZS0+bG9hZGVyKCktPmRpZEZpcnN0 VmlzdWFsbHlOb25FbXB0eUxheW91dCgpOworICAgICAgICB9CiAgICAgfQogCiAgICAgUmVuZGVy Vmlldyogcm9vdCA9IG1fZnJhbWUtPmNvbnRlbnRSZW5kZXJlcigpOwo=