[builtins] move String.prototype[@@iterator] to C++ builtin

src/objects.cc

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/objects.h"
 
 #include <cmath>
 #include <iomanip>
 #include <memory>
 #include <sstream>
@@ skipping 19475 matching lines @@
 }
 
 Handle<Object> JSModule::LoadExport(Handle<JSModule> module,
                                     Handle<String> name) {
   Isolate* isolate = module->GetIsolate();
   LookupIterator it(module, name);
   Handle<Cell> cell = Handle<Cell>::cast(JSObject::GetDataProperty(&it));
   return handle(cell->value(), isolate);
 }
 
+void JSIteratorResult::Initialize(Handle<JSIteratorResult> object,

[Benedikt Meurer, 2016/09/15 17:34:33]

This helper method seems a bit overkill; I'd propose to just inline the two
lines and remove the helper.

+                                  Handle<Object> value, Handle<Object> done) {
+  object->set_value(*value);
+  object->set_done(*done);
+}
+
 }  // namespace internal
 }  // namespace v8