James Teh [:Jamie] Reporter
	Description • 6 years ago

We should consider removing the help property from the Accessibility Inspector. At present, there is no "real" implementation of this in Gecko; it always returns an empty string. This is likely to be confusing for users. We can always add it back if anyone ever implements it.

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 1 • 6 years ago

Had to add two dependencies in browser.ini (that inspector's shared-head relies on) as the tests stopped working locally after clobber.

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 2 • 6 years ago

Sorry wrong bug number on the patch.

	Patrick Brosset <:pbro>
	Comment 3 • 6 years ago

Comment on attachment 8985370 [details] [diff] [review]
1467643 patch

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

It looks like the front-end for the tool does *not* use this property today, looking quickly through the code. Is that correct?

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 4 • 6 years ago

(In reply to Patrick Brosset <:pbro> from comment #3)
> Comment on attachment 8985370 [details] [diff] [review]
> 1467643 patch
> 
> Review of attachment 8985370 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> It looks like the front-end for the tool does *not* use this property today,
> looking quickly through the code. Is that correct?

Well it was a row in a11y properties sidebar but never populated. Is that still alright or do I need to account for backward compat?

	Patrick Brosset <:pbro>
	Comment 5 • 6 years ago

I think it's alright. Backward compat is impacted when some client-side code has some knowledge of actors, their methods, or what their return and when any of that may change depending on the target being debugged.
In your case, the UI never assumed anything about this property, so you should be fine.

	Pulsebot
	Comment 6 • 6 years ago

Pushed by yura.zenevich@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5d7ff9efd67f
remove 'help' accessibility property from accessibility inspector. r=pbro

	Dorel Luca [:dluca]
	Comment 7 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/5d7ff9efd67f