Convert HTMLFrameOwnerElement and FocusController to use Frame.

Source/bindings/v8/BindingSecurity.h

Index: Source/bindings/v8/BindingSecurity.h
diff --git a/Source/bindings/v8/BindingSecurity.h b/Source/bindings/v8/BindingSecurity.h
index 66db9920142a2f2c282e03578f0b85e6a5a2f78e..390eada0178608ab570d687c749d5e66a674a1de 100644
--- a/Source/bindings/v8/BindingSecurity.h
+++ b/Source/bindings/v8/BindingSecurity.h
@@ -31,6 +31,7 @@
 #ifndef BindingSecurity_h
 #define BindingSecurity_h
 
+#include "core/frame/LocalFrame.h"

[dcheng, 2014/03/17 18:17:44]

Hm... can't this have remained a forward include?

[kenrb, 2014/03/18 20:01:19]

No because a lot of clients call shouldAllowAccessToFrame() using LocalFrames,
and they don't include LocalFrame.h themselves, so if we forward declare then
they wouldn't be able to do the implicit conversion.

[dcheng, 2014/03/19 20:50:03]

Ugh. I think this is OK temporarily but we should clean this up in a follow
patch, so they correctly #include LocalFrame.h themselves.

 #include "wtf/text/WTFString.h"
 #include <v8.h>
 
@@ -38,7 +39,6 @@ namespace WebCore {
 
 class DOMWindow;
 class ExceptionState;
-class LocalFrame;
 class Node;
 
 enum SecurityReportingOption {
@@ -49,8 +49,8 @@ enum SecurityReportingOption {
 class BindingSecurity {
 public:
     static bool shouldAllowAccessToNode(v8::Isolate*, Node*, ExceptionState&);
-    static bool shouldAllowAccessToFrame(v8::Isolate*, LocalFrame*, SecurityReportingOption = ReportSecurityError);
-    static bool shouldAllowAccessToFrame(v8::Isolate*, LocalFrame*, ExceptionState&);
+    static bool shouldAllowAccessToFrame(v8::Isolate*, Frame*, SecurityReportingOption = ReportSecurityError);
+    static bool shouldAllowAccessToFrame(v8::Isolate*, Frame*, ExceptionState&);
 };
 
 }