Bug 140046 – Modernize and streamline HTMLToken and AtomicHTMLToken

• Status: RESOLVED
• Resolution: FIXED
• Component: Page Loading

[Darin Adler]

Modernize and streamline HTMLToken and AtomicHTMLToken

[Darin Adler]

Created attachment 243908 [details]
Patch

[Darin Adler]

Created attachment 243932 [details]
Patch

[Build Bot]

Comment on attachment 243932 [details]
Patch

Attachment 243932 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/6469179959934976

New failing tests:
html5lib/generated/run-tests6-write.html
fast/preloader/first-base-tag-scanned-wins.html
html5lib/generated/run-tests6-data.html
html5lib/generated/run-doctype01-write.html
html5lib/generated/run-doctype01-data.html
fast/preloader/understands-base-tag.html

[Build Bot]

Created attachment 243937 [details]
Archive of layout-test-results from ews100 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100  Port: mac-mountainlion  Platform: Mac OS X 10.8.5

[Build Bot]

Comment on attachment 243932 [details]
Patch

Attachment 243932 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5813054985994240

New failing tests:
html5lib/generated/run-tests6-write.html
fast/preloader/first-base-tag-scanned-wins.html
html5lib/generated/run-tests6-data.html
html5lib/generated/run-doctype01-write.html
html5lib/generated/run-doctype01-data.html
fast/preloader/understands-base-tag.html

[Build Bot]

Created attachment 243939 [details]
Archive of layout-test-results from ews106 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5

[Darin Adler]

Created attachment 243943 [details]
Patch

[Darin Adler]

This patch is now ready for review. The failures on the Mac and Mac-WK2 bots are a problem unrelated to this patch that I believe Antti has just fixed.

[Darin Adler]

Created attachment 243966 [details]
Patch

[Darin Adler]

Created attachment 243979 [details]
Patch

[Andreas Kling]

Comment on attachment 243979 [details]
Patch

r=me

[Darin Adler]

Committed r177952: <http://trac.webkit.org/changeset/177952>

[Csaba Osztrogonác]

*** Bug 140120 has been marked as a duplicate of this bug. ***

[Csaba Osztrogonác]

(In reply to comment #12)
> Committed r177952: <http://trac.webkit.org/changeset/177952>

It broke the Apple Windows build:
    1>c:\cygwin\home\buildbot\slave\win-release\build\source\webcore\html\parser\HTMLToken.h(387): error C3083: '{ctor}': the symbol to the left of a '::' must be a type (..\loader\TextResourceDecoder.cpp)
     1>c:\cygwin\home\buildbot\slave\win-release\build\source\webcore\html\parser\HTMLToken.h(388): error C2533: 'WebCore::HTMLToken::appendToAttributeValue' : constructors not allowed a return type (..\loader\TextResourceDecoder.cpp)
     1>c:\cygwin\home\buildbot\slave\win-release\build\source\webcore\html\parser\HTMLToken.h(395): error C3083: '{ctor}': the symbol to the left of a '::' must be a type (..\loader\TextResourceDecoder.cpp)
     1>c:\cygwin\home\buildbot\slave\win-release\build\source\webcore\html\parser\HTMLToken.h(396): error C2533: 'WebCore::HTMLToken::appendToAttributeValue' : constructors not allowed a return type (..\loader\TextResourceDecoder.cpp)


But unfortunately you didn't have any chance to notice it before landing,
because the Windows EWS didn't work since 2.5 weeks. Additionally you
didn't have any chance to notice it after landing, because Anders broke
the Windows build (bug140103) and nobody fixed it yet.

[Brent Fulgham]

(In reply to comment #14)

> But unfortunately you didn't have any chance to notice it before landing,
> because the Windows EWS didn't work since 2.5 weeks. Additionally you
> didn't have any chance to notice it after landing, because Anders broke
> the Windows build (bug140103) and nobody fixed it yet.

Windows EWS bots are back up and working now, but they are currently failing to process patches due to the build errors you mentioned.

I've fixed anders' build error (this morning), and will take a look at this new build failure now.

[Brent Fulgham]

Comment on attachment 243979 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=243979&action=review

> Source/WebCore/html/parser/HTMLToken.h:387
> +inline void HTMLToken::HTMLToken::appendToAttributeValue(UChar character)

MSVC chokes on this syntax. It wants HTMLToken::appendToAttributeValue.

> Source/WebCore/html/parser/HTMLToken.h:395
> +inline void HTMLToken::HTMLToken::appendToAttributeValue(unsigned i, StringView value)

Ditto above -- MSVC rejects the use of HTMLToken::HTMLToken::{name}

[Darin Adler]

Oops! Thanks for fixing that.