Introduce ES6 iterator for DOM objects.

Source/core/testing/Internals.cpp

Index: Source/core/testing/Internals.cpp
diff --git a/Source/core/testing/Internals.cpp b/Source/core/testing/Internals.cpp
index e2179351d80974b8f6042360a135805693161fe0..8429f0af557977f54bde744ccee685519abf0597 100644
--- a/Source/core/testing/Internals.cpp
+++ b/Source/core/testing/Internals.cpp
@@ -48,6 +48,7 @@
 #include "core/dom/DocumentMarkerController.h"
 #include "core/dom/Element.h"
 #include "core/dom/ExceptionCode.h"
+#include "core/dom/Iterator.h"
 #include "core/dom/NodeRenderStyle.h"
 #include "core/dom/PseudoElement.h"
 #include "core/dom/Range.h"
@@ -67,9 +68,9 @@
 #include "core/editing/TextIterator.h"
 #include "core/fetch/MemoryCache.h"
 #include "core/fetch/ResourceFetcher.h"
-#include "core/frame/LocalDOMWindow.h"
 #include "core/frame/EventHandlerRegistry.h"
 #include "core/frame/FrameView.h"
+#include "core/frame/LocalDOMWindow.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Settings.h"
 #include "core/frame/WebKitPoint.h"
@@ -141,6 +142,37 @@
 
 namespace blink {
 
+namespace {
+
+class InternalsIterator FINAL : public Iterator {
+public:
+    InternalsIterator() : m_current(0) { }
+
+    virtual ScriptValue next(ScriptState* scriptState, ExceptionState& exceptionState) OVERRIDE
+    {
+        v8::Isolate* isolate = scriptState->isolate();
+        v8::Local<v8::Value> value = v8::Integer::New(isolate, m_current * m_current);
+        bool done = (m_current >= 5);
+        ++m_current;
+
+        v8::Local<v8::Object> result = v8::Object::New(isolate);
+        result->Set(v8String(isolate, "value"), value);
+        result->Set(v8String(isolate, "done"), v8Boolean(done, isolate));
+        return ScriptValue(scriptState, result);

[arv (Not doing code reviews), 2014/09/02 14:31:24]

Once these becomes common we might want to introduce a helper...

[yhirano, 2014/09/03 06:42:29]

Yeah, the next CL has v8ResultIterator helper function. I will make
InternalsIterator use it.

[yhirano, 2014/09/03 07:21:05]

https://codereview.chromium.org/513203002/diff/220001/Source/core/testing/Int...

+    }
+
+    virtual ScriptValue next(ScriptState* scriptState, ScriptValue value, ExceptionState& exceptionState) OVERRIDE
+    {
+        exceptionState.throwTypeError("Not implemented");
+        return ScriptValue();
+    }
+
+private:
+    int m_current;
+};
+
+} // namespace
+
 // FIXME: oilpan: These will be removed soon.
 static MockPagePopupDriver* s_pagePopupDriver = 0;
 
@@ -2225,4 +2257,9 @@ void Internals::forcePluginPlaceholder(HTMLElement* element, const String& htmlS
     toHTMLPlugInElement(element)->setUsePlaceholderContent(true);
 }
 
+Iterator* Internals::iterator(ScriptState* scriptState, ExceptionState& exceptionState)
+{
+    return new InternalsIterator;
+}
+
 } // namespace blink