Title: [249187] trunk/Source/WebCore
Revision
249187
Author
[email protected]
Date
2019-08-27 18:22:37 -0700 (Tue, 27 Aug 2019)

Log Message

Unreviewed. Build fix after r249175.

Fix the condition which generates the declaration of vm in
GenerateGetOwnPropertySlotByIndex.

* bindings/scripts/CodeGeneratorJS.pm:
(GenerateGetOwnPropertySlotByIndex):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (249186 => 249187)


--- trunk/Source/WebCore/ChangeLog	2019-08-28 00:53:07 UTC (rev 249186)
+++ trunk/Source/WebCore/ChangeLog	2019-08-28 01:22:37 UTC (rev 249187)
@@ -1,3 +1,13 @@
+2019-08-27  Said Abou-Hallawa  <[email protected]>
+
+        Unreviewed. Build fix after r249175.
+
+        Fix the condition which generates the declaration of vm in
+        GenerateGetOwnPropertySlotByIndex.
+
+        * bindings/scripts/CodeGeneratorJS.pm:
+        (GenerateGetOwnPropertySlotByIndex):
+
 2019-08-27  Justin Fan  <[email protected]>
 
         [WebGPU] Implement GPUErrors for and relax GPUBuffer validation rules

Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm (249186 => 249187)


--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2019-08-28 00:53:07 UTC (rev 249186)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm	2019-08-28 01:22:37 UTC (rev 249187)
@@ -736,7 +736,7 @@
     
     push(@$outputArray, "bool ${className}::getOwnPropertySlotByIndex(JSObject* object, ExecState* state, unsigned index, PropertySlot& slot)\n");
     push(@$outputArray, "{\n");
-    if ($namedGetterOperation || $interface->extendedAttributes->{Plugin}) {
+    if ($namedGetterOperation || $interface->extendedAttributes->{Plugin} || ($indexedGetterOperation && $indexedGetterOperation->extendedAttributes->{MayThrowException})) {
         push(@$outputArray, "    VM& vm = state->vm();\n");
     }
     push(@$outputArray, "    auto* thisObject = jsCast<${className}*>(object);\n");
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to