Bug 19784 - REGRESSION (r34787?): Text areas in Wikipedia edit pages are empty, following content is missing
Summary: REGRESSION (r34787?): Text areas in Wikipedia edit pages are empty, following...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P1 Major
Assignee: Adam Barth
URL: http://en.wikipedia.org/w/index.php?t...
Keywords: Regression
Depends on:
Blocks:
 
Reported: 2008-06-26 14:11 PDT by mitz
Modified: 2008-06-27 12:05 PDT (History)
1 user (show)

See Also:


Attachments
Step 1: Don't allow entities to start comments (35.61 KB, patch)
2008-06-26 20:49 PDT, Adam Barth
darin: review+
Details | Formatted Diff | Diff
Step 2: Handle unterminated comments correctly (24.92 KB, patch)
2008-06-26 20:49 PDT, Adam Barth
darin: review+
Details | Formatted Diff | Diff
Test case with properly encoded comment in textarea. (148 bytes, text/html)
2008-06-27 11:18 PDT, Robert Wallis
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2008-06-26 14:11:44 PDT
It seems like no content is parsed after the beginning of the text area in Wikipedia edit pages, such as the above URL. The text area is empty and none of the rest of the DOM is present.
Comment 1 Adam Barth 2008-06-26 14:28:15 PDT
I'll take a look at this when I get home tonight.
Comment 2 Adam Barth 2008-06-26 17:30:34 PDT
Reduction <http://crypto.stanford.edu/~abarth/research/webkit/textarea/wikipedia.html>:

<html>
<body>
<textarea>
&lt;!-- 
</textarea>
</body>
</html>

Comment 3 Adam Barth 2008-06-26 20:49:04 PDT
Created attachment 21966 [details]
Step 1: Don't allow entities to start comments

This fixes the wikipedia issue.
Comment 4 Adam Barth 2008-06-26 20:49:51 PDT
Created attachment 21967 [details]
Step 2: Handle unterminated comments correctly
Comment 5 Darin Adler 2008-06-27 09:08:33 PDT
Comment on attachment 21966 [details]
Step 1: Don't allow entities to start comments

It's be better if more of the new tests were text-only "dumpAsText" tests so they could be more portable cross-platform.

r=me
Comment 6 Darin Adler 2008-06-27 09:08:54 PDT
Comment on attachment 21967 [details]
Step 2: Handle unterminated comments correctly

r=me
Comment 7 Darin Adler 2008-06-27 09:10:09 PDT
Comment on attachment 21966 [details]
Step 1: Don't allow entities to start comments

Actually, it would be better to put the new condition at the *end* of the if statement to minimize the possibility of a performance regression in code that might be pretty hot.
Comment 8 Robert Wallis 2008-06-27 11:18:00 PDT
Created attachment 21975 [details]
Test case with properly encoded comment in textarea.

This test case has an encoded comment tag in it, so it shouldn't be processing that comment tag.
Comment 9 Adam Barth 2008-06-27 11:25:24 PDT
> This test case has an encoded comment tag in it, so it shouldn't be processing
> that comment tag.

Thanks for the test case Robert.  I believe the change I just committed in r34831 fixes this issue (including the test case you just attached).  I'm waiting for the buildbots to cycle before marking this issue as fixed.