Simplify: remove PendingScript::releaseElementAndClear

third_party/WebKit/Source/core/dom/ScriptLoader.cpp

Index: third_party/WebKit/Source/core/dom/ScriptLoader.cpp
diff --git a/third_party/WebKit/Source/core/dom/ScriptLoader.cpp b/third_party/WebKit/Source/core/dom/ScriptLoader.cpp
index 6643e5e70a43bfba96a9c1534a44fe51a0e3c975..ff12ae9cb2411ed959a80ddcef381ef05bf7beb3 100644
--- a/third_party/WebKit/Source/core/dom/ScriptLoader.cpp
+++ b/third_party/WebKit/Source/core/dom/ScriptLoader.cpp
@@ -561,8 +561,7 @@ void ScriptLoader::execute() {
   DCHECK(m_pendingScript->resource());
   bool errorOccurred = false;
   ScriptSourceCode source = m_pendingScript->getSource(KURL(), errorOccurred);
-  Element* element = m_pendingScript->releaseElementAndClear();
-  ALLOW_UNUSED_LOCAL(element);
+  m_pendingScript->dispose();

[yhirano, 2016/11/29 01:29:00]

Can you tell me why we can call stopWatchingLoad here?

[kouhei (in TOK), 2016/11/29 01:52:57]

At this point, we are certain that we have finished the ScriptResource load.
This condition is checked inside PendingScript::getSource call just above this
line:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/dom/Pendi...

This means that ScriptLoader::notifyFinished has already called
stopWatchingForLoad, making this a no-op.

   if (errorOccurred) {
     dispatchErrorEvent();
   } else if (!m_resource->wasCanceled()) {