videocontrols.xml has: // Make the <video> element keyboard accessible. this.video.setAttribute("tabindex", 0); This is bad. We should not be modifying the DOM visible to the page. We need to find another way to do this. Neil: Is there a better way to do this?
You need to override nsIContent::IsHTMLFocusable for the video element.
Created attachment 711952 [details] [diff] [review] Don't modify the page DOM from videocontrols
5 years ago