Hack to fix Mac64 build break from r169638.

Hack to fix Mac64 build break from r169638.

This part of the source is fairly fragile. I had to temporarily add a
direct include to WebFrame.h because typedefs cannot be forward
declared. Unfortunately, this causes IntSize.h to be included before
Cocoa. Since Cocoa.h can set NSGEOMETRY_TYPES_SAME_AS_CGGEOMETRY_TYPES,
this can cause build failures with conflicting typedef redefinitions of
NSRect. To hack around this, we temporarily move the Cocoa.h include to
the header as well.

BUG=355268
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169826

[eseidel, 2014-03-24 02:48:07]

lgtm

I have no idea if this is kosher or not.  But I'd rather land a hack to fix this
unshipping configuration which has caused the Blink roll to revert, than have
Blink be 100s of revisions behind. :)

CC'd mac folks who can comment and hopefully guide towards a better fix as
needed.

[commit-bot: I haz the power, 2014-03-24 02:48:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dcheng@chromium.org/209663002/20001

[Mark Mentovai, 2014-03-24 03:43:44]

lgtm

I t

[Mark Mentovai, 2014-03-24 03:44:38]

I didn’t mean to hit that button.

I think that whatever file is using NSGEOMETRY_TYPES_SAME_AS_CGGEOMETRY_TYPES is
just missing an #import of <Foundation/NSGeometry.h>, or if Blink typically just
#imports the main header for a framework, <Foundation/Foundation.h>. This macro
is “owned” by the system.

[Mark Mentovai, 2014-03-24 03:46:59]

(“whatever file” = IntSize.h, at least.)

[eseidel, 2014-03-24 03:48:00]

I believe WebKit intentionally avoided including mac headers from those files,
but I don't really know why.  I'm happy to move away from that.

[Mark Mentovai, 2014-03-24 03:49:13]

They’re probably picking them up from some prefix header that we’re not using.

[commit-bot: I haz the power, 2014-03-24 06:55:44]

Change committed as 169826