1195471 - make getChildCountCB work with proxies

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Comment 1

[Trevor Saunders (:tbsaunde)]

Attachment: make getChildCountCB work with proxies

Comment 2

[David Bolter [:davidb] (NeedInfo me for attention)]

Comment on attachment 8648898 [details] [diff] [review]
make getChildCountCB work with proxies

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

r=me with note and nit.

::: accessible/atk/AccessibleWrap.cpp
@@ +830,5 @@
>      return static_cast<gint>(accWrap->EmbeddedChildCount());
> +  }
> +
> +  ProxyAccessible* proxy = GetProxy(aAtkObj);
> +  return proxy->MustPruneChildren() ? 0 : proxy->EmbeddedChildCount();

OK if you are sure that if GetAccessibleWrap fails means GetProxy will always succeed.

::: accessible/ipc/DocAccessibleChild.h
@@ +434,5 @@
>  
>    virtual bool RecvTakeFocus(const uint64_t& aID) override;
>  
> +  virtual bool RecvEmbeddedChildCount(const uint64_t& aID, uint32_t* aCount)
> +    override final;

Please a start new line for the second param (like surrounding method formats).

Comment 3

[Trevor Saunders (:tbsaunde)]

(In reply to David Bolter [:davidb] from comment #2)
> Comment on attachment 8648898 [details] [diff] [review]
> make getChildCountCB work with proxies
> 
> Review of attachment 8648898 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> r=me with note and nit.
> 
> ::: accessible/atk/AccessibleWrap.cpp
> @@ +830,5 @@
> >      return static_cast<gint>(accWrap->EmbeddedChildCount());
> > +  }
> > +
> > +  ProxyAccessible* proxy = GetProxy(aAtkObj);
> > +  return proxy->MustPruneChildren() ? 0 : proxy->EmbeddedChildCount();
> 
> OK if you are sure that if GetAccessibleWrap fails means GetProxy will
> always succeed.

err, that's a bug, but obviously returning 0 is what should happen if its null so I'll fix that

> 
> ::: accessible/ipc/DocAccessibleChild.h
> @@ +434,5 @@
> >  
> >    virtual bool RecvTakeFocus(const uint64_t& aID) override;
> >  
> > +  virtual bool RecvEmbeddedChildCount(const uint64_t& aID, uint32_t* aCount)
> > +    override final;
> 
> Please a start new line for the second param (like surrounding method
> formats).

I don't think that is the format? I'd say its put as much as possible on a line up to 80 chars then break reasonably as usual.  Though there is some stuff that seems to not follow that...

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/00d7cf364086

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/00d7cf364086

Comment 6

[Trevor Saunders (:tbsaunde)]

Attachment: make getChildCountCB work with proxies