54978 2011-02-22 11:57:59 -0800 REGRESSION (r72489): TinyMCE not working in nightlies 2011-03-03 23:00:03 -0800 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) PC OS X 10.5 RESOLVED FIXED http://tinymce.moxiecode.com/tryit/full.php InRadar, NeedsReduction, Regression P1 Normal --- 46719 1 alex msaboff aestes ap barraclough darin enrica estranged ggaren jparent leviw msaboff ojan pvarga rniwa spam tony oldest_to_newest 355461 0 alex 2011-02-22 11:57:59 -0800 I don't know when this started, but it seems to have broken some time ago. The attached TinyMCE example page (from their site) works in Safari release but not in WebKit. In Safari, the content inside the box is rendered. In the nightlies, the raw HTML source is displayed. My concern of course is making sure that the breakage doesn't find its way into a Safari release. 355726 1 ap 2011-02-22 17:26:07 -0800 <rdar://problem/9039914> 355740 2 aestes 2011-02-22 18:05:57 -0800 This is due to <http://trac.webkit.org/changeset/72489>. 358793 3 rniwa 2011-02-27 14:57:23 -0800 Could you make a reduction for this bug? In particular, it's not clear what javascript isn't ran properly. It'll be much easier for us to debug if we had 10 line html/javascript that demonstrates the bug. Also, this bug might be a duplicate of https://bugs.webkit.org/show_bug.cgi?id=52106. 359042 4 alex 2011-02-28 07:33:39 -0800 This is the simplest test case I know how to make, not being associated with TinyMCE in any way: http://www.sirensclef.com/webkit/tinymce/ It will render the contents of the textarea in Safari release, but show (broken) source in nightlies. If you need further reductions, you could contact the TinyMCE folks directly, so that they know about this bug and can help diagnose it further. 359972 5 msaboff 2011-03-01 09:57:35 -0800 This bug is caused by improper handling of the regular expression /<(?:(?:!--([\w\W]*?)-->)|(?:!\[CDATA\[([\w\W]*?)\]\]>)|(?:!DOCTYPE([\w\W]*?)>)|(?:\?([^\s\/<>]+) ?([\w\W]*?)[?/]>)|(?:\/([^>]+)>)|(?:([^\s\/<>]+)\s*((?:[^"'>]+(?:(?:"[^"]*")|(?:'[^']*')|[^>]*))*)>))/ This regular expression appears to be looking for the contents inside <> HTML constructs. The failing regular expression has been reduced to: /<((ABC>)|(\/([^>]+)>)|(([^>]+)>))/ with the (ABC>) added to make it fail. The bug appears to be an issue with the "Beginning Characters" optimization added that seems to have a problem in the YARR interpreter. When the beginning characters optimization is disabled, the failing page operates correctly. The reason the (ABC>) was added to the above expression was that without that alternative, the beginning characters optimization worked correctly. It seems that the bug has to do with how far down the alternatives the begin characters setup code analyzes when creating the data for the optimization to use at runtime. I will create a patch that will disable the beginning characters optimization and then create a new defect to address the beginning characters bug. 360013 6 84251 msaboff 2011-03-01 10:55:41 -0800 Created attachment 84251 Patch to temporarily disable begin characters optimization 360059 7 84251 darin 2011-03-01 11:48:54 -0800 Comment on attachment 84251 Patch to temporarily disable begin characters optimization OK. 360079 8 msaboff 2011-03-01 11:54:58 -0800 Committed r80018: <http://trac.webkit.org/changeset/80018> 360097 9 ap 2011-03-01 12:14:17 -0800 No regression test? 360101 10 darin 2011-03-01 12:15:59 -0800 (In reply to comment #9) > No regression test? I think the intent is to add the test when we turn the optimization back on, but I agree that it would be good to add the test now. 360835 11 msaboff 2011-03-02 09:15:31 -0800 *** Bug 55520 has been marked as a duplicate of this bug. *** 362297 12 rniwa 2011-03-03 23:00:03 -0800 *** Bug 52106 has been marked as a duplicate of this bug. *** 84251 2011-03-01 10:55:41 -0800 2011-03-01 11:48:53 -0800 Patch to temporarily disable begin characters optimization 54978.patch text/plain 1504 msaboff SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291 cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gODAwMTApCisrKyBTb3VyY2Uv SmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAK KzIwMTEtMDMtMDEgIE1pY2hhZWwgU2Fib2ZmICA8bXNhYm9mZkBhcHBsZS5jb20+CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGlueU1DRSBub3Qgd29y a2luZyBpbiBuaWdodGxpZXMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTU0OTc4CisKKyAgICAgICAgRGlzYWJsaW5nIHNldHVwQmVnaW5DaGFycygpIHRv IHRlbXBvcmFyaWx5IHdvcmsgYXJyb3VuZCB0aGUgdGVzdCAKKyAgICAgICAgZmFpbHVyZS4gIEZp bGVkIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD01NTQ3OQorICAgICAg ICB0byB0cmFjayBmaXhpbmcgdGhlIGlzc3VlLgorCisgICAgICAgICogeWFyci9ZYXJyUGF0dGVy bi5jcHA6CisgICAgICAgIChKU0M6OllhcnI6OllhcnJQYXR0ZXJuOjpjb21waWxlKToKKwogMjAx MS0wMi0yOCAgR2VvZmZyZXkgR2FyZW4gIDxnZ2FyZW5AYXBwbGUuY29tPgogCiAgICAgICAgIFJl dmlld2VkIGJ5IEdhdmluIEJhcnJhY2xvdWdoLgpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3Jl L3lhcnIvWWFyclBhdHRlcm4uY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9KYXZhU2NyaXB0Q29y ZS95YXJyL1lhcnJQYXR0ZXJuLmNwcAkocmV2aXNpb24gODAwMDkpCisrKyBTb3VyY2UvSmF2YVNj cmlwdENvcmUveWFyci9ZYXJyUGF0dGVybi5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTk5Miw3ICs5 OTIsMTEgQEAgY29uc3QgY2hhciogWWFyclBhdHRlcm46OmNvbXBpbGUoY29uc3QgVQogICAgIGNv bnN0cnVjdG9yLm9wdGltaXplQk9MKCk7CiAgICAgICAgIAogICAgIGNvbnN0cnVjdG9yLnNldHVw T2Zmc2V0cygpOwotICAgIGNvbnN0cnVjdG9yLnNldHVwQmVnaW5DaGFycygpOworICAgIC8vIFRP RE86IERpc2FibGluZyB0aGUgYmVnaW4gY2hhcmFjdGVycyBvcHRpbWl6YXRpb24gZHVlIHRvIHBy b2JsZW1zIGZvdW5kCisgICAgLy8gICAgaW4gaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTU0OTc4LgorICAgIC8vICAgIEEgbmV3IGRlZmVjdCB3YXMgZmlsZWQgdG8gdHJh Y2sgdGhpcyBpc3N1ZTogCisgICAgLy8gICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No b3dfYnVnLmNnaT9pZD01NTQ3OQorICAgIC8vIGNvbnN0cnVjdG9yLnNldHVwQmVnaW5DaGFycygp OwogCiAgICAgcmV0dXJuIDA7CiB9Cg==