708405 - "Assertion failure: !cx->isExceptionPending()" with Proxy, old NodeList binding

Status: RESOLVED FIXED

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

Description

[Jesse Ruderman]

Attachment: testcase (asserts fatally if pref is set)

1. user_pref("dom.new_bindings", false);
2. Load the testcase

Result:

Assertion failure: !cx->isExceptionPending(), at /builds/slave/m-cen-osx64-dbg/build/js/src/jsinterp.cpp:4781

Comment 1

[Jesse Ruderman]

Attachment: stack trace

Comment 2

[Boris Zbarsky [:bzbarsky]]

This predates the new DOM binding work, for what it's worth.  It's a regression from bug 633133.  The return value of JS_HasPropertyById is being eaten.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: Make sure to propagate out failures from JS_HasPropertyById in the old nodelist resolve hook.

Comment 4

[Boris Zbarsky [:bzbarsky]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/403c7bcfb42a

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/403c7bcfb42a