43440 2010-08-03 12:33:11 -0700 <noscript> is not allowed inside <p> 2010-12-17 08:09:48 -0800 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) All All RESOLVED FIXED http://i.imgur.com/rQSRl.png P2 Normal --- 1 mathias abarth abarth ap commit-queue mathias peter oldest_to_newest 259624 0 mathias 2010-08-03 12:33:11 -0700 Here's a "demo" page where the issue occurs: http://jsperf.com/slugs Note that this document validates as HTML5. I wasn't sure what to do with this, so I asked #whatwg: http://krijnhoetmer.nl/irc-logs/whatwg/20100803#l-952 # [21:20] <Matjas> What is up with this? Web Inspector gives the following error: "<noscript> is not allowed inside <p>" Screenshot: http://i.imgur.com/rQSRl.png # [21:21] <Matjas> But the HTML validates and everything… Is this is a Webkit bug or am I doing something wrong? # [21:21] <annevk5> sounds like a WebKit bug # [21:23] <Matjas> annevk5: http://www.w3.org/TR/REC-html40/interact/scripts.html#h-18.3.1 says "<!ELEMENT NOSCRIPT - - (%block;)+ " # [21:23] <Matjas> (That's from HTML4 though) # [21:24] <annevk5> HTML4 does not seem relevant # [21:24] <annevk5> and that does not tell anything about what <p> allows # [21:24] <annevk5> though I guess if <noscript> just allows %block; <p> does not allow <noscript> # [21:24] <annevk5> but still a bug in WebKit 259635 1 mathias 2010-08-03 12:47:32 -0700 Cross-posted on crbug.com, as recommended by Paul Irish: http://code.google.com/p/chromium/issues/detail?id=51085 259983 2 ap 2010-08-04 05:54:45 -0700 I can reproduce this with both Safari/WebKit 5 and ToT. Indeed, this seems to be something HTML4 didn't allow. 260057 3 abarth 2010-08-04 09:43:02 -0700 This is likely to get fixed today or tomorrow when we turn on the HTML5 tree builder. 260502 4 peter 2010-08-05 03:52:28 -0700 I can confirm Adam's comment, the error no longer appears. 260659 5 abarth 2010-08-05 09:38:18 -0700 Great! 260735 6 ap 2010-08-05 11:44:22 -0700 Adam, do we want to make and land a regression test? 260740 7 abarth 2010-08-05 11:46:47 -0700 I believe this case is already tested in the html5lib test suite, but we can add another test if you want to be sure. 260774 8 abarth 2010-08-05 12:31:41 -0700 Reopening to add a test. 260777 9 63619 abarth 2010-08-05 12:34:31 -0700 Created attachment 63619 Patch 261204 10 63619 commit-queue 2010-08-06 05:13:54 -0700 Comment on attachment 63619 Patch Clearing flags on attachment: 63619 Committed r64838: <http://trac.webkit.org/changeset/64838> 261205 11 commit-queue 2010-08-06 05:14:00 -0700 All reviewed patches have been landed. Closing bug. 323779 12 mathias 2010-12-17 08:09:48 -0800 *** Bug 51254 has been marked as a duplicate of this bug. *** 63619 2010-08-05 12:34:31 -0700 2010-08-06 05:13:54 -0700 Patch bug-43440-20100805123430.patch text/plain 1301 abarth ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCA3NTJjMDk0YmU5YWVkNzExN2FjODk3NzE2NDhjZWVmMjk2MDY3MTI3Li5jMjcwM2Zj OTIwM2NmNmQ4MjliMWJhZmVlZjk0ZTM1M2ZiNzRhYzhlIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0 cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0yLDYgKzIsMTcgQEAK IAogICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KIAorICAgICAgICA8bm9zY3Jp cHQ+IGlzIG5vdCBhbGxvd2VkIGluc2lkZSA8cD4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp dC5vcmcvc2hvd19idWcuY2dpP2lkPTQzNDQwCisKKyAgICAgICAgQWRkIGEgdGVzdCB0aGF0IHdl IG5vdyB3ZSBwYXJzZSB0aGlzIGV4YW1wbGUgY29ycmVjdGx5LgorCisgICAgICAgICogaHRtbDVs aWIvcmVzb3VyY2VzL3dlYmtpdDAyLmRhdDoKKworMjAxMC0wOC0wNSAgQWRhbSBCYXJ0aCAgPGFi YXJ0aEB3ZWJraXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor CiAgICAgICAgIFUrMDAwMCBpcyB0dXJuZWQgdG8gVStGRkZEIChyZXBsYWNlbWVudCBjaGFyYWN0 ZXIpCiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00MjEx MgogCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9odG1sNWxpYi9yZXNvdXJjZXMvd2Via2l0MDIu ZGF0IGIvTGF5b3V0VGVzdHMvaHRtbDVsaWIvcmVzb3VyY2VzL3dlYmtpdDAyLmRhdAppbmRleCA0 YmE4MTlmOGZjMDNlYzRkMTgxYTdmNjY0NzhiYmJhNzFiOTU3NGQzLi40NTA2NDNiMTYyMGI1YTZh ZWRhYzJhNTcwZTBjNDk1MDhhMDU5NTExIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9odG1sNWxp Yi9yZXNvdXJjZXMvd2Via2l0MDIuZGF0CisrKyBiL0xheW91dFRlc3RzL2h0bWw1bGliL3Jlc291 cmNlcy93ZWJraXQwMi5kYXQKQEAgLTcsMyArNywxNyBAQAogfCAgIDxib2R5PgogfCAgICAgPGZv bz4KIHwgICAgICAgYmFyPSJxdXgvIgorCisjZGF0YQorPHAgaWQ9InN0YXR1cyI+PG5vc2NyaXB0 PjxzdHJvbmc+QTwvc3Ryb25nPjwvbm9zY3JpcHQ+PHNwYW4+Qjwvc3Bhbj48L3A+CisjZXJyb3Jz CisjZG9jdW1lbnQKK3wgPGh0bWw+Cit8ICAgPGhlYWQ+Cit8ICAgPGJvZHk+Cit8ICAgICA8cD4K K3wgICAgICAgaWQ9InN0YXR1cyIKK3wgICAgICAgPG5vc2NyaXB0PgorfCAgICAgICAgICI8c3Ry b25nPkE8L3N0cm9uZz4iCit8ICAgICAgIDxzcGFuPgorfCAgICAgICAgICJCIgo=