448110 - |--disable-svg| does not build

Status: VERIFIED FIXED

(Core :: SVG, defect)

Description

[Serge Gautherie (:sgautherie)]

{{
.../mozilla/content/xbl/src/nsXBLService.cpp(167) :
error C2039: 'generic' : is not a member of 'nsGkAtoms'

...\mozilla\content\base\src\nsGkAtoms.h(49) :
see declaration of 'nsGkAtoms'

.../mozilla/content/xbl/src/nsXBLService.cpp(167) :
error C2065: 'generic' : undeclared identifier

make[6]: *** [nsXBLService.obj] Error 2
}}


Windows 2000: Visual C++ 2005sp1 (Express Ed.) + Platform SDK 2003r2
Current mozilla-central

[
ac_add_options --disable-vista-sdk-requirements
mk_add_options MOZ_OBJDIR=@TOPSRCDIR@/../../objdir-Firefox
ac_add_options --enable-application=browser

ac_add_options --disable-svg
]

Without the latter option, build completes and runs fine.

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) <nsXBLService.cpp> (Checkin: Comment 7)

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1a2pre) Gecko/2008072618 Minefield/3.1a2pre] (home made) (W2Ksp4)

Fixes comment 0 error.

Comment 2

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) <nsCSSProps.cpp>

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1a2pre) Gecko/2008072618 Minefield/3.1a2pre] (home made) (W2Ksp4)

Fixes (next error)
{{
.../mozilla/layout/style/nsCSSProps.cpp(358) :
error C2065: 'eCSSKeyword_round' : undeclared identifier

make[5]: *** [nsCSSProps.obj] Error 2
}}

***

<http://mxr.mozilla.org/comm-central/search?string=eCSSKeyword_round&case=on&find=%2Flayout%2Fstyle%2FnsCSSProps.cpp%24>
{{
/mozilla/layout/style/nsCSSProps.cpp
    * line 358 -- eCSSKeyword_round, NS_STYLE_BORDER_IMAGE_ROUND,
    * line 1078 -- eCSSKeyword_round, NS_STYLE_STROKE_LINECAP_ROUND,
    * line 1085 -- eCSSKeyword_round, NS_STYLE_STROKE_LINEJOIN_ROUND,
}}

The two latters are already in such an |#ifdef|.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 331434 [details] [diff] [review]
(Bv1) <nsCSSProps.cpp>

This is the wrong fix.  Instead, you should make the keyword available even when MOZ_SVG is not defined by moving the entry for this keyword in nsCSSKeywordList.h.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

(And I'm not sure why you'd think that disabling one of the values of 'border-image' when SVG was not enabled made any sense.)

Comment 5

[Serge Gautherie (:sgautherie)]

Attachment: (Cv1) <nsCSSKeywordList.h> (Checkin: Comment 8)

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1a2pre) Gecko/2008072701 Minefield/3.1a2pre] (home made) (W2Ksp4)

Bv1, with comment 3 suggestion(s),
plus:
*s/entires/entries/
*s/(name,id)/(name, id)/
*4 white space cleanups

(In reply to comment #4)
> (And I'm not sure why you'd think that disabling one of the values of
> 'border-image' when SVG was not enabled made any sense.)

Only because it compiled (and seemed to run) and I had no other clue...

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 331448 [details] [diff] [review]
(Cv1) <nsCSSKeywordList.h> (Checkin: Comment 8)

r=dbaron

Comment 7

[Ian Neal]

Comment on attachment 331432 [details] [diff] [review]
(Av1) <nsXBLService.cpp> (Checkin: Comment 7)

http://hg.mozilla.org/mozilla-central/index.cgi/rev/4dd52626cbe2

Comment 8

[Ian Neal]

Comment on attachment 331448 [details] [diff] [review]
(Cv1) <nsCSSKeywordList.h> (Checkin: Comment 8)

http://hg.mozilla.org/mozilla-central/index.cgi/rev/4dd52626cbe2

Comment 9

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1a2pre) Gecko/2008080208 SeaMonkey/2.0a1pre] (home, debug) (W2Ksp4)

V.Fixed