WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
107082
Introduce a method to build the tree from a CompactHTMLToken
https://bugs.webkit.org/show_bug.cgi?id=107082
Summary
Introduce a method to build the tree from a CompactHTMLToken
Tony Gentilcore
Reported
2013-01-16 18:34:43 PST
Introduce a method to build the tree from a CompactHTMLToken
Attachments
Patch
(6.42 KB, patch)
2013-01-16 18:35 PST
,
Tony Gentilcore
no flags
Details
Formatted Diff
Diff
Patch
(6.54 KB, patch)
2013-01-16 18:41 PST
,
Tony Gentilcore
no flags
Details
Formatted Diff
Diff
Patch for landing
(6.06 KB, patch)
2013-01-16 18:59 PST
,
Adam Barth
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Tony Gentilcore
Comment 1
2013-01-16 18:35:22 PST
Created
attachment 183088
[details]
Patch
Adam Barth
Comment 2
2013-01-16 18:39:42 PST
Comment on
attachment 183088
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=183088&action=review
> Source/WebCore/html/parser/HTMLToken.h:121 > + case HTMLTokenTypes::EndTag: {
I'd skip the { } here since we don't have them for other cases.
Tony Gentilcore
Comment 3
2013-01-16 18:40:29 PST
(In reply to
comment #2
)
> (From update of
attachment 183088
[details]
) > View in context:
https://bugs.webkit.org/attachment.cgi?id=183088&action=review
> > > Source/WebCore/html/parser/HTMLToken.h:121 > > + case HTMLTokenTypes::EndTag: { > > I'd skip the { } here since we don't have them for other cases.
Fixed
Tony Gentilcore
Comment 4
2013-01-16 18:41:27 PST
Created
attachment 183090
[details]
Patch
Adam Barth
Comment 5
2013-01-16 18:59:41 PST
Created
attachment 183094
[details]
Patch for landing
WebKit Review Bot
Comment 6
2013-01-16 19:38:14 PST
Comment on
attachment 183094
[details]
Patch for landing Clearing flags on attachment: 183094 Committed
r139953
: <
http://trac.webkit.org/changeset/139953
>
WebKit Review Bot
Comment 7
2013-01-16 19:38:17 PST
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug