[Bug 948788]
Trevor Saunders
948788 at bugs.launchpad.net
Fri Mar 15 13:29:14 UTC 2013
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))
>- atk_object_set_name(aAtkObj, NS_ConvertUTF16toUTF8(uniName).get());
>+ setNameCB(aAtkObj, NS_ConvertUTF16toUTF8(uniName).get());
you don't need to implement SetNameCB() or deal with atkobj->name at all
you should be able to just do g_object_notify(blah);
>@@ -1011,6 +1040,7 @@ AccessibleWrap::FirePlatformEvent(AccEvent* aEvent)
> accessible->Name(newName);
> NS_ConvertUTF16toUTF8 utf8Name(newName);
> if (!atkObj->name || !utf8Name.Equals(atkObj->name))
>+ /* XXX also use setNameCB() here? */
> atk_object_set_name(atkObj, utf8Name.get());
just change here too. Note we should do the same thing for
atk_object_set_parent() and atk_object_set_description()
btw I don't really care but most people around here like 8 lines of
context in diffs.
--
You received this bug notification because you are a member of Mozilla
Bugs, which is subscribed to firefox in Ubuntu.
https://bugs.launchpad.net/bugs/948788
Title:
thunderbird crashed on launch
To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/948788/+subscriptions
More information about the Ubuntu-mozillateam-bugs
mailing list