GoogleJavaFormatImportOptimizer: do not overwrite document.text if formatter results are unchanged, or if document.text has changed

This seemed to be responsible for some of the issues with the formatter seemingly not formatting a file.

Author: facboy

idea_plugin/src/main/java/com/google/googlejavaformat/intellij/GoogleJavaFormatImportOptimizer.java

@@ -64,14 +64,24 @@ public boolean supports(@NotNull PsiFile file) {
       return Runnables.doNothing();
     }
 
+    /* pointless to change document text if it hasn't changed, plus this can interfere with
+    e.g. GoogleJavaFormattingService's output, i.e. it can overwrite the results from the main
+    formatter. */
+    if (text.equals(origText)) {
+      return Runnables.doNothing();
+    }
+
     return () -> {
       if (documentManager.isDocumentBlockedByPsi(document)) {
         documentManager.doPostponedOperationsAndUnblockDocument(document);
-        String newText = document.getText();
-        if (!newText.equals(origText)) {
-          return;
-        }
       }
+
+      // similarly to above, don't overwrite new document text if it has changed
+      String newText = document.getText();
+      if (!newText.equals(origText)) {
+        return;
+      }
+
       document.setText(text);
     };
   }