HTML Imports: Get rid of needsProcessOrStop() from dom/custom/

Source/core/dom/custom/CustomElementScheduler.cpp

Index: Source/core/dom/custom/CustomElementScheduler.cpp
diff --git a/Source/core/dom/custom/CustomElementScheduler.cpp b/Source/core/dom/custom/CustomElementScheduler.cpp
index 1980496aa6322749dc9bc9e54466e5413e502078..407e715ddb6abfa4a83f548145bd16499d8edb81 100644
--- a/Source/core/dom/custom/CustomElementScheduler.cpp
+++ b/Source/core/dom/custom/CustomElementScheduler.cpp
@@ -84,13 +84,11 @@ CustomElementMicrotaskImportStep* CustomElementScheduler::scheduleImport(HTMLImp
     ASSERT(!import->isDone());
     ASSERT(import->parent());
 
-    OwnPtr<CustomElementMicrotaskImportStep> step = CustomElementMicrotaskImportStep::create(import);
-    CustomElementMicrotaskImportStep* rawStep = step.get();
-
     // Ownership of the new step is transferred to the parent
     // processing step, or the base queue.
+    OwnPtr<CustomElementMicrotaskImportStep> step = CustomElementMicrotaskImportStep::create(import);
+    CustomElementMicrotaskImportStep* rawStep = step.get();
     CustomElementMicrotaskDispatcher::instance().enqueue(import->parent()->loader(), step.release());

[dominicc (has gone to gerrit), 2014/05/27 23:41:25]

... would just passing isSync() here be neater than adding isSync to the step?

[Hajime Morrita, 2014/05/28 00:41:37]

Done.

-
     return rawStep;
 }