IDEA-106814 Insert both opening and closing parentheses when completing a method call with "Insert pair brace" option off

java/java-impl/src/com/intellij/codeInsight/completion/JavaCompletionUtil.java

@@ -764,12 +764,14 @@ public class JavaCompletionUtil {
       context.setAddCompletionChar(false);
     }
 
-    final boolean needRightParenth = forceClosingParenthesis || !smart && (CodeInsightSettings.getInstance().AUTOINSERT_PAIR_BRACKET || hasTail);
     if (hasTail) {
       hasParams = false;
     }
+    final boolean needRightParenth = forceClosingParenthesis || 
+                                     !smart && (CodeInsightSettings.getInstance().AUTOINSERT_PAIR_BRACKET || 
+                                                !hasParams && completionChar != '(');
 
-    PsiDocumentManager.getInstance(context.getProject()).commitDocument(context.getDocument());
+    context.commitDocument();
 
     final CommonCodeStyleSettings styleSettings = context.getCodeStyleSettings();
     final PsiElement elementAt = file.findElementAt(context.getStartOffset());

java/java-tests/testData/codeInsight/completion/normal/MethodNoPairBrace_after.java

@@ -3,6 +3,6 @@ class MyClass {
 void foo() {}
 
 {
-  foo(<caret>
+  foo();<caret>
 }
 }

java/java-tests/testData/codeInsight/completion/normal/MethodWithLeftParTailTypeNoPairBrace_after2.java

@@ -3,6 +3,6 @@ class MyClass {
 void foo() {}
 
 {
-  foo(<caret>
+  foo();<caret>
 }
 }

java/java-tests/testData/codeInsight/completion/smartType/NoTailWhenNoPairBracket-out.java

@@ -1,6 +1,6 @@
 class A {
   {
     String x;
-    Class c = x.getClass(<caret>
+    Class c = x.getClass()<caret>
   }
 }

java/java-tests/testData/codeInsight/completion/smartType/NoTailWhenNoPairBracket2-out.java

@@ -1,6 +1,6 @@
 class A {
   {
-    Class c = this.xxxxx(<caret>
+    Class c = this.xxxxx()<caret>
   }
 
   Class xxxxx() {}