30424 2009-10-15 20:42:40 -0700 REGRESSION (r48687): Pages on ucas.com appear blank 2009-10-27 19:09:34 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) All All RESOLVED FIXED InRadar, Regression P1 Major --- 1 ukai beidson ap beidson chmeeedalf chrishilbert darin mrowe sam simon.fraser oldest_to_newest 154983 0 ukai 2009-10-15 20:42:40 -0700 WebKit r48687 causes the regression reported on http://crbug.com/24123 Visit the following URLs. It is expected to show login pages, but show only blank page with WebKit r48687 or later. It works as expected with WebKit r48686. https://track.ucas.com/ucastrack/Login.jsp https://apply2.ucas.com/appreg2010/SecurityServlet As investigated Developer Tools in chrome, it doesn't show any body in Element page. However, view source shows correct html contents. 155030 1 mrowe 2009-10-15 23:31:41 -0700 In the future please describe the bug in the “Summary" field. If the bug is a regression you should include the REGRESSION tag at the start of the summary, and you should CC the author of the original change on the bug. 155031 2 mrowe 2009-10-15 23:32:28 -0700 <rdar://problem/7308952> 157831 3 beidson 2009-10-26 11:13:30 -0700 *** Bug 30704 has been marked as a duplicate of this bug. *** 157832 4 beidson 2009-10-26 11:14:59 -0700 In the bugzilla dupes and radar dupes we've seen of this issue, we've seen parsing of the page stop once a schedule redirection is hit. In a couple cases it's been a history.forward(), and in a couple it's been a <meta> refresh. That symptom matches quite obviously with the nature of r48687 157909 5 41892 beidson 2009-10-26 13:59:56 -0700 Created attachment 41892 Proposed fix and layout test 157948 6 41892 darin 2009-10-26 14:55:56 -0700 Comment on attachment 41892 Proposed fix and layout test > + // http://webkit.org/b/30424 - Invalid history navigations (such as history.forward() during a new load) have > + // the side effect of cancelling any scheduled redirects. We also avoid the possibility of cancelling the > + // current load by avoiding the scheduled redirection altogether. I am not a huge fan of including bug URLs in the code. Otherwise, this comment is fine. I'd just leave out the URL. > +if (window.layoutTestController) > + window.layoutTestController.dumpAsText(); The second line can just say "layoutTestController", not "window.layoutTestController". > +http://webkit.org/b/30424 and <rdar://problem/7308952> - REGRESSION (r48687) - Some pages appear blank.<br> You need an &lt; here instead of the < sign. 158008 7 beidson 2009-10-26 16:21:27 -0700 Committed r50111. http://trac.webkit.org/changeset/50111 158090 8 beidson 2009-10-26 21:41:58 -0700 *** Bug 30746 has been marked as a duplicate of this bug. *** 158397 9 simon.fraser 2009-10-27 15:48:27 -0700 #webkit is reporting crashes: http://nopaste.com/p/aeY6lni8fb 158419 10 darin 2009-10-27 17:09:15 -0700 Lets put those crashes in a new bug report. Are they reproducible? 158420 11 simon.fraser 2009-10-27 17:11:56 -0700 They are being tracked via bug 30839. 158421 12 beidson 2009-10-27 17:12:57 -0700 There was already a new bugzilla - https://bugs.webkit.org/show_bug.cgi?id=30839 But they are *NOT* fallout from this change. We have radars for them from weeks ago (<rdar://problem/7313414>) Sadly, they are not reliably reproducible. 158450 13 chmeeedalf 2009-10-27 19:09:34 -0700 Just tested with the nightly 50124, and it works correctly, thanks. 41892 2009-10-26 13:59:56 -0700 2009-10-26 14:55:56 -0700 Proposed fix and layout test patch.txt text/plain 4224 beidson SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA1MDA5MCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTggQEAKKzIwMDktMTAtMjYgIEJyYWR5IEVpZHNvbiAgPGJlaWRzb25AYXBwbGUu Y29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIDxy ZGFyOi8vcHJvYmxlbS83MzA4OTUyPiBhbmQgaHR0cDovL3dlYmtpdC5vcmcvYi8zMDQyNCAtIFJF R1JFU1NJT04gKHI0ODY4Nyk6IFBhZ2VzIG9uIHVjYXMuY29tIGFwcGVhciBibGFuay4KKworICAg ICAgICBUZXN0OiBmYXN0L2xvYWRlci9oaXN0b3J5LWZvcndhcmQtaW4taGVhZC5odG1sCisKKyAg ICAgICAgKiBsb2FkZXIvUmVkaXJlY3RTY2hlZHVsZXIuY3BwOgorICAgICAgICAoV2ViQ29yZTo6 UmVkaXJlY3RTY2hlZHVsZXI6OnNjaGVkdWxlSGlzdG9yeU5hdmlnYXRpb24pOiBSZXN0b3JlIHRo ZSBtb3ZlZCBjYW5Hb0JhY2tPckZvcndhcmQoKSBjaGVjay4KKyAgICAgICAgICBUaGlzIHJlc3Rv cmVzIHRoZSBzaWRlIGVmZmVjdCBvZiBhbiBpbnZhbGlkIGhpc3RvcnkgbmF2aWdhdGlvbiBjYW5j ZWxpbmcgcHJldmlvdXMgc2NoZWR1bGVkIHJlZGlyZWN0cworICAgICAgICAgIGFuZCBhbHNvIGZp eGVzIHRoZSBidWcgb2Ygc2NoZWR1bGluZyBhbiBpbnZhbGlkIGhpc3RvcnkgbmF2aWdhdGlvbiBj YW5jZWxpbmcgcGFyc2luZyBvZiB0aGUgY3VycmVudCBkb2N1bWVudC4KKyAgICAgICAgKFdlYkNv cmU6OlJlZGlyZWN0U2NoZWR1bGVyOjp0aW1lckZpcmVkKTogUmVtb3ZlZCB0aGUgY2FuR29CYWNr T3JGb3J3YXJkKCkgY2hlY2sgYXMgaXQgaXMgbm93IG1hZGUgYmVmb3JlCisgICAgICAgICAgdGhl IHJlZGlyZWN0IGlzIHNjaGVkdWxlZC4KKwogMjAwOS0xMC0yNiAgUnlvc3VrZSBOaXdhICA8cm5p d2FAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBFcmljIFNlaWRlbC4KSW5kZXg6 IFdlYkNvcmUvbG9hZGVyL1JlZGlyZWN0U2NoZWR1bGVyLmNwcAo9PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJD b3JlL2xvYWRlci9SZWRpcmVjdFNjaGVkdWxlci5jcHAJKHJldmlzaW9uIDUwMDY3KQorKysgV2Vi Q29yZS9sb2FkZXIvUmVkaXJlY3RTY2hlZHVsZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yNTUs NiArMjU1LDE0IEBAIHZvaWQgUmVkaXJlY3RTY2hlZHVsZXI6OnNjaGVkdWxlSGlzdG9yeU4KICAg ICBpZiAoIW1fZnJhbWUtPnBhZ2UoKSkKICAgICAgICAgcmV0dXJuOwogCisgICAgLy8gaHR0cDov L3dlYmtpdC5vcmcvYi8zMDQyNCAtIEludmFsaWQgaGlzdG9yeSBuYXZpZ2F0aW9ucyAoc3VjaCBh cyBoaXN0b3J5LmZvcndhcmQoKSBkdXJpbmcgYSBuZXcgbG9hZCkgaGF2ZQorICAgIC8vIHRoZSBz aWRlIGVmZmVjdCBvZiBjYW5jZWxsaW5nIGFueSBzY2hlZHVsZWQgcmVkaXJlY3RzLiBXZSBhbHNv IGF2b2lkIHRoZSBwb3NzaWJpbGl0eSBvZiBjYW5jZWxsaW5nIHRoZQorICAgIC8vIGN1cnJlbnQg bG9hZCBieSBhdm9pZGluZyB0aGUgc2NoZWR1bGVkIHJlZGlyZWN0aW9uIGFsdG9nZXRoZXIuCisg ICAgaWYgKCFtX2ZyYW1lLT5wYWdlKCktPmNhbkdvQmFja09yRm9yd2FyZChzdGVwcykpIHsgCisg ICAgICAgIGNhbmNlbCgpOyAKKyAgICAgICAgcmV0dXJuOyAKKyAgICB9IAorCiAgICAgc2NoZWR1 bGUobmV3IFNjaGVkdWxlZFJlZGlyZWN0aW9uKHN0ZXBzKSk7CiB9CiAKQEAgLTI4Miw4ICsyOTAs NyBAQCB2b2lkIFJlZGlyZWN0U2NoZWR1bGVyOjp0aW1lckZpcmVkKFRpbWVyCiAgICAgICAgICAg ICB9CiAgICAgICAgICAgICAvLyBnbyhpIT0wKSBmcm9tIGEgZnJhbWUgbmF2aWdhdGVzIGludG8g dGhlIGhpc3Rvcnkgb2YgdGhlIGZyYW1lIG9ubHksCiAgICAgICAgICAgICAvLyBpbiBib3RoIElF IGFuZCBOUyAoYnV0IG5vdCBpbiBNb3ppbGxhKS4gV2UgY2FuJ3QgZWFzaWx5IGRvIHRoYXQuCi0g ICAgICAgICAgICBpZiAobV9mcmFtZS0+cGFnZSgpLT5jYW5Hb0JhY2tPckZvcndhcmQocmVkaXJl Y3Rpb24tPmhpc3RvcnlTdGVwcykpCi0gICAgICAgICAgICAgICAgbV9mcmFtZS0+cGFnZSgpLT5n b0JhY2tPckZvcndhcmQocmVkaXJlY3Rpb24tPmhpc3RvcnlTdGVwcyk7CisgICAgICAgICAgICBt X2ZyYW1lLT5wYWdlKCktPmdvQmFja09yRm9yd2FyZChyZWRpcmVjdGlvbi0+aGlzdG9yeVN0ZXBz KTsKICAgICAgICAgICAgIHJldHVybjsKICAgICAgICAgY2FzZSBTY2hlZHVsZWRSZWRpcmVjdGlv bjo6Zm9ybVN1Ym1pc3Npb246CiAgICAgICAgICAgICAvLyBUaGUgc3VibWl0Rm9ybSBmdW5jdGlv biB3aWxsIGZpbmQgYSB0YXJnZXQgZnJhbWUgYmVmb3JlIHVzaW5nIHRoZSByZWRpcmVjdGlvbiB0 aW1lci4KSW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRU ZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9uIDUwMDkwKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9n CSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTIgQEAKKzIwMDktMTAtMjYgIEJyYWR5IEVpZHNv biAgPGJlaWRzb25AYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q UyEpLgorCisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS83MzA4OTUyPiBhbmQgaHR0cDovL3dlYmtp dC5vcmcvYi8zMDQyNCAtIFJFR1JFU1NJT04gKHI0ODY4Nyk6IFBhZ2VzIG9uIHVjYXMuY29tIGFw cGVhciBibGFuay4KKworICAgICAgICAqIGZhc3QvbG9hZGVyL2hpc3RvcnktZm9yd2FyZC1pbi1o ZWFkLWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC9sb2FkZXIvaGlzdG9yeS1m b3J3YXJkLWluLWhlYWQuaHRtbDogQWRkZWQuCisKIDIwMDktMTAtMjYgIFJ5b3N1a2UgTml3YSAg PHJuaXdhQHdlYmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRXJpYyBTZWlkZWwuCklu ZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2xvYWRlci9oaXN0b3J5LWZvcndhcmQtaW4taGVhZC1leHBl Y3RlZC50eHQKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC9sb2FkZXIvaGlzdG9yeS1m b3J3YXJkLWluLWhlYWQtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMv ZmFzdC9sb2FkZXIvaGlzdG9yeS1mb3J3YXJkLWluLWhlYWQtZXhwZWN0ZWQudHh0CShyZXZpc2lv biAwKQpAQCAtMCwwICsxLDIgQEAKK2h0dHA6Ly93ZWJraXQub3JnL2IvMzA0MjQgYW5kIC0gUkVH UkVTU0lPTiAocjQ4Njg3KSAtIFNvbWUgcGFnZXMgYXBwZWFyIGJsYW5rLgorSWYgeW91IGNhbiBz ZWUgdGhpcywgdGhlIHBhZ2UgbG9hZGVkIGZpbmUgYW5kIHRoZSBidWcgaXMgZml4ZWQuCkluZGV4 OiBMYXlvdXRUZXN0cy9mYXN0L2xvYWRlci9oaXN0b3J5LWZvcndhcmQtaW4taGVhZC5odG1sCj09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvbG9hZGVyL2hpc3RvcnktZm9yd2FyZC1pbi1o ZWFkLmh0bWwJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2xvYWRlci9oaXN0b3J5 LWZvcndhcmQtaW4taGVhZC5odG1sCShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDE0IEBACis8aHRt bD4KKzxoZWFkPgorPHNjcmlwdD4KK2lmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCisg ICAgd2luZG93LmxheW91dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQoKTsKKyAgICAKK2hpc3Rv cnkuZm9yd2FyZCgpOworPC9zY3JpcHQ+Cis8L2hlYWQ+Cis8Ym9keT4KK2h0dHA6Ly93ZWJraXQu b3JnL2IvMzA0MjQgYW5kIDxyZGFyOi8vcHJvYmxlbS83MzA4OTUyPiAtIFJFR1JFU1NJT04gKHI0 ODY4NykgLSBTb21lIHBhZ2VzIGFwcGVhciBibGFuay48YnI+CitJZiB5b3UgY2FuIHNlZSB0aGlz LCB0aGUgcGFnZSBsb2FkZWQgZmluZSBhbmQgdGhlIGJ1ZyBpcyBmaXhlZC4KKzwvYm9keT4KKzwv aHRtbD4K