wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 856833 part 3. Rip out no longer needed proxy-unwrapping code. r=waldo

Browse Source
This commit is contained in:
Boris Zbarsky 2013-04-02 20:20:37 -04:00
parent 03e9216e8f
commit 52855cafca
1 changed files with 0 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
dom/bindings/BindingUtils.h
View File

@ -189,30 +189,6 @@ inline bool
IsNotDateOrRegExp(JSContext* cx, JSObject* obj)
{
MOZ_ASSERT(obj);
// For simplicity, check for security wrappers up front. In case we
// have a security wrapper, don't forget to enter the compartment of
// the underlying object after unwrapping.
Maybe<JSAutoCompartment> ac;
if (js::IsWrapper(obj)) {
obj = js::UnwrapObjectChecked(obj, /* stopAtOuter = */ false);
if (!obj) {
// If this is a security wrapper, it'd be better to say not here. But
// this function is used to determine whether an object is convertible to
// a callback interface, and we've historically allowed COWs to be passed
// as callback interfaces (in part to allow things like:
//
// |contentWindow.wrappedJSObject.addEventListener(function() {...})|
//
// to work. We could check for COWs here, but bz says we can probably
// just get away with returning true until the JS engine switches to
// ObjectClassIs for this stuff and we can just kill this code.
return true;
}
ac.construct(cx, obj);
}
// Everything except dates and regexps is arraylike
return !JS_ObjectIsDate(cx, obj) && !JS_ObjectIsRegExp(cx, obj);
}