51317 - remove(?) @import & jar.mn of communicator/skin/menu.css (blue,modern skins)

Status: VERIFIED FIXED

(SeaMonkey :: Themes, defect, P3)

Description

[John Morrison]

This bug is somewhat of a fallout from bug 51267, which deals with a
general problem in loading CSS from JAR files. In this one case, it
appears the error message is (mostly) correct (although the error is
only reported in debug builds, loading from a .jar file).

When starting mozilla on Windows there has been this error message in
the console for debug builds.

CSSLoaderImpl::DidLoadStyle: Load of URL 'chrome://communicator/skin/menu.css
' failed.  Error code: 16389

http://lxr.mozilla.org/seamonkey/search?string=communicator%2Fskin%2Fmenu.css

shows the only places that this .css file is referenced, but when you track
down the actual file, 
 http://lxr.mozilla.org/seamonkey/source/themes/modern/communicator/menu.css,
you find that it is a zero-length file (hence no style to load). 

Filing this bug for ben's consideration: should the @import (and jar.mn line)
be removed (maybe, maybe not).

Comment 1

[(not reading, please use [e-mail] instead)]

this has been fixed.

Comment 2

[(not reading, please use [e-mail] instead)]

fixed for both the modern and blue skins.

Comment 3

[John Morrison]

verified fixed. All gone.