797961 - [AccessFu] Regression when nodes are hidden

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Eitan Isaacson [:eeejay]]

A regression for bug 758884 has appeared where the catch block does not seem to do the right thing if the node does not exist.

Comment 1

[Eitan Isaacson [:eeejay]]

Attachment: Fix disappearing node case

Comment 2

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

Comment on attachment 668087 [details] [diff] [review]
Fix disappearing node case

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

r=me with concerns addressed:

::: accessible/src/jsat/content-script.js
@@ +47,5 @@
>          }
>        } catch (x) {
> +        let acc = Utils.AccRetrieval.
> +          getAccessibleFor(content.document.activeElement);
> +        moved = vc.moveNext(rule, acc, true);

Please confirm what happens if acc is null (can it be null)?
Also, shouldn't this be vc[details.action] instead of moveNext or are you purposely defaulting to moveNext here?

@@ +89,5 @@
>        mm.sendAsyncMessage(aMessage.name, aMessage.json);
>        return true;
>      }
>    } catch (x) {
> +    // Frame may be hidden, we regard this case as false.

What was the exception/error when the frame is hidden?

Comment 3

[Eitan Isaacson [:eeejay]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5612c5ab7fcf

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/5612c5ab7fcf