Title: [281302] releases/WebKitGTK/webkit-2.32/Source/WebCore
Revision
281302
Author
[email protected]
Date
2021-08-20 02:18:31 -0700 (Fri, 20 Aug 2021)

Log Message

Merge r275143 - Don't add Frame as an opaque root of DOMWindow
https://bugs.webkit.org/show_bug.cgi?id=223838

Reviewed by Simon Fraser.

Don't add Frame as an opaque root of DOMWindow since we don't use Frame
as an opaque root of any JS wrappers since r248276.

* bindings/js/JSDOMWindowCustom.cpp:
(WebCore::JSDOMWindow::visitAdditionalChildren):

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.32/Source/WebCore/ChangeLog (281301 => 281302)


--- releases/WebKitGTK/webkit-2.32/Source/WebCore/ChangeLog	2021-08-20 09:16:55 UTC (rev 281301)
+++ releases/WebKitGTK/webkit-2.32/Source/WebCore/ChangeLog	2021-08-20 09:18:31 UTC (rev 281302)
@@ -1,3 +1,16 @@
+2021-03-27  Ryosuke Niwa  <[email protected]>
+
+        Don't add Frame as an opaque root of DOMWindow
+        https://bugs.webkit.org/show_bug.cgi?id=223838
+
+        Reviewed by Simon Fraser.
+
+        Don't add Frame as an opaque root of DOMWindow since we don't use Frame
+        as an opaque root of any JS wrappers since r248276.
+
+        * bindings/js/JSDOMWindowCustom.cpp:
+        (WebCore::JSDOMWindow::visitAdditionalChildren):
+
 2021-03-26  Ian Gilbert  <[email protected]>
 
         Dirty layout for floating children of inline on full layout

Modified: releases/WebKitGTK/webkit-2.32/Source/WebCore/bindings/js/JSDOMWindowCustom.cpp (281301 => 281302)


--- releases/WebKitGTK/webkit-2.32/Source/WebCore/bindings/js/JSDOMWindowCustom.cpp	2021-08-20 09:16:55 UTC (rev 281301)
+++ releases/WebKitGTK/webkit-2.32/Source/WebCore/bindings/js/JSDOMWindowCustom.cpp	2021-08-20 09:18:31 UTC (rev 281302)
@@ -122,9 +122,6 @@
 template<typename Visitor>
 void JSDOMWindow::visitAdditionalChildren(Visitor& visitor)
 {
-    if (Frame* frame = wrapped().frame())
-        visitor.addOpaqueRoot(frame);
-
     visitor.addOpaqueRoot(&wrapped());
     
     // Normally JSEventTargetCustom.cpp's JSEventTarget::visitAdditionalChildren() would call this. But
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to