Title: [283366] trunk/Source/_javascript_Core
Revision
283366
Author
commit-qu...@webkit.org
Date
2021-10-01 07:23:58 -0700 (Fri, 01 Oct 2021)

Log Message

[JSC][GLib] Fix typo in WrapperMap creation methods
https://bugs.webkit.org/show_bug.cgi?id=231076

Patch by Olivier Blin <olivier.b...@softathome.com> on 2021-10-01
Reviewed by Michael Catanzaro.

* API/glib/JSCContext.cpp:
(jscContextGetOrCreateJSWrapper):
(jscContextCreateContextWithJSWrapper):
* API/glib/JSCWrapperMap.cpp:
(JSC::WrapperMap::createJSWrapper):
(JSC::WrapperMap::createContextWithJSWrapper):
(JSC::WrapperMap::createJSWrappper): Deleted.
(JSC::WrapperMap::createContextWithJSWrappper): Deleted.
* API/glib/JSCWrapperMap.h:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/API/glib/JSCContext.cpp (283365 => 283366)


--- trunk/Source/_javascript_Core/API/glib/JSCContext.cpp	2021-10-01 13:41:33 UTC (rev 283365)
+++ trunk/Source/_javascript_Core/API/glib/JSCContext.cpp	2021-10-01 14:23:58 UTC (rev 283366)
@@ -235,12 +235,12 @@
     if (auto* jsWrapper = jscContextGetJSWrapper(context, wrappedObject))
         return jsWrapper;
 
-    return wrapperMap(context).createJSWrappper(context->priv->jsContext.get(), jsClass, prototype, wrappedObject, destroyFunction);
+    return wrapperMap(context).createJSWrapper(context->priv->jsContext.get(), jsClass, prototype, wrappedObject, destroyFunction);
 }
 
 JSGlobalContextRef jscContextCreateContextWithJSWrapper(JSCContext* context, JSClassRef jsClass, JSValueRef prototype, gpointer wrappedObject, GDestroyNotify destroyFunction)
 {
-    return wrapperMap(context).createContextWithJSWrappper(jscVirtualMachineGetContextGroup(context->priv->vm.get()), jsClass, prototype, wrappedObject, destroyFunction);
+    return wrapperMap(context).createContextWithJSWrapper(jscVirtualMachineGetContextGroup(context->priv->vm.get()), jsClass, prototype, wrappedObject, destroyFunction);
 }
 
 gpointer jscContextWrappedObject(JSCContext* context, JSObjectRef jsObject)

Modified: trunk/Source/_javascript_Core/API/glib/JSCWrapperMap.cpp (283365 => 283366)


--- trunk/Source/_javascript_Core/API/glib/JSCWrapperMap.cpp	2021-10-01 13:41:33 UTC (rev 283365)
+++ trunk/Source/_javascript_Core/API/glib/JSCWrapperMap.cpp	2021-10-01 14:23:58 UTC (rev 283366)
@@ -75,7 +75,7 @@
     return m_classMap.get(jsClass);
 }
 
-JSObject* WrapperMap::createJSWrappper(JSGlobalContextRef jsContext, JSClassRef jsClass, JSValueRef prototype, gpointer wrappedObject, GDestroyNotify destroyFunction)
+JSObject* WrapperMap::createJSWrapper(JSGlobalContextRef jsContext, JSClassRef jsClass, JSValueRef prototype, gpointer wrappedObject, GDestroyNotify destroyFunction)
 {
     ASSERT(toJSGlobalObject(jsContext)->wrapperMap() == this);
     JSGlobalObject* globalObject = toJS(jsContext);
@@ -93,7 +93,7 @@
     return object;
 }
 
-JSGlobalContextRef WrapperMap::createContextWithJSWrappper(JSContextGroupRef jsGroup, JSClassRef jsClass, JSValueRef prototype, gpointer wrappedObject, GDestroyNotify destroyFunction)
+JSGlobalContextRef WrapperMap::createContextWithJSWrapper(JSContextGroupRef jsGroup, JSClassRef jsClass, JSValueRef prototype, gpointer wrappedObject, GDestroyNotify destroyFunction)
 {
     Ref<VM> vm(*toJS(jsGroup));
     JSLockHolder locker(vm.ptr());

Modified: trunk/Source/_javascript_Core/API/glib/JSCWrapperMap.h (283365 => 283366)


--- trunk/Source/_javascript_Core/API/glib/JSCWrapperMap.h	2021-10-01 13:41:33 UTC (rev 283365)
+++ trunk/Source/_javascript_Core/API/glib/JSCWrapperMap.h	2021-10-01 14:23:58 UTC (rev 283366)
@@ -47,8 +47,8 @@
     void registerClass(JSCClass*);
     JSCClass* registeredClass(JSClassRef) const;
 
-    JSObject* createJSWrappper(JSGlobalContextRef, JSClassRef, JSValueRef prototype, gpointer, GDestroyNotify);
-    JSGlobalContextRef createContextWithJSWrappper(JSContextGroupRef, JSClassRef, JSValueRef prototype, gpointer, GDestroyNotify);
+    JSObject* createJSWrapper(JSGlobalContextRef, JSClassRef, JSValueRef prototype, gpointer, GDestroyNotify);
+    JSGlobalContextRef createContextWithJSWrapper(JSContextGroupRef, JSClassRef, JSValueRef prototype, gpointer, GDestroyNotify);
     JSObject* jsWrapper(gpointer wrappedObject) const;
     gpointer wrappedObject(JSGlobalContextRef, JSObjectRef) const;
 

Modified: trunk/Source/_javascript_Core/ChangeLog (283365 => 283366)


--- trunk/Source/_javascript_Core/ChangeLog	2021-10-01 13:41:33 UTC (rev 283365)
+++ trunk/Source/_javascript_Core/ChangeLog	2021-10-01 14:23:58 UTC (rev 283366)
@@ -1,3 +1,20 @@
+2021-10-01  Olivier Blin  <olivier.b...@softathome.com>
+
+        [JSC][GLib] Fix typo in WrapperMap creation methods
+        https://bugs.webkit.org/show_bug.cgi?id=231076
+
+        Reviewed by Michael Catanzaro.
+
+        * API/glib/JSCContext.cpp:
+        (jscContextGetOrCreateJSWrapper):
+        (jscContextCreateContextWithJSWrapper):
+        * API/glib/JSCWrapperMap.cpp:
+        (JSC::WrapperMap::createJSWrapper):
+        (JSC::WrapperMap::createContextWithJSWrapper):
+        (JSC::WrapperMap::createJSWrappper): Deleted.
+        (JSC::WrapperMap::createContextWithJSWrappper): Deleted.
+        * API/glib/JSCWrapperMap.h:
+
 2021-09-30  Yusuke Suzuki  <ysuz...@apple.com>
 
         Upgrade GCC requirement to 8.3.0
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to