If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

--disable-xul means NS_STYLE_DISPLAY_INLINE_GRID is undefined, compile failure

NEW
Unassigned

Status

()

Core
XUL
--
minor
8 years ago
a year ago

People

(Reporter: WeirdAl, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
Created attachment 422041 [details] [diff] [review]
patch, v1
(Reporter)

Updated

8 years ago
Attachment #422041 - Attachment is patch: true
Attachment #422041 - Attachment mime type: application/octet-stream → text/plain
(Reporter)

Updated

8 years ago
Attachment #422041 - Flags: superreview?(bzbarsky)
Attachment #422041 - Flags: review?(bzbarsky)
It looks like this makes the checks for NS_STYLE_DISPLAY_INLINE_{GRID,STACK} ifdef-ed out, but not the one for ...INLINE_BOX, which the bug's summary suggests is the problem.
(Reporter)

Comment 2

8 years ago
Whoops - I'm pretty sure the patch is right and the summary is wrong. :(
Summary: --disable-xul means NS_STYLE_DISPLAY_INLINE_BOX is undefined, compile failure → --disable-xul means NS_STYLE_DISPLAY_INLINE_GRID is undefined, compile failure
Attachment #422041 - Flags: superreview?(bzbarsky)
Attachment #422041 - Flags: review?(bzbarsky)
Attachment #422041 - Flags: review+
Comment on attachment 422041 [details] [diff] [review]
patch, v1

r=bzbarsky (no sr needed here).
Can this still be checked in?
I don't think we support --disable-xul anymore, but "#ifdef MOZ_XUL" might be good
anyway for documentation.  I would rather see that we try to *remove* XUL though,
than maintaining it.
Component: Layout → XUL
You need to log in before you can comment on or make changes to this bug.