1151402 - make IAccessibleText and IAccessibleEditableText use proxies

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Comment 1

[Trevor Saunders (:tbsaunde)]

Attachment: proxy most of IAccessibleText and IAccessibleEditableText

Comment 2

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

Comment on attachment 8588536 [details] [diff] [review]
proxy most of IAccessibleText and IAccessibleEditableText

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

LGTM. Just some formatting nits. I didn't grumble inline about how I wish we would always used braces even for single ifs. Onward!

::: accessible/ipc/DocAccessibleChild.h
@@ +179,5 @@
>  
>    virtual bool RecvDeleteText(const uint64_t& aID,
>                                const int32_t& aStartPos,
> +                              const int32_t& aEndPos, bool* aValid)
> +    MOZ_OVERRIDE;

The MOZ_OVERRIDE usually goes at the end of the previous line. (same for RecvPasteText)

::: accessible/ipc/ProxyAccessible.cpp
@@ +185,1 @@
>  ProxyAccessible::TextSubstring(int32_t aStartOffset, int32_t aEndOfset,

(We should fix the spelling of aEndOfset sometime but doesn't have to be here)

::: accessible/windows/ia2/ia2AccessibleEditableText.cpp
@@ +93,5 @@
>    A11Y_TRYBLOCK_BEGIN
>  
> +    if (ProxyAccessible* proxy = HyperTextProxyFor(this)) {
> +      return proxy->CutText(aStartOffset, aEndOffset) ? S_OK : E_INVALIDARG;
> +    }

nit: block is indented 2 spaces too far.

@@ +115,5 @@
>    A11Y_TRYBLOCK_BEGIN
>  
> +    if (ProxyAccessible* proxy = HyperTextProxyFor(this)) {
> +      return proxy->PasteText(aOffset) ? S_OK : E_INVALIDARG;
> +    }

nit: block is indented 2 spaces too far.

Comment 3

[Trevor Saunders (:tbsaunde)]

https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=e6a2e727bdc4

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/e6a2e727bdc4