Catch up ruby and its tag omission rule changes in HTML5 CR Feb 2014

Catch up ruby and its tag omission rule changes in HTML5 CR Feb 2014

This CL is based on http://trac.webkit.org/changeset/167437 by Koji
Ishii <kojishi@gmail.com>;

Fixed an rp element behavior to match with the HTML5 spec which was
changed at the following commit:
https://github.com/w3c/html/commit/c61397b989b28235ee2228f280aa8d475f3b9ebf

html5lib-test was also updated using my pull request:
https://github.com/darobin/html5lib-tests/pull/2

Changed expectation of tests19 because it showed number of total
test cases. Changed expectation of bidi-override-in-anonymous-block
because tag omission rule changed DOM tree structure.

The following is description from the original ChangeLog:

Catch up ruby and its tag omission rule changes in HTML5 CR Feb 2014
https://bugs.webkit.org/show_bug.cgi?id=131175

Patch by Koji Ishii <kojishi@gmail.com>; on 2014-04-17
Reviewed by Darin Adler.

Source/WebCore:
Tests: html5lib/resources/ruby.dat
       html5lib/resources/tests19.dat

This patch makes WebKit HTML parser to match to the tag omission rules of HTML5 CR Feb 2014:
http://www.w3.org/TR/2014/CR-html5-20140204/syntax.html#syntax-tag-omission
with the support for two new elements (rb, rtc) which were also defined in the spec:
http://www.w3.org/TR/2014/CR-html5-20140204/text-level-semantics.html#the-rb-element
http://www.w3.org/TR/2014/CR-html5-20140204/text-level-semantics.html#the-rtc-element
to better support use cases presented by W3C I18N WG:
http://www.w3.org/TR/ruby-use-cases/

* html/HTMLTagNames.in: rb and rtc added as HTMLElement interfaces.
* html/parser/HTMLConstructionSite.cpp:
(WebCore::hasImpliedEndTag): the spec says rb and rtc have implied end tags.
* html/parser/HTMLTreeBuilder.cpp:
(WebCore::HTMLTreeBuilder::processStartTagForInBody): rb and rtc added. rt excludes rtc from its implied end tags.

LayoutTests:
25 new test cases in 2 test files imported from pull requests to html5lib-tests,
with run- and expected-files generated.
All new tests pass with the changes included in this patch.

* html5lib/generated/run-ruby-data-expected.txt: Added.
* html5lib/generated/run-ruby-data.html: Added (generated by generate-test-wrappers.py)
* html5lib/generated/run-ruby-write-expected.txt: Added.
* html5lib/generated/run-ruby-write.html: Added (generated by generate-test-wrappers.py)
* html5lib/resources/ruby.dat: Added from https://github.com/darobin/html5lib-tests/pull/1
* html5lib/resources/tests19.dat: Updated from https://github.com/html5lib/html5lib-tests/pull/27

BUG=404892
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180523

[yuki.sekiguchi, 2014-08-19 04:52:21]

Hi tkent,

Could you review this?

[tkent, 2014-08-19 05:04:25]

https://codereview.chromium.org/482373002/diff/1/LayoutTests/TestExpectations
File LayoutTests/TestExpectations (right):

https://codereview.chromium.org/482373002/diff/1/LayoutTests/TestExpectations...
LayoutTests/TestExpectations:391: crbug.com/TODO
fast/css/bidi-override-in-anonymous-block.html [ NeedsRebaseline ]
TODO -> 404892

https://codereview.chromium.org/482373002/diff/1/Source/core/html/HTMLTagName...
File Source/core/html/HTMLTagNames.in (right):

https://codereview.chromium.org/482373002/diff/1/Source/core/html/HTMLTagName...
Source/core/html/HTMLTagNames.in:106: rb interfaceName=HTMLElement
Let's make the interface HTMLUnknownElement for ease of feature-detection.  We
don't support <rb> rendering.

https://codereview.chromium.org/482373002/diff/1/Source/core/html/HTMLTagName...
Source/core/html/HTMLTagNames.in:109: rtc interfaceName=HTMLElement
Ditto.

[yuki.sekiguchi, 2014-08-19 06:48:24]

On 2014/08/19 05:04:25, tkent wrote:
> https://codereview.chromium.org/482373002/diff/1/LayoutTests/TestExpectations
> File LayoutTests/TestExpectations (right):
> 
>
https://codereview.chromium.org/482373002/diff/1/LayoutTests/TestExpectations...
> LayoutTests/TestExpectations:391: crbug.com/TODO
> fast/css/bidi-override-in-anonymous-block.html [ NeedsRebaseline ]
> TODO -> 404892

Sorry. Fixed.

> 
>
https://codereview.chromium.org/482373002/diff/1/Source/core/html/HTMLTagName...
> File Source/core/html/HTMLTagNames.in (right):
> 
>
https://codereview.chromium.org/482373002/diff/1/Source/core/html/HTMLTagName...
> Source/core/html/HTMLTagNames.in:106: rb interfaceName=HTMLElement
> Let's make the interface HTMLUnknownElement for ease of feature-detection.  We
> don't support <rb> rendering.
> 
>
https://codereview.chromium.org/482373002/diff/1/Source/core/html/HTMLTagName...
> Source/core/html/HTMLTagNames.in:109: rtc interfaceName=HTMLElement
> Ditto.

Changed.

[yuki.sekiguchi, 2014-08-19 06:52:37]

Hi tkent,

I fixed your comments. Could you check them?

[tkent, 2014-08-19 06:52:55]

lgtm

CL description:
> because tag ommission rule changed DOM tree structure.

ommission -> omission.  I edited the description.

[tkent, 2014-08-19 06:53:03]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2014-08-19 06:53:53]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/yuki.sekiguchi@access-company.com/4823...

[commit-bot: I haz the power, 2014-08-19 06:57:39]

Committed patchset #2 (20001) as 180523

[yuki.sekiguchi, 2014-08-19 07:03:27]

On 2014/08/19 06:57:39, I haz the power (commit-bot) wrote:
> Committed patchset #2 (20001) as 180523

Thank you!