1110211 - [gtk3] Atk update

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Martin Stránský [:stransky] (ni? me)]

We need to update Platform init code for gtk3, it loads gtk2 libraries now what causes Firefox crash/freeze on start.

Comment 1

[Martin Stránský [:stransky] (ni? me)]

Attachment: atk-gtk3.patch

Alexander, can you look at it please? The a11y bridge has been integrated to gtk3 and the module is loaded automatically. But the NO_AT_BRIDGE variable still works, which is used by atk_bridge_adaptor_init().

Comment 2

[alexander :surkov (:asurkov)]

Comment on attachment 8537218 [details] [diff] [review]
atk-gtk3.patch

Review of attachment 8537218 [details] [diff] [review]:
-----------------------------------------------------------------

Trevor might know better this part of code than me. Trev are you ok with review?

Comment 3

[Trevor Saunders (:tbsaunde)]

Comment on attachment 8537218 [details] [diff] [review]
atk-gtk3.patch

r? glandium on the really trivial build bits (they lgtm to me though)

This is generally less scary than I thought it might be great

Comment 4

[Martin Stránský [:stransky] (ni? me)]

Try run: https://tbpl.mozilla.org/?tree=Try&rev=abf251f42f16

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2db5144ae9f3

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/2db5144ae9f3

Comment 7

[Mike Hommey [:glandium]]

Well, after having spent time getting elm to build after this change (because at-spi2-atk, which contains atk-bridge wasn't in the gtk3 bundle we used to build), it turns out this adds a runtime dependency on libatk-bridge-2.0.so.0, which isn't available before ubuntu 14.04, and the test slaves run on ubuntu 12.04... I'm afraid this means this needs to be backed out.

Comment 8

[Mike Hommey [:glandium]]

See https://treeherder.mozilla.org/#/jobs?repo=elm&revision=85ba19ec87ff

Comment 9

[Martin Stránský [:stransky] (ni? me)]

Well, the latest Gtk3 is 3.15, recent Fedora one is 3.14. AFAIK Ubuntu 12.04 has 3.6...which is far away from recent gtk3 development. We should pick up some minimal target Gtk3 release but IMHO that version is not even released yet.

Comment 10

[Mike Hommey [:glandium]]

Well, the problem is that what we currently have on automation is Ubuntu 12.04. We can't currently test on anything else.

Comment 11

[Martin Stránský [:stransky] (ni? me)]

I understand. Is it a solution to add atk-bridge-2.0 for gtk3 > 3.6?

Comment 12

[Trevor Saunders (:tbsaunde)]

(In reply to Mike Hommey [:glandium] from comment #7)
> Well, after having spent time getting elm to build after this change
> (because at-spi2-atk, which contains atk-bridge wasn't in the gtk3 bundle we
> used to build), it turns out this adds a runtime dependency on
> libatk-bridge-2.0.so.0, which isn't available before ubuntu 14.04, and the
> test slaves run on ubuntu 12.04... I'm afraid this means this needs to be
> backed out.

I'm a little confused how this bundle is possible, I'd expect libgtk-3.so depends on libatk-bridge-2.so as well, and infact on my machine it does.  I wonder what the accessibility story for this setup is, maybe gtk was compiled with some --disable-accessibility option?

However we could certainly remove -latk-bridge-2 and dlsym the symbol we need, but if gtk should bring the dep in anyway that seems silly.

Comment 13

[Mike Hommey [:glandium]]

atk-bridge didn't exist before at-spi2-stk 2.5, released in April 2012. The first Gtk release that comes after that date is 3.4, but it doesn't require atk-bridge.