Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(364)

Unified Diff: Source/bindings/v8/V8Initializer.cpp

Issue 19564003: Populate cross-origin location access SecurityError messages. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase. Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/bindings/v8/V8Binding.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/bindings/v8/V8Initializer.cpp
diff --git a/Source/bindings/v8/V8Initializer.cpp b/Source/bindings/v8/V8Initializer.cpp
index b7fc31df3e7ca6c43fb90e2ecce99f8579fd9fd5..fe2ada86f695b838d07c0d0602029c3606d764fe 100644
--- a/Source/bindings/v8/V8Initializer.cpp
+++ b/Source/bindings/v8/V8Initializer.cpp
@@ -103,12 +103,13 @@ static void failedAccessCheckCallbackInMainThread(v8::Local<v8::Object> host, v8
if (!target)
return;
DOMWindow* targetWindow = target->domWindow();
- targetWindow->printErrorMessage(targetWindow->crossDomainAccessErrorMessage(activeDOMWindow()));
// Throw an exception for failed-access checks against Location objects, otherwise write to the console.
WrapperTypeInfo* typeInfo = WrapperTypeInfo::unwrap(data);
if (V8Location::info.equals(typeInfo))
- setDOMException(SecurityError, v8::Isolate::GetCurrent());
+ setDOMException(SecurityError, targetWindow->crossDomainAccessErrorMessage(activeDOMWindow()), v8::Isolate::GetCurrent());
+ else
+ targetWindow->printErrorMessage(targetWindow->crossDomainAccessErrorMessage(activeDOMWindow()));
}
static bool codeGenerationCheckCallbackInMainThread(v8::Local<v8::Context> context)
« no previous file with comments | « Source/bindings/v8/V8Binding.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698