[FIR] Consider fullyExpandedType instead of original type in isSubtypeForTypeMismatch, consider lookupTag in isError method

compiler/fir/checkers/src/org/jetbrains/kotlin/fir/analysis/checkers/FirHelpers.kt

@@ -382,8 +382,10 @@ private fun lowerThanBound(context: ConeInferenceContext, argument: ConeKotlinTy
 fun FirMemberDeclaration.isInlineOnly(): Boolean = isInline && hasAnnotation(INLINE_ONLY_ANNOTATION_CLASS_ID)
 
 fun isSubtypeForTypeMismatch(context: ConeInferenceContext, subtype: ConeKotlinType, supertype: ConeKotlinType): Boolean {
-    return AbstractTypeChecker.isSubtypeOf(context, subtype, supertype)
+    val subtypeFullyExpanded = subtype.fullyExpandedType(context.session)
-            || isSubtypeOfForFunctionalTypeReturningUnit(context.session.typeContext, subtype, supertype)
+    val supertypeFullyExpanded = supertype.fullyExpandedType(context.session)
+    return AbstractTypeChecker.isSubtypeOf(context, subtypeFullyExpanded, supertypeFullyExpanded)
+            || isSubtypeOfForFunctionalTypeReturningUnit(context.session.typeContext, subtypeFullyExpanded, supertypeFullyExpanded)
 }
 
 private fun isSubtypeOfForFunctionalTypeReturningUnit(

compiler/fir/resolve/src/org/jetbrains/kotlin/fir/types/ConeTypeContext.kt

@@ -85,7 +85,8 @@ interface ConeTypeContext : TypeSystemContext, TypeSystemOptimizationContext, Ty
 
     override fun KotlinTypeMarker.isError(): Boolean {
         assert(this is ConeKotlinType)
-        return this is ConeClassErrorType || this is ConeKotlinErrorType || this.typeConstructor().isError()
+        return this is ConeClassErrorType || this is ConeKotlinErrorType || this.typeConstructor().isError() ||
+                (this is ConeClassLikeType && this.lookupTag is ConeClassLikeErrorLookupTag)
     }
 
     override fun KotlinTypeMarker.isUninferredParameter(): Boolean {

compiler/testData/diagnostics/tests/generics/projectionsScope/unsafeVarianceInAliasedFunctionalType.fir.kt

@@ -1,11 +0,0 @@
-class Foo<out T>(val baz: Baz<T>)
-
-class Bar {
-    val foo: Foo<*> = TODO()
-
-    fun <T> bar(): Baz<T> {
-        return <!RETURN_TYPE_MISMATCH!>foo.baz<!>
-    }
-}
-
-typealias Baz<T> = (@UnsafeVariance T) -> Unit

compiler/testData/diagnostics/tests/generics/projectionsScope/unsafeVarianceInAliasedFunctionalType.kt

@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 class Foo<out T>(val baz: Baz<T>)
 
 class Bar {

compiler/testData/diagnostics/testsWithStdLib/java/functionN.fir.kt

@@ -47,7 +47,7 @@ fun main() {
     a.baz(listOf())
 
     a.manyParams(null)
-    a.<!NONE_APPLICABLE!>manyParams<!>(any<kotlin.jvm.functions.FunctionN<Unit>>())
+    a.manyParams(any<kotlin.jvm.functions.FunctionN<Unit>>())
 
     // Potentially, this would have better to forbid calling manyParams, too.
     // But it might be complicated because we need to match that it is an override