Closed Bug 775661 Opened 10 years ago Closed 10 years ago

[AccessFu] Add dump tree and accessible states methods to logger

Categories

(Core :: Disability Access APIs, defect)

ARM
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: eeejay, Assigned: eeejay)

Details

Attachments

(1 file)

No description provided.
Comment on attachment 643935 [details] [diff] [review]
Added dump tree Logger method, and states to string function.

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

::: accessible/src/jsat/Utils.jsm
@@ +168,5 @@
> +  },
> +
> +  statesToString: function statesToString(aAccessible) {
> +    let [state, extState] = Utils.getStates(aAccessible);
> +    let statesArray = [];

I'd probably call it stringArray, or strings, but not stingRay because that would be silly.
Attachment #643935 - Flags: review?(dbolter) → review+
http://hg.mozilla.org/integration/mozilla-inbound/rev/69a69e2a873d
Assignee: nobody → eitan
Target Milestone: --- → mozilla17
(In reply to David Bolter [:davidb] from comment #2)
> Comment on attachment 643935 [details] [diff] [review]
> Added dump tree Logger method, and states to string function.
> 
> Review of attachment 643935 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: accessible/src/jsat/Utils.jsm
> @@ +168,5 @@
> > +  },
> > +
> > +  statesToString: function statesToString(aAccessible) {
> > +    let [state, extState] = Utils.getStates(aAccessible);
> > +    let statesArray = [];
> 
> I'd probably call it stringArray, or strings, but not stingRay because that
> would be silly.

LOL. Changed.
https://hg.mozilla.org/mozilla-central/rev/69a69e2a873d
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.