Changeset 258731 in webkit

Timestamp:

Mar 19, 2020 2:57:00 PM (4 years ago)

Author:

Andres Gonzalez

Message:

URL needs to be isolatedCopied when cached in AXIsolatedObject.
​https://bugs.webkit.org/show_bug.cgi?id=209298

Reviewed by Chris Fleizach.

AXIsolatedObject needs to isolatedCopy the URL property in order to use
it on the secondary thread.

• accessibility/isolatedtree/AXIsolatedObject.cpp:

(WebCore::AXIsolatedObject::initializeAttributeData):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• accessibility/isolatedtree/AXIsolatedObject.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2020-03-19  Andres Gonzalez  <andresg_22@apple.com>
+
+        URL needs to be isolatedCopied when cached in AXIsolatedObject.
+        https://bugs.webkit.org/show_bug.cgi?id=209298
+
+        Reviewed by Chris Fleizach.
+
+        AXIsolatedObject needs to isolatedCopy the URL property in order to use
+        it on the secondary thread.
+
+        * accessibility/isolatedtree/AXIsolatedObject.cpp:
+        (WebCore::AXIsolatedObject::initializeAttributeData):
+
 2020-03-19  Takashi Komori  <Takashi.Komori@sony.com>
 

trunk/Source/WebCore/accessibility/isolatedtree/AXIsolatedObject.cpp

@@ -193 +193 @@
     setProperty(AXPropertyName::ComputedRoleString, object.computedRoleString());
     setProperty(AXPropertyName::ValueAutofillButtonType, static_cast<int>(object.valueAutofillButtonType()));
-    setProperty(AXPropertyName::URL, object.url());
+    setProperty(AXPropertyName::URL, object.url().isolatedCopy());
     setProperty(AXPropertyName::AccessKey, object.accessKey());
     setProperty(AXPropertyName::ActionVerb, object.actionVerb());