Closed Bug 1001157 Opened 6 years ago Closed 6 years ago
Assert in bindings code if there is a pending exception on the cx but no corresponding error marked on the Error
And in particular, we need to assert if there is an exception pending but the method/property is not marked [Throws].
Have you pushed this to try? I wonder how many existing issues this will uncover!
I thought I had, but it looks like I accidentally did an opt push, which is not helpful. Doing another try push now (linux-only, since we don't have debug b2g coverage anyway).
I suppose I could do a MOZ_CRASH + try run on all platforms...
(In reply to comment #4) > I suppose I could do a MOZ_CRASH + try run on all platforms... Why would we use MOZ_CRASH?
We wouldn't use it in production. We'd use it in a try run to cause failures even on b2g, where we only run tests on opt builds.
Incidentally, my try run is at https://tbpl.mozilla.org/?tree=Try&rev=90a639fc29d6
Attachment #8412159 - Flags: review?(peterv) → review+
Backed this out in http://hg.mozilla.org/integration/mozilla-inbound/rev/c46fb6bada28 along with backing out bug 997285 for various busted things: https://tbpl.mozilla.org/php/getParsedLog.php?id=39073608&tree=Mozilla-Inbound https://tbpl.mozilla.org/php/getParsedLog.php?id=39073379&tree=Mozilla-Inbound https://tbpl.mozilla.org/php/getParsedLog.php?id=39073678&tree=Mozilla-Inbound
None of those failures were from this patch. Relanded: https://hg.mozilla.org/integration/mozilla-inbound/rev/823c0199a31d
Flags: needinfo?(bzbarsky) → in-testsuite+
Whiteboard: [need review]
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.