[V8] Make an Isolate parameter mandatory in throwError()
Created attachment 184157 [details] Patch
Comment on attachment 184157 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=184157&action=review > Source/WebCore/bindings/v8/V8Binding.h:466 > + // FIXME: This should receive an Isolate. I found that this requires a bunch of fixes. I'll do it later.
Comment on attachment 184157 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=184157&action=review > Source/WebCore/bindings/v8/WorkerScriptController.cpp:174 > - state->exception = throwError(v8GeneralError, state->errorMessage.utf8().data()); > + state->exception = throwError(v8GeneralError, state->errorMessage.utf8().data(), m_context.get()->GetIsolate()); You shouldn't need to call get(). We've overloaded operator->
Created attachment 184163 [details] patch for landing
Comment on attachment 184163 [details] patch for landing Clearing flags on attachment: 184163 Committed r140516: <http://trac.webkit.org/changeset/140516>
All reviewed patches have been landed. Closing bug.