When the list-style-image CSS property is set on an audio/video element or an ancestor (see example in URL), that image gets rendered on top of the play and volume buttons. It ought to be ignored, else the controls look weird on pages that happen to use list-style-image for other purposes. Tested in Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.3a4pre) Gecko/20100324 Minefield/3.7a4pre

The controls should probably style themselves with list-style-image:none.

Actually, I would have expected inheritstyle="false" to prevent this. ...and it probably does, except it seems I forgot to add that somewhere along the line! :-o

Hrm. It doesn't. I thought that controlled if external style affected the binding, but upon rereading the XBL docs it seems to imply it just controls style from other bindings? [Slightly oddly, I can't content style rules to affect the video controls directly, though.] This seems somewhat unfortunate; to fully fix this bug it seems like I'd have to figure out every style attribute in the videocontrols that defaults to inherit, and make sure it's explicitly set to some value at the root binding node? Would be nice to have a way to say inheritstyle="ignorecontent". Also, XBL2 and a pony. :/

(Exporting stuff from my patch queue before embarking on a new project.) I think we can go ahead and land this. In a quick search, I couldn't find any other obvious style properties that exhibit this inheritance problem with video/audio controls. I will also file a bug for preventing style inheritance into <xbl:content/> in the general case.

Oh wait. Font properties also get messed up. Ah, I see that bug 335281 got marked INVALID. If anything, a stylesheet-based "CSS reset" would be handy here, e.g. something like: [resetstyles=true] { list-style-image: none; font-weight: normal; font-style: normal; /* insert other properties that need to be reset */ }

I went through the list of inherited properties, and this seems to cover it.

Comment on attachment 550264 [details] [diff] [review] patch v2 Review of attachment 550264 [details] [diff] [review]: ----------------------------------------------------------------- Is that just the inherited properties that currently can impact videocontrols? Seems short. :) Anyway, better than nothing.

Pushed to fx-team: https://hg.mozilla.org/integration/fx-team/rev/bf5d54a5717a (In reply to comment #7) > Is that just the inherited properties that currently can impact > videocontrols? Seems short. :) Anyway, better than nothing. I checked this table of CSS2.1 properties: http://www.w3.org/TR/CSS21/propidx.html There could be properties in CSS3: http://meiert.com/en/indices/css-properties/ or Mozilla CSS Extensions: https://developer.mozilla.org/en/CSS_Reference/Mozilla_Extensions that could impact videocontrols, but those lists don't have a convenient boolean "inherited" column, so I didn't take the time to check them all.

There is a potential spoof vector here. I'd suggest [sg:spoof] or something in the whiteboard.