Take nullability annotations into account in QF correcting override

So Java NotNull annotated is converted to `Type`
and Java Nullable annotated to `Type?` accordingly

So #KT-19299 Fixed

idea/src/org/jetbrains/kotlin/idea/quickfix/ChangeMemberFunctionSignatureFix.kt

@@ -35,6 +35,9 @@ import org.jetbrains.kotlin.idea.util.IdeDescriptorRenderers
 import org.jetbrains.kotlin.idea.core.ShortenReferences
 import org.jetbrains.kotlin.idea.util.application.executeWriteCommand
 import org.jetbrains.kotlin.incremental.components.NoLookupLocation
+import org.jetbrains.kotlin.load.java.NOT_NULL_ANNOTATIONS
+import org.jetbrains.kotlin.load.java.NULLABLE_ANNOTATIONS
+import org.jetbrains.kotlin.name.FqName
 import org.jetbrains.kotlin.psi.KtFile
 import org.jetbrains.kotlin.psi.KtNamedFunction
 import org.jetbrains.kotlin.psi.KtParameterList
@@ -202,8 +205,7 @@ class ChangeMemberFunctionSignatureFix private constructor(
                     SourceElement.NO_SOURCE
             )
 
-            val parameters = newParameters.withIndex().map {
+            val parameters = newParameters.withIndex().map { (index, parameter) ->
-                val (index, parameter) = it
                 ValueParameterDescriptorImpl(
                         descriptor, null, index,
                         parameter.annotations, parameter.name, parameter.returnType!!, parameter.declaresDefaultValue(),
@@ -319,6 +321,15 @@ class ChangeMemberFunctionSignatureFix private constructor(
                     ShortenReferences.DEFAULT.process(newTypeRef)
                 }
 
+                patternFunction.valueParameters.forEach { param ->
+                    param.annotationEntries.forEach { a ->
+                        a.typeReference?.run {
+                            val fqName = FqName(this.text)
+                            if (fqName in (NULLABLE_ANNOTATIONS + NOT_NULL_ANNOTATIONS)) a.delete()
+                        }
+                    }
+                }
+
                 val newParameterList = function.valueParameterList!!.replace(patternFunction.valueParameterList!!) as KtParameterList
                 ShortenReferences.DEFAULT.process(newParameterList)
             }

idea/testData/quickfix/override/nothingToOverride/overrideJavaMethodWithAnnotation.test

@@ -0,0 +1,38 @@
+// FILE: test.before.kt
+// "Change function signature to 'fun foo(a: String, b: String?, c: String?)'" "true"
+// ERROR: 'foo' overrides nothing
+package foo
+
+class KotlinClass : JavaClass() {
+    <caret>override fun foo() {
+    }
+}
+
+// FILE: foo/JavaClass.java
+package foo;
+
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
+
+public class JavaClass {
+    void foo(@NotNull String a,
+             @Nullable String b,
+             @JavaAnnotation String c) {
+    }
+}
+
+// FILE: foo/JavaAnnotation.java
+package foo;
+
+public @interface JavaAnnotation {
+}
+
+// FILE: test.after.kt
+// "Change function signature to 'fun foo(a: String, b: String?, c: String?)'" "true"
+// ERROR: 'foo' overrides nothing
+package foo
+
+class KotlinClass : JavaClass() {
+    <caret>override fun foo(a: String, b: String?, @JavaAnnotation c: String?) {
+    }
+}

idea/tests/org/jetbrains/kotlin/idea/quickfix/QuickFixMultiFileTestGenerated.java

@@ -1855,6 +1855,12 @@ public class QuickFixMultiFileTestGenerated extends AbstractQuickFixMultiFileTes
                 doTestWithExtraFile(fileName);
             }
 
+            @TestMetadata("overrideJavaMethodWithAnnotation.test")
+            public void testOverrideJavaMethodWithAnnotation() throws Exception {
+                String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/override/nothingToOverride/overrideJavaMethodWithAnnotation.test");
+                doTestWithExtraFile(fileName);
+            }
+
             @TestMetadata("twoPackages.before.Main.kt")
             public void testTwoPackages() throws Exception {
                 String fileName = KotlinTestUtils.navigationMetadata("idea/testData/quickfix/override/nothingToOverride/twoPackages.before.Main.kt");