fix(android)(9_0_X): obtain holder for module references (#11561)

Fixes TIMOB-27777

android/kroll-apt/src/main/resources/org/appcelerator/kroll/annotations/generator/ProxyBinding.fm

@@ -755,8 +755,17 @@ beprovided to handle the result.
 		holder = holder->FindInstanceInPrototypeChain(getProxyTemplate(isolate));
 	}
 	if (holder.IsEmpty() || holder->IsNull()) {
-		LOGE(TAG, "Couldn't obtain argument holder");
-		args.GetReturnValue().Set(v8::Undefined(isolate));
-		return;
+		if (!moduleInstance.IsEmpty()) {
+			holder = moduleInstance.Get(isolate);
+			if (holder.IsEmpty() || holder->IsNull()) {
+				LOGE(TAG, "Couldn't obtain argument holder");
+				args.GetReturnValue().Set(v8::Undefined(isolate));
+				return;
+			}
+		} else {
+			LOGE(TAG, "Couldn't obtain argument holder");
+			args.GetReturnValue().Set(v8::Undefined(isolate));
+			return;
+		}
 	}
 </#macro>

android/kroll-apt/src/main/resources/org/appcelerator/kroll/annotations/generator/ProxyBindingV8.cpp.fm

@@ -37,6 +37,7 @@ using namespace v8;
 <#assign className = Proxy.className(proxyClassName)>
 
 Persistent<FunctionTemplate> ${className}::proxyTemplate;
+Persistent<Object> ${className}::moduleInstance;
 jclass ${className}::javaClass = NULL;
 
 ${className}::${className}() : titanium::Proxy()
@@ -60,12 +61,13 @@ void ${className}::bindProxy(Local<Object> exports, Local<Context> context)
 	Local<String> nameSymbol = NEW_SYMBOL(isolate, "${proxyAttrs.name}"); // use symbol over string for efficiency
 	<#if isModule>
 	MaybeLocal<Object> maybeInstance = constructor->NewInstance(context);
-	Local<Object> moduleInstance;
-	if (!maybeInstance.ToLocal(&moduleInstance)) {
+	Local<Object> instance;
+	if (!maybeInstance.ToLocal(&instance)) {
 		titanium::V8Util::fatalException(isolate, tryCatch);
 		return;
 	}
-	exports->Set(context, nameSymbol, moduleInstance);
+	exports->Set(context, nameSymbol, instance);
+	moduleInstance.Reset(isolate, instance);
 	<#else>
 	exports->Set(context, nameSymbol, constructor);
 	</#if>
@@ -77,6 +79,9 @@ void ${className}::dispose(Isolate* isolate)
 	if (!proxyTemplate.IsEmpty()) {
 		proxyTemplate.Reset();
 	}
+	if (!moduleInstance.IsEmpty()) {
+	    moduleInstance.Reset();
+	}
 
 	<#if superProxyClassName??>
 	<@Proxy.superNamespace/>${superProxyClassName}::dispose(isolate);

android/kroll-apt/src/main/resources/org/appcelerator/kroll/annotations/generator/ProxyBindingV8.h.fm

@@ -27,6 +27,7 @@ public:
 
 private:
 	static v8::Persistent<v8::FunctionTemplate> proxyTemplate;
+	static v8::Persistent<v8::Object> moduleInstance;
 
 	// Methods -----------------------------------------------------------
 	<@Proxy.listMethods ; isFirst, name, method>

tests/Resources/core.runtime.addontest.js

@@ -0,0 +1,27 @@
+/*
+ * Appcelerator Titanium Mobile
+ * Copyright (c) 2020 by Axway, Inc. All Rights Reserved.
+ * Licensed under the terms of the Apache Public License
+ * Please see the LICENSE included with this distribution for details.
+ */
+/* eslint-env mocha */
+/* eslint no-unused-expressions: "off" */
+
+'use strict';
+
+const should = require('./utilities/assertions');
+
+describe.windowsBroken('Core', () => {
+	describe('Runtime', () => {
+		describe('Static Method Reference', () => {
+
+			// Reference static method.
+			const createProxy = Ti.createProxy;
+			should(createProxy).be.a.Function;
+
+			// Attempt to call static method.
+			const result = createProxy({});
+			should(result).be.a.Object;
+		});
+	});
+});