Add forEach() to CSSVariablesMap

Source/bindings/templates/callback_interface.cpp.tmpl

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 36 matching lines @@
     , m_world(DOMWrapperWorld::current())
 {
 }
 
 {{v8_class_name}}::~{{v8_class_name}}()
 {
 }
 
 {% for method in methods %}
 {% if not method.custom %}
-{{method.return_cpp_type}} {{v8_class_name}}::{{method.name}}({{method.argument_declaration}})
+{% set this_value_param = 'ScriptValue thisValue, ' if method.call_with_this_handle else '' %}
+{{method.return_cpp_type}} {{v8_class_name}}::{{method.name}}({{this_value_param}}{{method.argument_declaration}})

[haraken, 2013/08/06 05:02:03]

How about putting 'ScriptValue thisValue, ' into method.argument_declaration, so
that we don't need to introduce this_value_param in the template?

Probably my previous comment was confusing? I meant, you can put all the
information into operation.arguments in the code generator (which will be
converted to method.argument_declaration and method.arguments), and just use
them in the template.

[alancutter (OOO until 2018), 2013/08/07 00:40:24]

Done.

 {
     if (!canInvokeCallback())
         return true;
 
     v8::Isolate* isolate = v8::Isolate::GetCurrent();
     v8::HandleScope handleScope(isolate);
 
     v8::Handle<v8::Context> v8Context = toV8Context(scriptExecutionContext(), m_world.get());
     if (v8Context.IsEmpty())
         return true;
 
     v8::Context::Scope scope(v8Context);
 
+{% if method.call_with_this_handle %}
+    v8::Handle<v8::Value> thisHandle = thisValue.v8Value();
+    if (thisHandle.IsEmpty()) {
+        if (!isScriptControllerTerminating())
+            CRASH();
+        return true;
+    }
+    ASSERT(thisHandle->IsObject());
+{% endif %}
 {% for argument in method.arguments %}
     {{argument.cpp_to_js_conversion | indent(4)}}
     if ({{argument.name}}Handle.IsEmpty()) {
         if (!isScriptControllerTerminating())
             CRASH();
         return true;
     }
 {% endfor %}
 {% if method.arguments | length > 0 %}
     v8::Handle<v8::Value> argv[] = { {{method.handles}} };
 {% else %}
     v8::Handle<v8::Value> *argv = 0;
 {% endif %}
 
     bool callbackReturnValue = false;
-    return !invokeCallback(m_callback.newLocal(isolate), {{method.arguments | length}}, argv, callbackReturnValue, scriptExecutionContext());
+{% set this_handle_param = 'v8::Handle<v8::Object>::Cast(thisHandle), ' if method.call_with_this_handle else '' %}
+    return !invokeCallback(m_callback.newLocal(isolate), {{this_handle_param}}{{method.arguments | length}}, argv, callbackReturnValue, scriptExecutionContext());

[haraken, 2013/08/06 05:02:03]

Ditto. You might want to put 'v8::Handle<v8::Object>::Cast(thisHandle), ' into
method.arguments, so that you can avoid this_handle_param.

[alancutter (OOO until 2018), 2013/08/07 00:40:24]

This will change the value of {{method.arguments | length}} and insert the
thisHandle parameter as an argument to the Javascript callback which is not what
is intended. It only needs to be an argument to the C++ callback method.

 }
 
 {% endif %}
 {% endfor %}{# for method in methods #}
 } // namespace WebCore
 
 {% if conditional_string %}
 #endif // {{conditional_string}}
 {% endif %}