Last Comment Bug 709486 - "Warning: Ignoring obsolete chrome registration modifier 'xpcnativewrappers=no'" message in error console should not contain the prefix "Warning"
: "Warning: Ignoring obsolete chrome registration modifier 'xpcnativewrappers=n...
: polish
Product: Core
Classification: Components
Component: XPCOM (show other bugs)
: Trunk
: All All
-- trivial (vote)
: mozilla12
Assigned To: :aceman
: Nathan Froyd [:froydnj][high latency until 6 March]
Depends on:
  Show dependency treegraph
Reported: 2011-12-10 09:35 PST by :aceman
Modified: 2012-02-23 08:16 PST (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (1.01 KB, patch)
2011-12-10 10:13 PST, :aceman
benjamin: review+
Details | Diff | Splinter Review

Description User image :aceman 2011-12-10 09:35:37 PST
The severity of the message (warning) is already shown by the icon. Also, when copying this message into clipboard, the text representation gets another "Warning: " prefix before the text. The pasted result is then:

Warning: Warning: Ignoring obsolete chrome registration modifier 'xpcnativewrappers=no'.
Source File: jar:file:///.../.thunderbird/nightly.default/extensions/!/chrome.manifest
Line: 17
Comment 1 User image :aceman 2011-12-10 10:13:08 PST
Created attachment 580658 [details] [diff] [review]
Comment 2 User image Daniel Holbert [:dholbert] 2012-01-11 15:47:18 PST
Thanks for the patch! Landed:

Also -- in the future, it'd be awesome if you could set up hg to tag you as the author in your patches (and also a commit message, ideally), as described here:

Thanks again!
Comment 3 User image :aceman 2012-01-11 23:43:54 PST
Yes, I do that, but this patch was created before I started doing it.
Comment 4 User image Daniel Holbert [:dholbert] 2012-01-12 00:05:35 PST
Ah, great then. :)

Note You need to log in before you can comment on or make changes to this bug.