586453 - Function.caller is null after Cu.import or mozIJSSubScriptLoader.loadSubScript

Status: RESOLVED WONTFIX

(Core :: JavaScript Engine, defect)

Description

[Danil Ivanov]

User-Agent:       Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b4pre) Gecko/20100810 Minefield/4.0b4pre
Build Identifier: 

Function.caller is null after Components.utils.import(..) or mozIJSSubScriptLoader.loadSubScript(..)

Reproducible: Always

Comment 1

[Danil Ivanov]

Attachment: Sample extension

Comment 2

[Danil Ivanov]

Actual Results
(Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b4pre) Gecko/20100810 Minefield/4.0b4pre)

-== TestCaller.caller ==-
function (outer, iid) {
    if (outer) {
        throw Cr.NS_ERROR_NO_AGGREGATION;
    }
    return (new component).QueryInterface(iid);
}
-== testFn.caller ==-
function TestCaller() {
    dump("-== TestCaller.caller ==-\n" + TestCaller.caller + "\n");
    testFn();
    dump("-== TestCaller.caller ==-\n" + TestCaller.caller + "\n");
}
-== callertest.caller (1) ==-
function testFn() {
    dump("-== testFn.caller ==-\n" + testFn.caller + "\n");
    callertest(1);
    try {
    } catch (e) {
    }
    Cu.import("resource://testcaller/test.jsm");
    callertest(2);
    dump("-== testFn.caller ==-\n" + testFn.caller + "\n");
}
-== callertest.caller (2) ==-
null
-== testFn.caller ==-
null
-== TestCaller.caller ==-
null


Expected Results
(Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.9pre) Gecko/20100810 Namoroka/3.6.9pre)

-== TestCaller.caller ==-
function (outer, iid) {
    if (outer) {
        throw Cr.NS_ERROR_NO_AGGREGATION;
    }
    return (new component).QueryInterface(iid);
}
-== testFn.caller ==-
function TestCaller() {
    dump("-== TestCaller.caller ==-\n" + TestCaller.caller + "\n");
    testFn();
    dump("-== TestCaller.caller ==-\n" + TestCaller.caller + "\n");
}
-== callertest.caller (1) ==-
function testFn() {
    dump("-== testFn.caller ==-\n" + testFn.caller + "\n");
    callertest(1);
    try {
    } catch (e) {
    }
    Cu.import("resource://testcaller/test.jsm");
    callertest(2);
    dump("-== testFn.caller ==-\n" + testFn.caller + "\n");
}
-== callertest.caller (2) ==-
function testFn() {
    dump("-== testFn.caller ==-\n" + testFn.caller + "\n");
    callertest(1);
    try {
    } catch (e) {
    }
    Cu.import("resource://testcaller/test.jsm");
    callertest(2);
    dump("-== testFn.caller ==-\n" + testFn.caller + "\n");
}
-== testFn.caller ==-
function TestCaller() {
    dump("-== TestCaller.caller ==-\n" + TestCaller.caller + "\n");
    testFn();
    dump("-== TestCaller.caller ==-\n" + TestCaller.caller + "\n");
}
-== TestCaller.caller ==-
function (outer, iid) {
    if (outer) {
        throw Cr.NS_ERROR_NO_AGGREGATION;
    }
    return (new component).QueryInterface(iid);
}

Comment 3

[Danil Ivanov]

ok: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b3pre) Gecko/20100730 Minefield/4.0b3pre
ok: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b3pre) Gecko/20100731 Minefield/4.0b3pre
not ok: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b3pre) Gecko/20100801 Minefield/4.0b3pre

Comment 4

[:Gavin Sharp [email: [e-mail]]]

I think I found an hg bisect bug in the process, and perhaps this is already obvious given the CC list, but I finally ended up finding that this was introduced by:

changeset:   48503:3cdb70716688
user:        Blake Kaplan <[e-mail]>
date:        Mon Jul 19 13:36:49 2010 -0700
summary:     Bug 563106 - Tie XPConnect to compartments. r=jorendorff/gal/jst

Comment 5

[Kagami Rosylight [:saschanaz] (they/them) (OOO until Dec 11)]

Maybe wontfix at this point?

Comment 6

[Iain Ireland [:iain]]

Function.caller is non-standard and deprecated.