Make internal: don't suggest in interface

#KT-35346 Fixed

idea/idea-core/src/org/jetbrains/kotlin/idea/core/psiModificationUtils.kt

@@ -348,7 +348,10 @@ fun KtModifierListOwner.canBeProtected(): Boolean {
 }
 
 fun KtModifierListOwner.canBeInternal(): Boolean {
-    if (containingClass()?.isInterface() == true && hasJvmFieldAnnotation()) return false
+    if (containingClass()?.isInterface() == true) {
+        val objectDeclaration = getStrictParentOfType<KtObjectDeclaration>() ?: return false
+        if (objectDeclaration.isCompanion() && hasJvmFieldAnnotation()) return false
+    }
     return !isAnnotationClassPrimaryConstructor()
 }
 

idea/testData/intentions/changeVisibility/internal/hasJvmFieldInInterfaceObject.kt

@@ -0,0 +1,7 @@
+// WITH_RUNTIME
+interface KotlinInterface {
+    object O {
+        @JvmField
+        <caret>val bar = Any()
+    }
+}

idea/testData/intentions/changeVisibility/internal/hasJvmFieldInInterfaceObject.kt.after

@@ -0,0 +1,7 @@
+// WITH_RUNTIME
+interface KotlinInterface {
+    object O {
+        @JvmField
+        internal val bar = Any()
+    }
+}

idea/testData/intentions/changeVisibility/internal/inInterface.kt

@@ -0,0 +1,4 @@
+// IS_APPLICABLE: false
+interface A {
+    <caret>fun foo(name: String)
+}

idea/testData/intentions/changeVisibility/internal/inInterfaceCompanion.kt

@@ -0,0 +1,5 @@
+interface KotlinInterface {
+    companion object {
+        <caret>val bar = Any()
+    }
+}

idea/testData/intentions/changeVisibility/internal/inInterfaceCompanion.kt.after

@@ -0,0 +1,5 @@
+interface KotlinInterface {
+    companion object {
+        internal val bar = Any()
+    }
+}

idea/tests/org/jetbrains/kotlin/idea/intentions/IntentionTestGenerated.java

@@ -3333,9 +3333,24 @@ public class IntentionTestGenerated extends AbstractIntentionTest {
                 KotlinTestUtils.assertAllTestsPresentByMetadataWithExcluded(this.getClass(), new File("idea/testData/intentions/changeVisibility/internal"), Pattern.compile("^([\\w\\-_]+)\\.(kt|kts)$"), null, true);
             }
 
-            @TestMetadata("hasJvmFieldInInterface.kt")
-            public void testHasJvmFieldInInterface() throws Exception {
-                runTest("idea/testData/intentions/changeVisibility/internal/hasJvmFieldInInterface.kt");
+            @TestMetadata("hasJvmFieldInInterfaceCompanion.kt")
+            public void testHasJvmFieldInInterfaceCompanion() throws Exception {
+                runTest("idea/testData/intentions/changeVisibility/internal/hasJvmFieldInInterfaceCompanion.kt");
+            }
+
+            @TestMetadata("hasJvmFieldInInterfaceObject.kt")
+            public void testHasJvmFieldInInterfaceObject() throws Exception {
+                runTest("idea/testData/intentions/changeVisibility/internal/hasJvmFieldInInterfaceObject.kt");
+            }
+
+            @TestMetadata("inInterface.kt")
+            public void testInInterface() throws Exception {
+                runTest("idea/testData/intentions/changeVisibility/internal/inInterface.kt");
+            }
+
+            @TestMetadata("inInterfaceCompanion.kt")
+            public void testInInterfaceCompanion() throws Exception {
+                runTest("idea/testData/intentions/changeVisibility/internal/inInterfaceCompanion.kt");
             }
 
             @TestMetadata("notForAnnotationClassPrimaryConstructor.kt")