Add [CachedAccessor] attribute to cache (almost) constant accessors (window.document).

third_party/WebKit/Source/bindings/templates/attributes.cpp

Index: third_party/WebKit/Source/bindings/templates/attributes.cpp
diff --git a/third_party/WebKit/Source/bindings/templates/attributes.cpp b/third_party/WebKit/Source/bindings/templates/attributes.cpp
index e45ecf8f418c59954ccd5f154cb6a74ec9f4a3b4..c22cdd4677803c7956bfd95aadfd172ef9e134d4 100644
--- a/third_party/WebKit/Source/bindings/templates/attributes.cpp
+++ b/third_party/WebKit/Source/bindings/templates/attributes.cpp
@@ -499,6 +499,10 @@ bool {{v8_class}}::PrivateScript::{{attribute.name}}AttributeSetter(LocalFrame*
        'V8DOMConfiguration::OnlyExposedToPrivateScript'
        if attribute.only_exposed_to_private_script else
        'V8DOMConfiguration::ExposedToAllScripts' %}
+{% set surrogate_name =
+       '"%s"' % attribute.surrogate_name
+       if attribute.is_store_in_field else
+       '0' %}

[jochen (gone - plz use gerrit), 2016/09/15 08:20:51]

nullptr?

[Alfonso, 2016/09/16 14:21:48]

All the others callbacks seems to be using '0', I'm lost here.
Is this part of the updated guidelines, should I follow the new style or  mimic
the existing code?.

 {% set holder_check = 'V8DOMConfiguration::DoNotCheckHolder'
        if attribute.is_lenient_this else 'V8DOMConfiguration::CheckHolder' %}
 {% set attribute_configuration_list = [
@@ -508,6 +512,7 @@ bool {{v8_class}}::PrivateScript::{{attribute.name}}AttributeSetter(LocalFrame*
        getter_callback_for_main_world,
        setter_callback_for_main_world,
        getter_builder_callback,
+       surrogate_name,
        wrapper_type_info,
        access_control,
        property_attribute,