Bug 16254

Summary: Attributes not interpreted correctly if there is a syntax error
Product: WebKit Reporter: Anantha Keesara <anantha>
Component: DOMAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Major CC: ap, commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
URL: http://www.yogausa.com/venkatesasuprabhatam.html
Attachments:
Description Flags
screenshot
none
test case
none
Patch commit-queue: commit-queue-

Anantha Keesara
Reported 2007-12-02 10:00:35 PST
See that there is some html displayed literally. Please see the screenshot. behavior in other browsers: IE, Opera: ignores the incorrect syntax and interprets the HTML. FF, Safari: doesn't ignore incorrect syntax.
Attachments
screenshot (45.04 KB, image/jpeg)
2007-12-02 10:02 PST, Anantha Keesara 		no flags
Details
test case (16 bytes, text/html)
2007-12-03 01:58 PST, Alexey Proskuryakov 		no flags
Details
Patch (1.15 KB, patch)
2010-09-20 02:27 PDT, Adam Barth 		commit-queue: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anantha Keesara
Comment 1 2007-12-02 10:02:14 PST
Created attachment 17651 [details] screenshot
Alexey Proskuryakov
Comment 2 2007-12-03 01:58:57 PST
Created attachment 17670 [details] test case html5lib agrees with IE.
Adam Barth
Comment 3 2010-09-20 02:27:41 PDT
Created attachment 68062 [details] Patch
WebKit Commit Bot
Comment 4 2010-09-20 11:51:44 PDT
Comment on attachment 68062 [details] Patch Clearing flags on attachment: 68062 Committed r67868: <http://trac.webkit.org/changeset/67868>
WebKit Commit Bot
Comment 5 2010-09-20 11:51:50 PDT
All reviewed patches have been landed. Closing bug.
WebKit Commit Bot
Comment 6 2010-09-20 11:52:24 PDT
Comment on attachment 68062 [details] Patch Rejecting patch 68062 from commit-queue. Failed to run "[u'/Users/abarth/git/webkit-queue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Alexey Proskuryakov', u'--force']" exit_code: 1 Parsed 2 diffs from patch file(s). patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/html5lib/resources/webkit01.dat Hunk #1 FAILED at 507. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/html5lib/resources/webkit01.dat.rej Full output: http://queues.webkit.org/results/4046088
Note You need to log in before you can comment on or make changes to this bug.