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

Status: VERIFIED FIXED

(Toolkit :: UI Widgets, defect)

Description

[Henrik Gemal]

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?

Comment 1

[Wayne Mery (:wsmwk)]

http://lxr.mozilla.org/seamonkey/source/chrome/src/nsChromeRegistry.cpp#719

Comment 2

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) removes the point(s) [Checkin: Comment 4]

Comment 3

[Damon Sicore (:damons)]

Comment on attachment 287993 [details] [diff] [review]
(Av1) removes the point(s)
[Checkin: Comment 4]

a=release drivers

Comment 4

[Reed Loden [:reed]]

Checking in chrome/src/nsChromeRegistry.cpp;
/cvsroot/mozilla/chrome/src/nsChromeRegistry.cpp,v  <--  nsChromeRegistry.cpp
new revision: 1.358; previous revision: 1.357
done

Comment 5

[Serge Gautherie (:sgautherie)]

(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.