createElementFromSavedToken shouldn't have the code to create a non-HTML element

• Component: DOM
• Status: RESOLVED FIXED

[Ryosuke Niwa]

HTMLConstructionSite::createElementFromSavedToken is only called on a stack item which in m_activeFormattingElements.
It shouldn't have the code to create a non-HTML element.

[Ryosuke Niwa]

Created attachment 269499 [details]
Clenaup

[Chris Dumez]

Comment on attachment 269499 [details]
Clenaup

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

r=me with comments.

> Source/WebCore/html/parser/HTMLConstructionSite.cpp:623
> +    setAttributes(element.ptr(), token, m_parserContentPolicy);

Looks like setAttributes() should take a reference. It'd be nice to do it in this patch if the change is not too large.

> Source/WebCore/html/parser/HTMLConstructionSite.cpp:655
> +    return element.releaseNonNull();

HTMLElementFactory::createElement() actually returns a Ref<> so you should change the element variable type above and simply return element here.

> Source/WebCore/html/parser/HTMLConstructionSite.cpp:660
>      RefPtr<Element> element;

Should be dropped.

[Ryosuke Niwa]

Created attachment 269527 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 269527 [details]
Patch for landing

Rejecting attachment 269527 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'validate-changelog', '--check-oops', '--non-interactive', 269527, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in Source/WebCore/ChangeLog contains OOPS!.

Full output: http://webkit-queues.webkit.org/results/723503

[Ryosuke Niwa]

Created attachment 269528 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 269528 [details]
Patch for landing

Rejecting attachment 269528 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'apply-attachment', '--no-update', '--non-interactive', 269528, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
WS/WebKit

Parsed 4 diffs from patch file(s).
patching file Source/WebCore/ChangeLog
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file Source/WebCore/ChangeLog.rej
patching file Source/WebCore/html/parser/HTMLConstructionSite.cpp
patching file Source/WebCore/html/parser/HTMLConstructionSite.h
patching file Source/WebCore/html/parser/HTMLTreeBuilder.cpp

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: http://webkit-queues.webkit.org/results/723509

[Ryosuke Niwa]

Created attachment 269529 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 269529 [details]
Patch for landing

Rejecting attachment 269529 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'build', '--no-clean', '--no-update', '--build-style=release', '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
.dia -c /Volumes/Data/EWS/WebKit/Source/WebCore/html/parser/TextDocumentParser.cpp -o /Volumes/Data/EWS/WebKit/WebKitBuild/WebCore.build/Release/WebCore.build/Objects-normal/x86_64/TextDocumentParser.o

** BUILD FAILED **


The following build commands failed:
	CompileC /Volumes/Data/EWS/WebKit/WebKitBuild/WebCore.build/Release/WebCore.build/Objects-normal/x86_64/HTMLConstructionSite.o html/parser/HTMLConstructionSite.cpp normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)

Full output: http://webkit-queues.webkit.org/results/723533

[Ryosuke Niwa]

Created attachment 269539 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 269539 [details]
Patch for landing

Clearing flags on attachment: 269539

Committed r195438: <http://trac.webkit.org/changeset/195438>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.