Merge ExecutionContextClient and ExecutionContext

Source/core/testing/NullExecutionContext.cpp

Index: Source/core/testing/NullExecutionContext.cpp
diff --git a/Source/core/testing/NullExecutionContext.cpp b/Source/core/testing/NullExecutionContext.cpp
index 00d79dff1e1004fd37940dcf8b8fb2f756df928d..028c049dd1e437348a5d853f4bbb63ae08365607 100644
--- a/Source/core/testing/NullExecutionContext.cpp
+++ b/Source/core/testing/NullExecutionContext.cpp
@@ -5,7 +5,9 @@
 #include "config.h"
 #include "core/testing/NullExecutionContext.h"
 
+#include "core/dom/ExecutionContextTask.h"

[vsevik, 2014/09/10 21:26:34]

redundant.

[kozyatinskiy1, 2014/09/11 07:27:24]

No, it's not redundant. Here we use PassOwnPtr (not PassRefPtr) in
OwnPtrCommon.h you can find next line: COMPILE_ASSERT(sizeof(T) > 0,
TypeMustBeComplete); in OwnedPtrDeleter::deletePtr function. Type must be
complete.

 #include "core/events/Event.h"
+#include "core/frame/DOMTimer.h"
 
 namespace blink {
 
@@ -28,4 +30,13 @@ NullExecutionContext::NullExecutionContext()
 {
 }
 
+void NullExecutionContext::postTask(PassOwnPtr<ExecutionContextTask>)

[vsevik, 2014/09/10 21:26:34]

We could inline this one.

[kozyatinskiy1, 2014/09/11 07:27:24]

Because we can't define this function without ExecutionContextTask header, I
suppose that it's not good to make this function inline.

+{
+}
+
+double NullExecutionContext::timerAlignmentInterval() const
+{
+    return DOMTimer::visiblePageAlignmentInterval();
+}
+
 } // namespace blink