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
Assignee: nobody → roc
Attachment #711952 - Flags: review?
6 years ago
Attachment #711952 - Flags: review? → review?(enndeakin)
Attachment #711952 - Flags: review?(enndeakin) → review+
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in before you can comment on or make changes to this bug.