[KT-4124] Add diagnostic that reports of local classes. Remove test for diagnostic of nested types

js/js.translator/src/org/jetbrains/kotlin/js/translate/context/StaticContext.java

@@ -545,20 +545,22 @@ public final class StaticContext {
                 @Nullable
                 @Override
                 public JsExpression apply(@NotNull DeclarationDescriptor descriptor) {
-                    if (isNativeObject(descriptor) || isBuiltin(descriptor)) {
-                        return null;
-                    }
                     if (!(descriptor instanceof ClassDescriptor)) {
                         return null;
                     }
-                    ClassDescriptor cls = (ClassDescriptor) descriptor;
-                    if (cls.getKind() == ClassKind.ENUM_ENTRY || cls.getKind() == ClassKind.OBJECT) {
-                        return null;
-                    }
                     DeclarationDescriptor container = descriptor.getContainingDeclaration();
                     if (container == null) {
                         return null;
                     }
+
+                    if (isNativeObject(descriptor) || isBuiltin(descriptor)) {
+                        return null;
+                    }
+                    ClassDescriptor cls = (ClassDescriptor) descriptor;
+                    if (cls.getKind() == ClassKind.ENUM_ENTRY || cls.getKind() == ClassKind.OBJECT) {
+                        return null;
+                    }
+
                     JsExpression result = getQualifiedReference(container);
                     if (DescriptorUtils.isCompanionObject(container)) {
                         result = Namer.getCompanionObjectAccessor(result);

js/js.translator/testData/expression/when/cases/whenWithRangeClause.kt

@@ -29,4 +29,4 @@ fun get(value: Int): Int {
     invocationCount++
     return value
 }
-var invocationCount = 0
+var invocationCount = 0

js/js.translator/testData/kotlin_lib_ecma5.js

@@ -119,14 +119,14 @@ var Kotlin = {};
         }
         if (typeof staticProperties !== 'undefined') {
             for (p in staticProperties) {
-                if (!staticProperties.hasOwnProperty(p)) {
-                    continue;
-                }
+                //noinspection JSUnfilteredForInLoop
                 property = staticProperties[p];
-                if (typeof property === "function" && typeof property.type !== "undefined" && property.type === Kotlin.TYPE.INIT_FUN) {
+                if (typeof property === "function" && property.type === Kotlin.TYPE.INIT_FUN) {
+                    //noinspection JSUnfilteredForInLoop
                     metadata.types[p] = property;
                 }
                 else {
+                    //noinspection JSUnfilteredForInLoop
                     metadata.staticMembers[p] = property;
                 }
             }
@@ -196,14 +196,15 @@ var Kotlin = {};
 
     Kotlin.defineInnerTypes = function(constructor, types) {
         for (var innerTypeName in types) {
-            if (types.hasOwnProperty(innerTypeName)) {
-                var innerType = types[innerTypeName];
-                innerType.className = innerTypeName;
-                Object.defineProperty(constructor, innerTypeName, {
-                    get: innerType,
-                    configurable: true
-                });
-            }
+            // since types object does not inherit from anything, it's just a map
+            //noinspection JSUnfilteredForInLoop
+            var innerType = types[innerTypeName];
+            innerType.className = innerTypeName;
+            //noinspection JSUnfilteredForInLoop
+            Object.defineProperty(constructor, innerTypeName, {
+                get: innerType,
+                configurable: true
+            });
         }
     };
 
@@ -213,12 +214,12 @@ var Kotlin = {};
         var obj = new noNameClass();
         noNameClass.$metadata$.type = Kotlin.TYPE.OBJECT;
         for (var innerTypeName in metadata.types) {
-            if (metadata.types.hasOwnProperty(innerTypeName)) {
-                Object.defineProperty(obj, innerTypeName, {
-                    get : metadata.types[innerTypeName],
-                    configurable: true
-                });
-            }
+            // since types object does not inherit from anything, it's just a map
+            //noinspection JSUnfilteredForInLoop
+            Object.defineProperty(obj, innerTypeName, {
+                get : metadata.types[innerTypeName],
+                configurable: true
+            });
         }
         return obj;
     };