Attempts to edit styles on resource: stylesheets fail because of bug 343508. For some reason, this completely borks the domi UI. We should disable editing of styles on resource: stylesheets.

I don't like this at all. I see what you're trying to do, and it's acceptable to me in the short term... but I'd request (not require) that a new bug be filed to back this code out once bug 343508 is fixed. I would very much prefer we fix the underlying bug instead. This patch merely wallpapers over the real bug (i.e. makes it harder to reproduce the underlying bug).

(In reply to comment #2) > I'd request (not require) that a new bug be > filed to back this code out once bug 343508 is fixed. That was my intention. (In reply to comment #2) > I would very much prefer we fix the underlying bug instead. Me too, but hacking layout is beyond my abilities. I realized after I left the house that there's no reason to be disabling Copy... New patch coming up.

Don't disable Copy.

Comment on attachment 238863 [details] [diff] [review] patch v2 >+ rule.parentStyleSheet.href.indexOf("resource:") == 0); Nit: /^resource:/.test(href)

Updated to comment.

I've filed bug 354082 to back this out once bug 343508's fixed.

extensions/inspector/resources/content/viewers/styleRules/styleRules.js 1.28

Actually... the fact that the sheets are resource: sheets has nothing to do with bug 343508. To produce bug 343508 you need two things: 1) An @import rule in a sheet 2) Cloning said sheet This means, for example, any chrome sheet with an @import rule. But there are ways to trigger sheet cloning with web pages -- just have a web page point twice to the same sheet URI (e.g. for different media).