Expose FontFaceSetLoadEvent and add a constructor for it

third_party/WebKit/Source/bindings/core/v8/V8PrivateProperty.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8PrivateProperty_h
 #define V8PrivateProperty_h
 
 #include "bindings/core/v8/ScopedPersistent.h"
 #include "bindings/core/v8/ScriptPromiseProperties.h"
 #include "bindings/core/v8/V8BindingMacros.h"
 #include "bindings/core/v8/V8PerIsolateData.h"
 #include "core/CoreExport.h"
 #include "wtf/Allocator.h"
 #include "wtf/PtrUtil.h"
 #include <memory>
 #include <v8.h>
 
 namespace blink {
 
 class ScriptState;
 class ScriptWrappable;
 
 // Apply |X| for each pair of (InterfaceName, PrivateKeyName).
-#define V8_PRIVATE_PROPERTY_FOR_EACH(X) \
-  X(CustomEvent, Detail)                \
-  X(DOMException, Error)                \
-  X(ErrorEvent, Error)                  \
-  X(IDBObserver, Callback)              \
-  X(IntersectionObserver, Callback)     \
-  X(MessageEvent, CachedData)           \
-  X(MutationObserver, Callback)         \
-  X(PerformanceObserver, Callback)      \
-  X(SameObject, NotificationActions)    \
-  X(SameObject, NotificationData)       \
-  X(SameObject, NotificationVibrate)    \
-  X(V8NodeFilterCondition, Filter)      \
+#define V8_PRIVATE_PROPERTY_FOR_EACH(X)        \
+  X(CustomEvent, Detail)                       \
+  X(DOMException, Error)                       \
+  X(ErrorEvent, Error)                         \
+  X(IDBObserver, Callback)                     \
+  X(IntersectionObserver, Callback)            \
+  X(MessageEvent, CachedData)                  \
+  X(MutationObserver, Callback)                \
+  X(PerformanceObserver, Callback)             \
+  X(SameObject, FontFaceSetLoadEventFontfaces) \

[foolip, 2016/12/16 14:46:36]

I didn't know we had to do this for SaveSameObject. Can you blame and ask
whoever wrote the surrounding code if it's expected that this will grow by a lot
and if that's a problem?

[lunalu1, 2016/12/16 15:44:56]

If we don't do that, it simply won't compile. 

[foolip, 2016/12/16 16:13:07]

Right, I'm just wondering if there's any cost to introducing more of these, in
startup time or anything else. If the list can grow endlessly without problem,
then all is well.

+  X(SameObject, NotificationActions)           \
+  X(SameObject, NotificationData)              \
+  X(SameObject, NotificationVibrate)           \
+  X(V8NodeFilterCondition, Filter)             \
   X(Window, DocumentCachedAccessor)
 
 // The getter's name for a private property.
 #define V8_PRIVATE_PROPERTY_GETTER_NAME(InterfaceName, PrivateKeyName) \
   get##InterfaceName##PrivateKeyName
 
 // The member variable's name for a private property.
 #define V8_PRIVATE_PROPERTY_MEMBER_NAME(InterfaceName, PrivateKeyName) \
   m_symbol##InterfaceName##PrivateKeyName
 
@@ skipping 122 matching lines @@
 #define V8_PRIVATE_PROPERTY_DECLARE_MEMBER(InterfaceName, KeyName) \
   ScopedPersistent<v8::Private> V8_PRIVATE_PROPERTY_MEMBER_NAME(   \
       InterfaceName, KeyName);  // NOLINT(readability/naming/underscores)
   V8_PRIVATE_PROPERTY_FOR_EACH(V8_PRIVATE_PROPERTY_DECLARE_MEMBER)
 #undef V8_PRIVATE_PROPERTY_DECLARE_MEMBER
 };
 
 }  // namespace blink
 
 #endif  // V8PrivateProperty_h