Throw an exception when denying access to 'Frame's 'location' setter.

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 4387597b96df3713184c19b2be26769eb61104d2..8fba900cb7f9620df77d787598698da115ee9819 100644
--- a/Source/bindings/v8/BindingSecurity.h
+++ b/Source/bindings/v8/BindingSecurity.h
@@ -36,8 +36,8 @@
 namespace WebCore {
 
 class DOMWindow;
+class ExceptionState;
 class Frame;
-class HTMLFrameElementBase;
 class Node;
 
 enum SecurityReportingOption {
@@ -49,7 +49,7 @@ class BindingSecurity {
 public:
     static bool shouldAllowAccessToNode(Node*);
     static bool shouldAllowAccessToFrame(Frame*, SecurityReportingOption = ReportSecurityError);
-    static bool allowSettingFrameSrcToJavascriptUrl(HTMLFrameElementBase*, const String& value);
+    static bool shouldAllowAccessToFrame(Frame*, ExceptionState&);

[abarth-chromium, 2013/08/12 19:57:03]

How does this function differ from the one above it?

[Mike West, 2013/08/13 08:30:51]

The new method accepts an 'ExceptionState&' object and always uses it to report
the access violation. My goal here is to slowly replace the code using the
current version of the method (which always writes an unsuppressable message to
the console) with the new version (which throws a catchable exception).

 };
 
 }