1133717 - Dying in NativeJavaPackage contructor

Status: RESOLVED INACTIVE

(Rhino Graveyard :: Core, defect)

Description

[Kim]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/40.0.2214.93 Safari/537.36

Steps to reproduce:

<Test Case>
public void test(){
  NativeJavaPackage pack = new NativeJavaPackage("package");
}


Actual results:

NullPointerException in the constructor when currentContext is not available. 
public NativeJavaPackage(String packageName) {
  this(false, packageName,        Context.getCurrentContext().getApplicationClassLoader());
} 


Expected results:

I think there should be a null-checker for Context.getCurrentContext() in the constructor like  getContext() method in Context.java throws an exception in the same situation. 
    static Context getContext()
    {
        Context cx = getCurrentContext();
        if (cx == null) {
            throw new RuntimeException(
                "No Context associated with current Thread");
        }
        return cx;
    }

Comment 1

[Sylvestre Ledru [:Sylvestre]]

Closing. Bug management is now done here:
https://github.com/mozilla/rhino