Rik Cabanier Assignee
	Description • 10 years ago

The HTML A11y group decided to rename drawSystemFocusRing to drawFocusIfNeeded since it is less confusing.

	Rik Cabanier Assignee
	Comment 1 • 10 years ago

Try run: https://tbpl.mozilla.org/?tree=Try&rev=0d606c9e0eeb

	Robert O'Callahan (:roc) (email my personal email if necessary)
	Comment 2 • 10 years ago

Comment on attachment 8360152 [details] [diff] [review]
Renamed idl + impl + test files

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

Thanks!

	Robert O'Callahan (:roc) (email my personal email if necessary)
	Comment 3 • 10 years ago

By the way, should we remove drawCustomFocusRing? It seems the name and/or behavior are likely to change and it won't be standardized for some time.

	Rik Cabanier Assignee
	Comment 4 • 10 years ago

(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #3)
> By the way, should we remove drawCustomFocusRing? It seems the name and/or
> behavior are likely to change and it won't be standardized for some time.

I was planning on leaving it behind the runtime flag for now. 
Maybe it's better to remove it since it is not going to be adopted? If you think so, I will create a bug to remove it.

	Robert O'Callahan (:roc) (email my personal email if necessary)
	Comment 5 • 10 years ago

Maybe it should have its own runtime flag. We probably should enable drawFocusIfNeeded before drawCustomFocusRing.

	Rik Cabanier Assignee
	Comment 6 • 10 years ago

(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #5)
> Maybe it should have its own runtime flag. We probably should enable
> drawFocusIfNeeded before drawCustomFocusRing.

OK. I will introduce a new flag for customfocusring.

	Ryan VanderMeulen [:RyanVM]
	Comment 7 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/81e542a2ab70

	Ryan VanderMeulen [:RyanVM]
	Comment 8 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/81e542a2ab70

	Florian Scholz (Open Web Docs)
	Comment 9 • 9 years ago

https://developer.mozilla.org/en-US/Firefox/Releases/29#Interfaces.2FAPIs.2FDOM
https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D.drawFocusIfNeeded