Closed Bug 330447 Opened 16 years ago Closed 14 years ago

remove " ." from "No chrome package registered" error message

Categories

(Toolkit :: XUL Widgets, defect)

defect
Not set
trivial

Tracking

()

VERIFIED FIXED
mozilla1.9beta2

People

(Reporter: bugzilla, Assigned: sgautherie)

Details

(Keywords: polish)

Attachments

(1 file)

in:
http://lxr.mozilla.org/seamonkey/source/chrome/src/nsChromeRegistry.cpp#713
we have:
"No chrome package registered for chrome://%s/%s/%s ."
please remove the " .". It looks very strange in the error console log.

it's also written out when you run in safe-mode. shouldn't it ignore it in that situation?
Keywords: polish
OS: Windows XP → All
Hardware: PC → All
Version: unspecified → Trunk
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #287993 - Flags: review?
Attachment #287993 - Flags: review? → review?(gavin.sharp)
Attachment #287993 - Flags: review?(gavin.sharp) → review+
Attachment #287993 - Flags: approval1.9?
Comment on attachment 287993 [details] [diff] [review]
(Av1) removes the point(s)
[Checkin: Comment 4]

a=release drivers
Attachment #287993 - Flags: approval1.9? → approval1.9+
Keywords: checkin-needed
Checking in chrome/src/nsChromeRegistry.cpp;
/cvsroot/mozilla/chrome/src/nsChromeRegistry.cpp,v  <--  nsChromeRegistry.cpp
new revision: 1.358; previous revision: 1.357
done
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M10
Attachment #287993 - Attachment description: (Av1) removes the point(s) → (Av1) removes the point(s) [Checkin: Comment 4]
(In reply to comment #0)
> please remove the " .". It looks very strange in the error console log.

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.0.2pre) Gecko/2008072102 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

V.Fixed, tested with bug 392570.

> it's also written out when you run in safe-mode. shouldn't it ignore it in that
> situation?

I'm not sure what you meant.
File a follow-up bug, as needed.
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.