Move ScriptState::GetExecutionContext (Part 4)

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

Index: third_party/WebKit/Source/core/dom/Document.cpp
diff --git a/third_party/WebKit/Source/core/dom/Document.cpp b/third_party/WebKit/Source/core/dom/Document.cpp
index 49364b1931cb671dded685d71bb0bffb12ffe968..6dc153e1f8c8404b8443949465a5bfd03936572e 100644
--- a/third_party/WebKit/Source/core/dom/Document.cpp
+++ b/third_party/WebKit/Source/core/dom/Document.cpp
@@ -261,6 +261,7 @@
 #include "wtf/text/TextEncodingRegistry.h"
 
 #include <memory>
+#include "core/dom/ExecutionContext.h"

[jbroman, 2017/04/11 00:07:57]

nit: this should be in the list of includes above, among the other core/dom
includes...except that it's not needed at all, as Document.h includes it
(because Document extends ExecutionContext)

[adithyas, 2017/04/11 15:13:47]

Ah good catch, my script didn't really account for this include style and the
fact that the include could already be in a header file... I should make it
smarter :)

 
 #ifndef NDEBUG
 using WeakDocumentSet =
@@ -4522,7 +4523,7 @@ Event* Document::createEvent(ScriptState* script_state,
                              const String& event_type,
                              ExceptionState& exception_state) {
   Event* event = nullptr;
-  ExecutionContext* execution_context = script_state->GetExecutionContext();
+  ExecutionContext* execution_context = ExecutionContext::From(script_state);
   for (const auto& factory : EventFactories()) {
     event = factory->Create(execution_context, event_type);
     if (event) {