Bug 30489 – [Gtk] The document frame/html container claims to be parentless

• Status: RESOLVED
• Resolution: FIXED
• Component: Accessibility

[Joanmarie Diggs]

+++ This bug was initially created as a clone of Bug #25411 +++

Steps to reproduce:

1. In WebKit Gtk or Epiphany, view http://google.com

2. In Accerciser's tree of accessibles, locate the app chosen in step 1, and select the scroll pane (i.e. the parent of the document frame/html container)

3. In Accerciser's iPython console, type:

   acc[0].parent == acc

Expected results: Step 3 would cause True to be returned.

Actual results: Step 3 causes False to be returned. For some reason asking for the parent of the document frame returns None rather than the scroll pane which is its parent. I *believe* this has to do with how apps embed WebKit because we saw this very issue with Yelp (embeds Gecko currently). Besides, what seems to fix 25411 doesn't fix this issue so it's a different bug. :-)

[Joanmarie Diggs]

Created attachment 41392 [details]
possible fix

It's ugly, but it works.

If there's already a method that works to reliably get the accessible parent of the WebView, and I managed to miss it, do please let me know. Otherwise, please review. Thanks!

[Joanmarie Diggs]

Created attachment 41411 [details]
revision 2 - made changes Xan suggested

xan: OK, two really minor things: I think it's worth adding a comment here explaining why are we doing this, and the last NULL check I think you can get rid of, since you'd return NULL anyway
xan: I mean the one checking what gtk_widget_get_accessible returns, just return gtk_widget_get_accessible

----

Done.

[Xan Lopez]

Comment on attachment 41411 [details]
revision 2 - made changes Xan suggested

Bam!

[WebKit Commit Bot]

Comment on attachment 41411 [details]
revision 2 - made changes Xan suggested

Clearing flags on attachment: 41411

Committed r49775: <http://trac.webkit.org/changeset/49775>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.