Precompile more in Blink in Windows for faster compilations

Source/core/frame/Frame.h

Index: Source/core/frame/Frame.h
diff --git a/Source/core/frame/Frame.h b/Source/core/frame/Frame.h
index 224a779b6bfda21c49748b126a09e4c678cf3ac3..ca4c8ec996bf78f989ea945b1140fd4e6130ca97 100644
--- a/Source/core/frame/Frame.h
+++ b/Source/core/frame/Frame.h
@@ -29,6 +29,7 @@
 #define Frame_h
 
 #include "core/CoreExport.h"
+#include "core/frame/FrameHost.h"

[Nico, 2015/06/25 23:14:44]

Why this?

[sof, 2015/06/26 06:23:03]

It should be FrameOwner.h

It is needed to address the following without it:

c:\src\blink\src\third_party\webkit\source\platform\heap\visitor.h(159) : error
C2338: T needs to be a garbage collected object
        c:\src\blink\src\third_party\webkit\source\platform\heap\visitor.h(169)
: see reference to function template instantiation 'void
blink::VisitorHelper<blink::Visitor>::mark<T>(T *)' being compiled
        with
        [
            T=blink::FrameOwner
        ]
        c:\src\blink\src\third_party\webkit\source\platform\heap\visitor.h(169)
: see reference to function template instantiation 'void
blink::VisitorHelper<blink::Visitor>::mark<T>(T *)' being compiled
        with
        [
            T=blink::FrameOwner
        ]
        c:\src\blink\src\third_party\webkit\source\core\frame\frame.cpp(81) :
see reference to function template instantiation 'void
blink::VisitorHelper<blink::Visitor>::trace<blink::FrameOwner>(const
blink::Member<blink::FrameOwner> &)' being compiled
        c:\src\blink\src\third_party\webkit\source\core\frame\frame.cpp(81) :
see reference to function template instantiation 'void
blink::VisitorHelper<blink::Visitor>::trace<blink::FrameOwner>(const
blink::Member<blink::FrameOwner> &)' being compiled
        c:\src\blink\src\third_party\webkit\source\core\frame\frame.cpp(77) :
see reference to function template instantiation 'void
blink::Frame::traceImpl<blink::Visitor*>(VisitorDispatcher)' being compiled
        with
        [
            VisitorDispatcher=blink::Visitor *
        ]

i.e., the full declaration of FrameOwner.h is not in scope when compiling the
trace() method for Frame (this is with MSVC2013)

Which makes little sense, as it is #include'd at the top of Frame.cpp.

We're not running into this on trunk with MSVC, so some unexplained pch "issue".
Bringing the declaration of FrameOwner into scope in Frame.h addresses this; I
don't know why.

It'd be good to leave behind a comment/TODO.

 #include "core/frame/FrameTypes.h"
 #include "core/loader/FrameLoaderTypes.h"
 #include "core/page/FrameTree.h"