Closed Bug 1173876 Opened 4 years ago Closed 4 years ago

make getText{At,After,Before}OffsetCB work with proxies

Categories

(Core :: Disability Access APIs, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: tbsaunde, Assigned: tbsaunde)

Details

Attachments

(1 file)

No description provided.
Comment on attachment 8621098 [details] [diff] [review]
make getText{At,After,Before}OffsetCB work with proxies

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

::: accessible/atk/nsMaiInterfaceText.cpp
@@ +160,5 @@
>  getTextAfterOffsetCB(AtkText *aText, gint aOffset,
>                       AtkTextBoundary aBoundaryType,
>                       gint *aStartOffset, gint *aEndOffset)
>  {
> +    nsAutoString autoStr;

should be indented with 2 spaces not 4

@@ +172,5 @@
> +    text->TextAfterOffset(aOffset, aBoundaryType, &startOffset, &endOffset, autoStr);
> +    ConvertTexttoAsterisks(accWrap, autoStr);
> +  } else if (ProxyAccessible* proxy = GetProxy(ATK_OBJECT(aText))) {
> +    proxy->GetTextAfterOffset(aOffset, aBoundaryType, autoStr, &startOffset,
> +                              &endOffset);

not in the purview of this patch, but it seems pretty inconsistent that the arguments for GetText*Offset methods in ProxyAccessible are not in the same order as for AccessibleWraps
Attachment #8621098 - Flags: review?(lorien) → review+
Comment on attachment 8621098 [details] [diff] [review]
make getText{At,After,Before}OffsetCB work with proxies

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

::: accessible/atk/nsMaiInterfaceText.cpp
@@ +200,5 @@
>      ConvertTexttoAsterisks(accWrap, autoStr);
> +  } else if (ProxyAccessible* proxy = GetProxy(ATK_OBJECT(aText))) {
> +    proxy->GetTextAtOffset(aOffset, aBoundaryType, autoStr, &startOffset,
> +                           &endOffset);
> +  }

Shouldn't we be calling ConvertTexttoAsterisks for proxies too?
https://hg.mozilla.org/mozilla-central/rev/6dc447b14cd7
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Assignee: nobody → tbsaunde+mozbugs
Flags: needinfo?(tbsaunde+mozbugs)
You need to log in before you can comment on or make changes to this bug.