1389408 - Compile busted: Windows: ImageBitmap.h(66): error C3646: 'mAlphaType': unknown override specifier - Linux: ImageBitmap.h:66:3: error: 'gfxAlphaType' does not name a type - Mac: ImageBitmap.h:66:3: error: unknown type name 'gfxAlphaType'

Status: RESOLVED DUPLICATE of bug 1387912

(Core :: Graphics, defect)

Description

[Jorg K (CEST = GMT+2)]

As per:
https://treeherder.mozilla.org/#/jobs?repo=comm-central&revision=2d37489b09aa224a3931bfe7bf913053994dafb0

Quoting Linux, error varies on Mac and Windows.

/builds/slave/tb-c-cen-lx-000000000000000000/build/objdir-tb/dist/include/mozilla/dom/ImageBitmap.h:66:3: error: 'gfxAlphaType' does not name a type [log…]
 
/builds/slave/tb-c-cen-lx-000000000000000000/build/objdir-tb/dist/include/mozilla/dom/ImageBitmap.h:203:15: error: 'gfxAlphaType' has not been declared [log…]
 
/builds/slave/tb-c-cen-lx-000000000000000000/build/objdir-tb/dist/include/mozilla/dom/ImageBitmap.h:275:9: error: 'gfxAlphaType' does not name a type [log…]
 
/builds/slave/tb-c-cen-lx-000000000000000000/build/objdir-tb/dist/include/mozilla/dom/ImageBitmap.h:203:41: error: 'gfxAlphaType' has not been declared [log…]

Comment 1

[Jorg K (CEST = GMT+2)]

M-C last good: 5322c03f4c85
M-C first bad: 253a8560dc34

Range: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5322c03f4c85&tochange=253a8560dc34

Comment 2

[Jorg K (CEST = GMT+2)]

Attachment: 1389408.patch

Tomcat, can you please land this on M-C.

Comment 3

[Jorg K (CEST = GMT+2)]

Oh, Tomcat is on PTO.

Comment 4

[Jorg K (CEST = GMT+2)]

Comment on attachment 8896168 [details] [diff] [review]
1389408.patch

Requesting review from a few Graphics people listed for dom/canvas.

Comment 5

[Jonathan Kew [:jfkthame]]

An equivalent patch was recently r+'d in bug 1387912, and I just landed it there.

Comment 6

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 8896168 [details] [diff] [review]
1389408.patch

Review of attachment 8896168 [details] [diff] [review]:
-----------------------------------------------------------------

Please put the #include "gfxTypes.h" before the #include "mozilla/*" so some sense of order is maintained.

Comment 7

[Jorg K (CEST = GMT+2)]

Comment on attachment 8896168 [details] [diff] [review]
1389408.patch

Thanks, but that already got landed with the gfxTypes.h after the "mozilla/" block:
https://hg.mozilla.org/mozilla-central/rev/65826179c86e#l1.12