Bug 43762 - Fix three ASSERTs hit with the HTML5 TreeBuilder in fragment mode
Summary: Fix three ASSERTs hit with the HTML5 TreeBuilder in fragment mode
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Eric Seidel (no email)
URL:
Keywords:
Depends on: 43773
Blocks:
  Show dependency tree / graph
 
Reported: 2010-08-09 17:58 PDT by Eric Seidel (no email)
Modified: 2010-08-11 00:51 PDT (History)
4 users (show)

See Also:

Attachments
Patch (7.48 KB, patch)
2010-08-09 18:02 PDT, Eric Seidel (no email) 		no flags Details | Formatted Diff | Diff
The missing pieces (2.83 KB, patch)
2010-08-10 10:28 PDT, Eric Seidel (no email) 		no flags Details | Formatted Diff | Diff
merged patch (9.60 KB, patch)
2010-08-11 00:31 PDT, Adam Barth 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2010-08-09 17:58:14 PDT 
Fix three ASSERTs hit with the HTML5 TreeBuilder in fragment mode
Comment 1 Eric Seidel (no email) 2010-08-09 18:02:52 PDT 
Created attachment 63963 [details]
Patch
Comment 2 Adam Barth 2010-08-09 20:34:11 PDT 
Comment on attachment 63963 [details]
Patch

LayoutTests/ChangeLog:8
 +          In order to fix the insertAdjacentHTML I had to some behavior.
change?
Comment 3 Eric Seidel (no email) 2010-08-10 00:29:45 PDT 
Committed r65056: <http://trac.webkit.org/changeset/65056>
Comment 4 WebKit Review Bot 2010-08-10 01:04:16 PDT 
http://trac.webkit.org/changeset/65056 might have broken Qt Linux Release
Comment 5 Csaba Osztrogonác 2010-08-10 03:53:31 PDT 
Reopened, because it was rolled out by http://trac.webkit.org/changeset/65064
Comment 6 Csaba Osztrogonác 2010-08-10 03:54:18 PDT 
Comment on attachment 63963 [details]
Patch

remove r+ from this obsolete patch
Comment 7 Eric Seidel (no email) 2010-08-10 10:28:44 PDT 
Created attachment 64025 [details]
The missing pieces
Comment 8 Adam Barth 2010-08-11 00:31:20 PDT 
Created attachment 64083 [details]
merged patch
Comment 9 Adam Barth 2010-08-11 00:32:48 PDT 
This patch seems to cause two inspector tests to crash...  Not really sure what's going on.
Comment 10 Adam Barth 2010-08-11 00:47:44 PDT 
Hum...  Running the tests a few more times, it just looks like flakiness.
Comment 11 Adam Barth 2010-08-11 00:51:25 PDT 
Committed r65134: <http://trac.webkit.org/changeset/65134>