1100580 - Get rid of EvaluateOptions::needResult

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Boris Zbarsky [:bzbarsky]]

We can use JS::CompileOptions::noScriptRval for this purpose.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: part 1. Drop support for having an off-thread compilation token and wanting a script rval from nsJSUtils::EvaluateString, since we never need that in practice

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: part 2. Get rid of EvaluateOptions::needResult, since we can use JS::CompileOptions::noScriptRval (with the opposite meaning, but same default behavior) for this purpose

Comment 3

[Boris Zbarsky [:bzbarsky]]

   https://hg.mozilla.org/integration/mozilla-inbound/rev/c46d86623e15
   https://hg.mozilla.org/integration/mozilla-inbound/rev/e6fdb771bce0

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/c46d86623e15
https://hg.mozilla.org/mozilla-central/rev/e6fdb771bce0