Summary: | Update a list of tags to match changes in the HTML5 spec | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Barth <abarth> | ||||
Component: | New Bugs | Assignee: | Adam Barth <abarth> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, eric | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Other | ||||||
OS: | OS X 10.5 | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 41123 | ||||||
Attachments: |
|
Description
Adam Barth
2010-07-21 14:53:53 PDT
Created attachment 62237 [details]
Patch
Comment on attachment 62237 [details] Patch Since we're going to need these in HTMLTags.in later anyway, why not add them now? > + || token.name() == "figcaption" > + || token.name() == "summary" Comment on attachment 62237 [details]
Patch
We definitely have a bunch of tags to add to htmlnames.in. Soon we'll do them all in one batch when we can sing the praises of the 5% perf win it gets us. :)
Comment on attachment 62237 [details]
Patch
I'll add them all at once.
Comment on attachment 62237 [details] Patch Rejecting patch 62237 from commit-queue. Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Eric Seidel', u'--force']" exit_code: 1 Last 500 characters of output: file(s). patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/html5lib/runner-expected-html5.txt Hunk #1 FAILED at 432. Hunk #2 FAILED at 452. 2 out of 2 hunks FAILED -- saving rejects to file LayoutTests/html5lib/runner-expected-html5.txt.rej patching file WebCore/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file WebCore/html/HTMLTreeBuilder.cpp Hunk #1 succeeded at 810 (offset 1 line). Hunk #2 succeeded at 820 with fuzz 2 (offset 1 line). Full output: http://queues.webkit.org/results/3559331 Comment on attachment 62237 [details] Patch Clearing flags on attachment: 62237 Committed r63869: <http://trac.webkit.org/changeset/63869> All reviewed patches have been landed. Closing bug. |