Just noticed this in my error console: ===== Warning: Error in parsing value for 'clip'. Declaration dropped. Source File: resource://gre-resources/ua.css Line: 273 ===== This is from this line: >276 clip:none !important; http://mxr.mozilla.org/mozilla-central/source/layout/style/ua.css#276 which was added in bug 685779. There is no 'none' value for clip -- the equivalent value (for no clipping) is 'auto': http://www.w3.org/TR/CSS21/visufx.html#propdef-clip
bug 685779 landed before the last migration (and is in aurora for Firefox 10), so we probably want to fix this on aurora after it's fixed on trunk.
Created attachment 577687 [details] [diff] [review] fix (included in patch on bug 702146)
I included this fix in my patch for bug 702146, I'm just waiting for review on it. I'm not sure if we need to take this fix on Aurora/Firefox 10 or not... I've previously discussed it with Roc, and we thought we'd let the patch in bug 702146 (along with this fix) ship in Firefox 11. You're welcome to try to convince me otherwise! :)
(In reply to Chris Pearce (:cpearce) (Mozilla Corporation) from comment #3) > I've previously discussed it with Roc, and we thought we'd let the > patch in bug 702146 (along with this fix) ship in Firefox 11. You're welcome > to try to convince me otherwise! :) Ah -- I was assuming there might be security and/or rendering-potentially-busted implications, but it sounds like it's not too bad. Marking this as depends-on bug 702146 for now (to be updated to FIXED by that bug when it lands). Feel free to dupe if that's cleaner.
Looks like bug 702146's patch has now landed, so this is FIXED by that bug.