Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(95)

Unified Diff: third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp

Issue 2723793002: De-Element ScriptLoader (Closed)
Patch Set: De-Element ScriptLoader Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp b/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
index c10bbc9f3bed550259edfc807254f8dcab7fea2c..d8a9c416060819558fd797b2ae48c857b0781cd5 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
+++ b/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
@@ -288,7 +288,7 @@ void HTMLConstructionSite::attachLater(ContainerNode* parent,
bool selfClosing) {
ASSERT(scriptingContentIsAllowed(m_parserContentPolicy) ||
!child->isElementNode() ||
- !toScriptLoaderIfPossible(toElement(child)));
+ !ScriptLoaderClient::fromElementIfPossible(toElement(child)));
hiroshige 2017/03/01 00:58:14 Same as HTMLOptionElement.cpp.
Nate Chapin 2017/03/01 21:34:59 Done.
ASSERT(pluginContentIsAllowed(m_parserContentPolicy) ||
!isHTMLPlugInElement(child));
@@ -758,8 +758,9 @@ void HTMLConstructionSite::insertForeignElement(
Element* element = createElement(token, namespaceURI);
if (scriptingContentIsAllowed(m_parserContentPolicy) ||
- !toScriptLoaderIfPossible(element))
+ !ScriptLoaderClient::fromElementIfPossible(element)) {
hiroshige 2017/03/01 00:58:15 ditto.
Nate Chapin 2017/03/01 21:34:59 Done.
attachLater(currentNode(), element, token->selfClosing());
+ }
if (!token->selfClosing())
m_openElements.push(HTMLStackItem::create(element, token, namespaceURI));
}

Powered by Google App Engine
This is Rietveld 408576698