Trevor Saunders (:tbsaunde) Reporter
	Description • 9 years ago

the methods in nsMaiInterfaceAction.cpp need to use ProxyAccessible methods for proxied accessibles

	Trevor Saunders (:tbsaunde) Reporter
	Comment 2 • 9 years ago

Comment on attachment 8669221 [details] [diff] [review]
v1

>+void
>+AccessibleWrap::GetKeyBinding(Accessible* aAccessible, nsAString& aResult)
>+{

I'm going to assume this is an exact copy of what was in nsMaiInterfaceAction.cpp

this seems kind of gross, making this a method on Accessible seems a little nicer, but not really so whatever.

also the name doesn't seem that great its more than a key binding, but again meh whatever.

	Pulsebot
	Comment 3 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/2155f5b0b053

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 4 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/2155f5b0b053