Populate cross-origin location access SecurityError messages.

Source/bindings/v8/V8Initializer.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 85 matching lines @@
     String resource = shouldUseDocumentURL ? firstWindow->document()->url() : toWebCoreString(resourceName);
     firstWindow->document()->reportException(errorMessage, message->GetLineNumber(), resource, callStack);
 }
 
 static void failedAccessCheckCallbackInMainThread(v8::Local<v8::Object> host, v8::AccessType type, v8::Local<v8::Value> data)
 {
     Frame* target = findFrame(host, data, v8::Isolate::GetCurrent());
     if (!target)
         return;
     DOMWindow* targetWindow = target->document()->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()).utf8().data(), v8::Isolate::GetCurrent());

[abarth-chromium, 2013/07/17 18:27:59]

Why utf8().data()?  Can't we just pass in the string?  That way we won't have to
make a bunch of copies.

[arv (Not doing code reviews), 2013/07/17 18:48:56]

I'm fine with us changing all the exception methods to be changed to String
instead of char* if Adam prefers that.

+    else
+        targetWindow->printErrorMessage(targetWindow->crossDomainAccessErrorMessage(activeDOMWindow()));
 }
 
 static bool codeGenerationCheckCallbackInMainThread(v8::Local<v8::Context> context)
 {
     if (ScriptExecutionContext* scriptExecutionContext = toScriptExecutionContext(context)) {
         if (ContentSecurityPolicy* policy = toDocument(scriptExecutionContext)->contentSecurityPolicy())
             return policy->allowEval(ScriptState::forContext(context));
     }
     return false;
 }
@@ skipping 63 matching lines @@
 
     v8::ResourceConstraints resourceConstraints;
     uint32_t here;
     resourceConstraints.set_stack_limit(&here - kWorkerMaxStackSize / sizeof(uint32_t*));
     v8::SetResourceConstraints(&resourceConstraints);
 
     V8PerIsolateData::ensureInitialized(isolate);
 }
 
 } // namespace WebCore