Implement and ship PushManager.supportedContentEncodings

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

Index: third_party/WebKit/Source/bindings/templates/attributes.cpp.tmpl
diff --git a/third_party/WebKit/Source/bindings/templates/attributes.cpp.tmpl b/third_party/WebKit/Source/bindings/templates/attributes.cpp.tmpl
index 93dd7c8e93b4aa240283c6c2d70bc6a8232ac12d..7a4cc8ef4443910efb5a8c9b7c08e3e100d9780d 100644
--- a/third_party/WebKit/Source/bindings/templates/attributes.cpp.tmpl
+++ b/third_party/WebKit/Source/bindings/templates/attributes.cpp.tmpl
@@ -20,7 +20,7 @@ const v8::FunctionCallbackInfo<v8::Value>& info
     return; // Return silently because of [LenientThis].
   {% endif %}
 
-  {% if not attribute.is_static %}
+  {% if not attribute.is_static or attribute.is_save_same_object %}
   v8::Local<v8::Object> holder = info.Holder();

[Peter Beverloo, 2017/05/05 13:18:59]

SaveSameObject uses |holder| (see lines 27-43 below), but this now introduces a
use of [SaveSameObject] on a static attribute.

Looking at FunctionCallbackInfo<>::Holder(), Ross and I suspect that the holder
ends up being the prototype object as opposed to the instance object, which
should be fine in this case. However, neither of us is sure.

This does work as expected, but verifying this would be most appreciated!

   {% endif %}