23970 2009-02-15 18:45:27 -0800 HTML parser double quote error recovery doesn't match Firefox 3.0.x 2010-09-21 10:05:54 -0700 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) Mac OS X 10.5 RESOLVED FIXED http://www.myoffers.co.uk/win-a-year-free-commuting-7075-4.asp?in=1&email=geraldcornish%40googlemail%2Ecom&enpass=wfhmjq HasReduction P2 Normal --- 1 ddkilzer webkit-unassigned ap commit-queue oldest_to_newest 109821 0 ddkilzer 2009-02-15 18:45:27 -0800 * SUMMARY The page linked to in the URL is blank when loaded in Safari (WebKit) due to the way error recovery is handled for a missing double-quote character within <script></script> tags. The same page is displayed fully in Firefox 3.0.x. * STEPS TO REPRODUCE 1. Launch Safari/WebKit. 2. Open URL or attached test case. * EXPECTED RESULTS The contents of the page should be displayed. * ACTUAL RESULTS Nothing is displayed. * REGRESSION The original page and the test case both are blank in Safari 2.0.4, so this is likely not a regression. 109822 1 27687 ddkilzer 2009-02-15 18:46:45 -0800 Created attachment 27687 Reduced test case Originally reported in Bug 12454 Comment #8. 109827 2 27688 ap 2009-02-15 23:19:59 -0800 Created attachment 27688 further reduced test case In fact, the missing quote in JavaScript is handled correctly - it's another missing quote in <img alt="> that's causing the problem. 109844 3 ddkilzer 2009-02-16 05:47:43 -0800 Thanks, Alexey! Note that WebKit matches the html5lib parser <http://james.html5.org/parsetree.html>: http://james.html5.org/cgi-bin/parsetree/parsetree.py?uri=https%3A%2F%2Fbugs.webkit.org%2Fattachment.cgi%3Fid%3D27688 282195 4 68212 abarth 2010-09-21 04:00:27 -0700 Created attachment 68212 Patch 282385 5 68212 commit-queue 2010-09-21 10:05:49 -0700 Comment on attachment 68212 Patch Clearing flags on attachment: 68212 Committed r67962: <http://trac.webkit.org/changeset/67962> 282386 6 commit-queue 2010-09-21 10:05:54 -0700 All reviewed patches have been landed. Closing bug. 27687 2009-02-15 18:46:45 -0800 2009-02-15 23:19:59 -0800 Reduced test case bug-23970-test.html text/html 779 ddkilzer PGh0bWw+CjxoZWFkPgo8L2hlYWQ+Cjxib2R5Pgo8c2NyaXB0IGxhbmd1YWdlPSJKYXZhU2NyaXB0 IiB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPiAKaWYodHlwZW9mKG9yZCk9PSJ1bmRlZmluZWQiKXt2 YXIgb3JkPU1hdGguZmxvb3IoTWF0aC5yYW5kb20oKSoxMDAwMDAwMDAwMCl9OwppZih0eXBlb2Yo dGlsZSk9PSJ1bmRlZmluZWQiKXt2YXIgdGlsZT0xfSBlbHNlIHt0aWxlKyt9OwppZih0eXBlb2Yo ZGNvcHQpPT0idW5kZWZpbmVkIil7dmFyIGRjb3B0PSJkY29wdD1pc3Q7In0gZWxzZSB7ZGNvcHQ9 In07CmRvY3VtZW50LndyaXRlKCc8c2NyaXB0IGxhbmd1YWdlPSJKYXZhU2NyaXB0IiBzcmM9Imh0 dHA6Ly9hZC51ay5kb3VibGVjbGljay5uZXQvYWRqL215ZmZyc2FkMjE7JyArIGRjb3B0ICsgJ3Rp bGU9JyArIHRpbGUgKyAnO3N6PTF4MTtvcmQ9JyArIG9yZCArICc/IiB0eXBlPSJ0ZXh0L2phdmFz Y3JpcHQiPjxcL3NjcmlwdD4nKTsKPC9zY3JpcHQ+PG5vc2NyaXB0PiAKPGEgaHJlZj0iaHR0cDov L2FkLnVrLmRvdWJsZWNsaWNrLm5ldC9qdW1wL215ZmZyc2FkMjE7c3o9MXgxO29yZD0xMjM0NTY3 ODk/IiB0YXJnZXQ9Il9ibGFuayI+IAo8aW1nIHNyYz0iaHR0cDovL2FkLnVrLmRvdWJsZWNsaWNr Lm5ldC9hZC9teWZmcnNhZDIxO3N6PTF4MTtvcmQ9MTIzNDU2Nzg5PyIgd2lkdGg9IjEiIGhlaWdo dD0iMSIgYm9yZGVyPSIwIiBhbHQ9Ij4KPC9hPjwvbm9zY3JpcHQ+CjxkaXY+UEFTUzogVGhpcyB0 ZXh0IGlzIHZpc2libGUuPC9kaXY+CjwvYm9keT4KPC9odG1sPgo= 27688 2009-02-15 23:19:59 -0800 2009-02-15 23:19:59 -0800 further reduced test case bug-23970-test.html text/html 99 ap PGh0bWw+Cjxib2R5Pgo8aW1nIHNyYz0iIiBib3JkZXI9IjAiIGFsdD0iPgo8ZGl2PlBBU1M6IFRo aXMgdGV4dCBpcyB2aXNpYmxlLjwvZGl2Pgo8L2JvZHk+CjwvaHRtbD4K 68212 2010-09-21 04:00:27 -0700 2010-09-21 10:05:49 -0700 Patch bug-23970-20100921040026.patch text/plain 2141 abarth SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDY3OTQwKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTAtMDktMjEgIEFkYW0gQmFydGggIDxhYmFy dGhAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor ICAgICAgICBIVE1MIHBhcnNlciBkb3VibGUgcXVvdGUgZXJyb3IgcmVjb3ZlcnkgZG9lc24ndCBt YXRjaCBGaXJlZm94IDMuMC54CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df YnVnLmNnaT9pZD0yMzk3MAorCisgICAgICAgIFRoaXMgYnVnIGlzIGZpeGVkIGJ5IHRoZSBIVE1M NSBwYXJzZXIuICBBZGRpbmcgdGVzdC4KKworICAgICAgICAqIGZhc3QvdG9rZW5pemVyL3VudGVy bWluYXRlZC1xdW90ZS1pbi1ub3NjcmlwdC1leHBlY3RlZC50eHQ6IEFkZGVkLgorICAgICAgICAq IGZhc3QvdG9rZW5pemVyL3VudGVybWluYXRlZC1xdW90ZS1pbi1ub3NjcmlwdC5odG1sOiBBZGRl ZC4KKyAgICAgICAgKiBodG1sNWxpYi9yZXNvdXJjZXMvd2Via2l0MDIuZGF0OgorCiAyMDEwLTA5 LTIxICBTYXRpc2ggU2FtcGF0aCAgPHNhdGlzaEBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgVW5y ZXZpZXdlZCwgbW92ZSBhbiB4cC1zcGVjaWZpYyByZXN1bHRzIHRvIHdpbi14cApJbmRleDogTGF5 b3V0VGVzdHMvZmFzdC90b2tlbml6ZXIvdW50ZXJtaW5hdGVkLXF1b3RlLWluLW5vc2NyaXB0LWV4 cGVjdGVkLnR4dAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L3Rva2VuaXplci91bnRl cm1pbmF0ZWQtcXVvdGUtaW4tbm9zY3JpcHQtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysg TGF5b3V0VGVzdHMvZmFzdC90b2tlbml6ZXIvdW50ZXJtaW5hdGVkLXF1b3RlLWluLW5vc2NyaXB0 LWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSBAQAorUEFTUzogVGhpcyB0ZXh0 IGlzIHZpc2libGUuCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L3Rva2VuaXplci91bnRlcm1pbmF0 ZWQtcXVvdGUtaW4tbm9zY3JpcHQuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0 L3Rva2VuaXplci91bnRlcm1pbmF0ZWQtcXVvdGUtaW4tbm9zY3JpcHQuaHRtbAkocmV2aXNpb24g MCkKKysrIExheW91dFRlc3RzL2Zhc3QvdG9rZW5pemVyL3VudGVybWluYXRlZC1xdW90ZS1pbi1u b3NjcmlwdC5odG1sCShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDEyIEBACis8aHRtbD4KKzxib2R5 PgorPHNjcmlwdD4KK2lmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCisgICAgbGF5b3V0 VGVzdENvbnRyb2xsZXIuZHVtcEFzVGV4dCgpOworPC9zY3JpcHQ+Cis8bm9zY3JpcHQ+IAorPGlt ZyBzcmM9ImFib3V0OmJsYW5rIiBhbHQ9Ij4KKzwvbm9zY3JpcHQ+Cis8ZGl2PlBBU1M6IFRoaXMg dGV4dCBpcyB2aXNpYmxlLjwvZGl2PgorPC9ib2R5PgorPC9odG1sPgpJbmRleDogTGF5b3V0VGVz dHMvaHRtbDVsaWIvcmVzb3VyY2VzL3dlYmtpdDAyLmRhdAo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRU ZXN0cy9odG1sNWxpYi9yZXNvdXJjZXMvd2Via2l0MDIuZGF0CShyZXZpc2lvbiA2NzkyNikKKysr IExheW91dFRlc3RzL2h0bWw1bGliL3Jlc291cmNlcy93ZWJraXQwMi5kYXQJKHdvcmtpbmcgY29w eSkKQEAgLTMyLDMgKzMyLDExIEBACiB8ICAgICA8ZGl2PgogfCAgICAgICA8c2FyY2FzbT4KIHwg ICAgICAgICA8ZGl2PgorCisjZGF0YQorPGh0bWw+PGJvZHk+PGltZyBzcmM9IiIgYm9yZGVyPSIw IiBhbHQ9Ij48ZGl2PkE8L2Rpdj48L2JvZHk+PC9odG1sPgorI2Vycm9ycworI2RvY3VtZW50Cit8 IDxodG1sPgorfCAgIDxoZWFkPgorfCAgIDxib2R5Pgo=