[EFL] REGRESSION after r153432: some accessibility tests crash.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Aug 2013 21:13:41 +0000 (21:13 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Aug 2013 21:13:41 +0000 (21:13 +0000)
https://bugs.webkit.org/show_bug.cgi?id=119333

Patch by Mario Sanchez Prada <mario.prada@samsung.com> on 2013-08-07
Reviewed by Darin Adler.

Use a RefPtr to store a reference to the parent element of
an AccessibilityUIElement instead of the raw pointer, so we
never derefence a function from an invalid pointer.

* WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:
(WTR::attributesOfElement): Use a RefPtr instead of a raw pointer.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@153798 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp

index b8d6409..d3243ac 100644 (file)
@@ -1,3 +1,17 @@
+2013-08-07  Mario Sanchez Prada  <mario.prada@samsung.com>
+
+        [EFL] REGRESSION after r153432: some accessibility tests crash.
+        https://bugs.webkit.org/show_bug.cgi?id=119333
+
+        Reviewed by Darin Adler.
+
+        Use a RefPtr to store a reference to the parent element of
+        an AccessibilityUIElement instead of the raw pointer, so we
+        never derefence a function from an invalid pointer.
+
+        * WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:
+        (WTR::attributesOfElement): Use a RefPtr instead of a raw pointer.
+
 2013-08-07  Cosmin Truta  <ctruta@blackberry.com>
 
         Unreviewed. Add myself as a committer.
index 58f9313..a44ad46 100644 (file)
@@ -291,7 +291,7 @@ static String attributesOfElement(AccessibilityUIElement* element)
 
     // For the parent we print its role and its name, if available.
     builder.append("AXParent: ");
-    AccessibilityUIElement* parent = element->parentElement().get();
+    RefPtr<AccessibilityUIElement> parent = element->parentElement();
     AtkObject* atkParent = parent ? parent->platformUIElement().get() : 0;
     if (atkParent) {
         builder.append(roleToString(atk_object_get_role(atkParent)));