https://bugzilla.mozilla.org/attachment.cgi?id=747398&action=edit This needs another update for devs who build without enable-metro.

d3dcommon.h is available in both d3d sdk sources we support. If you are using the win8 sdk (and have your paths set right) the 8.0 sdk version should get included. Otherwise the June 2010 d3d sdk should get included. Only thing I'm not too sure about is if this needs to be wrapped in some sort of D3D ifdef. Doesn't seem like it since we require D3D headers to build, so I think checks like CAIRO_HAS_D2D_SURFACE are obsolete.

FWIW, this is causing Windows B2G desktop build bustage as well (which we didn't notice because they're hidden by default on inbound/m-c).

When this patch gets r+, please land it on birch rather than inbound.

Comment on attachment 747910 [details] [diff] [review] patch whoever gets to it first.

Comment on attachment 747910 [details] [diff] [review] patch Review of attachment 747910 [details] [diff] [review]: ----------------------------------------------------------------- ::: gfx/thebes/gfxWindowsPlatform.h @@ +33,5 @@ > #ifdef CAIRO_HAS_D2D_SURFACE > #include <dxgi.h> > #endif > > +// This header is available in the June 2010 SDK and in the Win8 SDK nit: whitespace at eol.

birch: https://hg.mozilla.org/projects/birch/rev/25c04296c675

Would't it be safer to test for the definition of each of these separately rather than assuming one being undefined means they all are? There might be another SDK 7 update that breaks this the way it is now.