1275586 - Add description caching tests.

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Yura Zenevich [:yzen] (please use "needinfo")]

Continuing e10s compatible tests effort: adding description caching tests inspired by https://dxr.mozilla.org/mozilla-central/source/accessible/tests/mochitest/test_descr.html

Comment 1

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: MozReview Request: Bug 1275586 - adding description caching tests. r=eeejay

Review commit: https://reviewboard.mozilla.org/r/55144/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/55144/

Comment 2

[Eitan Isaacson [:eeejay]]

Comment on attachment 8756386 [details]
MozReview Request: Bug 1275586 - adding description caching tests. r=eeejay

https://reviewboard.mozilla.org/r/55144/#review51904

This looks great! Big fan..

::: accessible/tests/browser/browser_caching_description.js:157
(Diff revision 1)
> +          yield invokeSetAttribute(browser, 'image', attr, value);
> +        }
> +      }
> +      yield onUpdate;
> +      // When attribute change (alt) triggers reorder event, accessible will
> +      // become defunct.

That's strange behavior. Shouldn't an alt change just be a name change? The tree doesn't change. Does it?

Comment 3

[Yura Zenevich [:yzen] (please use "needinfo")]

(In reply to Eitan Isaacson [:eeejay] from comment #2)
> Comment on attachment 8756386 [details]
> MozReview Request: Bug 1275586 - adding description caching tests. r=eeejay
> 
> https://reviewboard.mozilla.org/r/55144/#review51904
> 
> This looks great! Big fan..
> 
> ::: accessible/tests/browser/browser_caching_description.js:157
> (Diff revision 1)
> > +          yield invokeSetAttribute(browser, 'image', attr, value);
> > +        }
> > +      }
> > +      yield onUpdate;
> > +      // When attribute change (alt) triggers reorder event, accessible will
> > +      // become defunct.
> 
> That's strange behavior. Shouldn't an alt change just be a name change? The
> tree doesn't change. Does it?

I believe this is specific to image accessibles and alt attributes only. At least that's what comments in original mochitests suggest: whenever alt attribute is removed/added, it triggers show/hide/reorder sequence.

Comment 4

[Yura Zenevich [:yzen] (please use "needinfo")]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9b505e0c2334

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fbcbc3e1aa5b

Comment 6

[Ryan VanderMeulen [:RyanVM]]

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