Slim V8DOMConfiguration::MethodConfiguration by refactoring per-world bindings.

third_party/WebKit/Source/bindings/core/v8/V8DOMConfiguration.cpp

 /*
  * Copyright (C) 2012 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 306 matching lines @@
   v8::Local<v8::Context> context = isolate->GetCurrentContext();
   v8::Local<v8::Name> name = v8AtomicString(isolate, constant.name);
   v8::PropertyAttribute attributes =
       static_cast<v8::PropertyAttribute>(v8::ReadOnly | v8::DontDelete);
   v8::Local<v8::Primitive> value = valueForConstant(isolate, constant);
   interface->DefineOwnProperty(context, name, value, attributes).ToChecked();
   prototype->DefineOwnProperty(context, name, value, attributes).ToChecked();
 }
 
 template <class Configuration>
+bool worldConfigurationApplies(const Configuration& config,
+                               const DOMWrapperWorld& world) {
+  const auto currentWorldConfig = world.isMainWorld()
+                                      ? V8DOMConfiguration::MainWorld
+                                      : V8DOMConfiguration::NonMainWorlds;
+  return config.worldConfiguration & currentWorldConfig;
+}
+
+template <>
+bool worldConfigurationApplies(
+    const V8DOMConfiguration::SymbolKeyedMethodConfiguration&,
+    const DOMWrapperWorld&) {
+  return true;
+}
+
+template <class Configuration>
 void installMethodInternal(v8::Isolate* isolate,
                            v8::Local<v8::ObjectTemplate> instanceTemplate,
                            v8::Local<v8::ObjectTemplate> prototypeTemplate,
                            v8::Local<v8::FunctionTemplate> interfaceTemplate,
                            v8::Local<v8::Signature> signature,
                            const Configuration& method,
                            const DOMWrapperWorld& world) {
+  if (!worldConfigurationApplies(method, world))
+    return;
+
   v8::Local<v8::Name> name = method.methodName(isolate);
-  v8::FunctionCallback callback = method.callbackForWorld(world);
+  v8::FunctionCallback callback = method.callback;
   // Promise-returning functions need to return a reject promise when
   // an exception occurs.  This includes a case that the receiver object is not
   // of the type.  So, we disable the type check of the receiver object on V8
   // side so that V8 won't throw.  Instead, we do the check on Blink side and
   // convert an exception to a reject promise.
   if (method.holderCheckConfiguration == V8DOMConfiguration::DoNotCheckHolder)
     signature = v8::Local<v8::Signature>();
 
   DCHECK(method.propertyLocationConfiguration);
   if (method.propertyLocationConfiguration &
@@ skipping 30 matching lines @@
 }
 
 void installMethodInternal(
     v8::Isolate* isolate,
     v8::Local<v8::Object> instance,
     v8::Local<v8::Object> prototype,
     v8::Local<v8::Function> interface,
     v8::Local<v8::Signature> signature,
     const V8DOMConfiguration::MethodConfiguration& method,
     const DOMWrapperWorld& world) {
+  if (!worldConfigurationApplies(method, world))
+    return;
+
   v8::Local<v8::Name> name = method.methodName(isolate);
-  v8::FunctionCallback callback = method.callbackForWorld(world);
+  v8::FunctionCallback callback = method.callback;
   // Promise-returning functions need to return a reject promise when
   // an exception occurs.  This includes a case that the receiver object is not
   // of the type.  So, we disable the type check of the receiver object on V8
   // side so that V8 won't throw.  Instead, we do the check on Blink side and
   // convert an exception to a reject promise.
   if (method.holderCheckConfiguration == V8DOMConfiguration::DoNotCheckHolder)
     signature = v8::Local<v8::Signature>();
 
   DCHECK(method.propertyLocationConfiguration);
   if (method.propertyLocationConfiguration &
@@ skipping 272 matching lines @@
 void V8DOMConfiguration::setClassString(
     v8::Isolate* isolate,
     v8::Local<v8::ObjectTemplate> objectTemplate,
     const char* classString) {
   objectTemplate->Set(
       v8::Symbol::GetToStringTag(isolate), v8AtomicString(isolate, classString),
       static_cast<v8::PropertyAttribute>(v8::ReadOnly | v8::DontEnum));
 }
 
 }  // namespace blink