[Bug 948788]

2013-04-17 Thread Trevor Saunders
remote: https://hg.mozilla.org/integration/mozilla- inbound/rev/a5aa0a654611 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/948788 Title: thunderbird crashed on launch To manage notifications

[Bug 948788]

2013-04-17 Thread Trevor Saunders
Comment on attachment 738132 Implement a replacement of atk_object_set_name() which mimics the behavior without calling atk_object_get_name() landed again, but forgot to qref -u sorry :( -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to

[Bug 948788]

2013-04-16 Thread Trevor Saunders
remote: https://hg.mozilla.org/integration/mozilla- inbound/rev/ecd327272240 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/948788 Title: thunderbird crashed on launch To manage notifications

[Bug 948788]

2013-04-16 Thread Trevor Saunders
Comment on attachment 729066 Implement a replacement of atk_object_set_name() which mimics the behavior without calling atk_object_get_name() sorry it took me so long, but I finally got to test your patch today. It seems good so I checked it in for you ask you just saw :-) -- You received this

[Bug 948788]

2013-03-26 Thread Trevor Saunders
Comment on attachment 728261 Implement a replacement of atk_object_set_name() which mimics the behavior without calling atk_object_get_name() diff --git i/accessible/src/atk/AccessibleWrap.cpp w/accessible/src/atk/AccessibleWrap.cpp index e35da5d..208e955 100644 ---

[Bug 948788]

2013-03-21 Thread Trevor Saunders
(In reply to alexander :surkov from comment #39) (In reply to Trevor Saunders (:tbsaunde) from comment #38) (In reply to alexander :surkov from comment #37) atk_object_set_name() called inside get_name is a wrong thing we try to I think the only thing we try to fix here is the infinite

[Bug 948788]

2013-03-20 Thread Trevor Saunders
Comment on attachment 726580 Implement a replacement of atk_object_set_name() which mimics the behavior without calling atk_object_get_name() +static void +AtkObjectSetName(AtkObject *aAtkObj, const gchar *name) so, both of the times this is called we have to check if we actually want to fire

[Bug 948788]

2013-03-20 Thread Trevor Saunders
(In reply to alexander :surkov from comment #35) (In reply to Trevor Saunders (:tbsaunde) from comment #34) Comment on attachment 726580 Implement a replacement of atk_object_set_name() which mimics the behavior without calling atk_object_get_name() +static void +AtkObjectSetName

[Bug 948788]

2013-03-20 Thread Trevor Saunders
(In reply to alexander :surkov from comment #37) atk_object_set_name() called inside get_name is a wrong thing we try to I think the only thing we try to fix here is the infinite recurssion. remove here. It made us fire name change events which is ridiculous I think. If I get right then ATK

[Bug 948788]

2013-03-16 Thread Trevor Saunders
Comment on attachment 725350 Implement setNameCB() in AccessibleWrap.cpp and use it in getNameCB() instead of atk_object_set_name() @@ -644,11 +644,40 @@ getNameCB(AtkObject* aAtkObj) NS_ConvertUTF8toUTF16 objName(aAtkObj-name); if (!uniName.Equals(objName)) -

[Bug 948788]

2012-04-06 Thread Trevor Saunders
(In reply to alexander :surkov from comment #25) (In reply to JeffreyDecker from comment #24) I would like to take on this bug if that's alright. done, thank you btw. Just reading through the comments though it looks like it's a little unclear as to how we want to refactor to fix the

[Bug 948788]

2012-03-16 Thread Trevor Saunders
(In reply to alexander :surkov from comment #21) (In reply to Ginn Chen from comment #20) We need to copy the string to gchar* anyway. yep, just copy the string every time when we were asked. Doesn't sound good? Actually it's not possible. atk_object_get_name() returns

[Bug 857153]

2012-01-27 Thread Trevor Saunders
landed https://hg.mozilla.org/releases/mozilla-aurora/rev/c0df4333b5b7 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/857153 Title: Needs to get accessibility settings from GSettings To manage

[Bug 857153]

2012-01-13 Thread Trevor Saunders
[Approval Request Comment] Regression caused by (bug #): no bug, gnome's move away from gconf Testing completed (on m-c, etc.): Risk to taking this patch (and alternatives if risky): The old behavior of checking gconf will be used if the dbus call does not succeed (ie, if AT-SPI2 is not

[Bug 857153]

2012-01-11 Thread Trevor Saunders
https://hg.mozilla.org/mozilla-central/rev/e95cb5b98154 https://hg.mozilla.org/mozilla-central/rev/39acd9b60ebc -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/857153 Title: Needs to get

[Bug 857153]

2012-01-05 Thread Trevor Saunders
I wonder whether the PreInit() in ShouldA11yBeEnabled() is really necessary or could be replaced with an assertion to check that PreInit had already been called. Perhaps it is needed for unit tests? I don't think I'm familiar enough with when things happen in widget/ to be sure that call to

[Bug 857153]

2012-01-05 Thread Trevor Saunders
(In reply to Karl Tomlinson (:karlt) from comment #47) (In reply to Karl Tomlinson (:karlt) from comment #44) If there is no session, then I assume there will be no org.a11y.Bus. Actually there probably can still be a org.a11y.Bus if registered in /usr/share/dbus-1/services/

[Bug 857153]

2012-01-05 Thread Trevor Saunders
(In reply to Karl Tomlinson (:karlt) from comment #50) (In reply to Trevor Saunders (:tbsaunde) from comment #49) well, the firstthing that comes to mind is arranging for build machines and test machines to have GNOME_ACCESSIBILITY set, do we have an idea how hard this is? It would

[Bug 857153]

2012-01-04 Thread Trevor Saunders
(In reply to Karl Tomlinson (:karlt) from comment #44) That was really just guessing in comment 42, but would it make sense to skip the check if DBUS_SESSION_BUS_ADDRESS is unset anyway? If there is no session, then I assume there will be no org.a11y.Bus. I was thinking the same thing, but

[Bug 857153]

2011-12-30 Thread Trevor Saunders
(In reply to Reed Loden [:reed] (very busy) from comment #3) probably karlt for the gtk2 widget stuff, and maybe surkov or dbolter for the a11y stuff... or maybe just karlt alone. actually, me or Ginn Chen are probably the better a11y people here, and I think one of us should look at that sort