http://lxr.mozilla.org/seamonkey/source/extensions/inspector/resources/content/viewers/jsObject/jsObjectView.js is no longer packaged: http://lxr.mozilla.org/seamonkey/source/extensions/inspector/jar.mn

Is SR needed here?

Comment on attachment 426947 [details] [diff] [review] remove it [Checkin: Comment 7] Probably not for an unused file, no ;-)

Comment on attachment 426947 [details] [diff] [review] remove it [Checkin: Comment 7] sr+ seems sufficient

(In reply to comment #3) > (From update of attachment 426947 [details] [diff] [review]) > sr+ seems sufficient I'm not sure whether to object to this or not. True, for this purpose (removing a single unused file) it probably only needs one review. But I don't remember ever seeing a patch where sr+ substituted for r+. It does sound childish of me to request adherence to the rules, especially since I'm not very active in contributing to the Mozilla code base these days. What I do believe is that it is not the patch author's decision whether sr+ seems sufficient or not, but the module owner's.

rs=me :)

And objection withdrawn :)

Comment on attachment 426947 [details] [diff] [review] remove it [Checkin: Comment 7] http://hg.mozilla.org/dom-inspector/rev/2bbdd20af50a