Comment 48 for bug 948788

Revision history for this message
In , Trevor Saunders (trev-saunders) wrote :

(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(AtkObject *aAtkObj, const gchar *name)
> >
> > so, both of the times this is called we have to check if we actually want to
> > fire an event first.
>
> we don't want an event in the former case

not sure what you mean. both places we call atk_object_set_name() we guard it with something that is more or less oldName != newName

now, the first of those shouldn't be needed in an ideal world because we'd always fire name change event, but we don't do that because its between hard and impossible to do that because name algorithm is complicated and depends on all sorts of stuff.