platform: minor fixes
authorRoman Shevchenko <roman.shevchenko@jetbrains.com>
Sun, 8 Feb 2015 09:13:41 +0000 (10:13 +0100)
committerRoman Shevchenko <roman.shevchenko@jetbrains.com>
Sun, 8 Feb 2015 09:13:41 +0000 (10:13 +0100)
 - symmetric cancellation check
 - strict test mode assertion

platform/core-impl/src/com/intellij/psi/impl/PsiManagerImpl.java

index ce78e78b26095a2ac27c6d4a47bd86215ca8445d..0d9f2aae3d0360f4bab54c803fe194e07faaecf6 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2014 JetBrains s.r.o.
+ * Copyright 2000-2015 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -180,25 +180,20 @@ public class PsiManagerImpl extends PsiManagerEx {
 
   @Override
   public PsiFile findFile(@NotNull VirtualFile file) {
+    ProgressIndicatorProvider.checkCanceled();
     return myFileManager.findFile(file);
   }
 
   @Override
   @Nullable
   public FileViewProvider findViewProvider(@NotNull VirtualFile file) {
+    ProgressIndicatorProvider.checkCanceled();
     return myFileManager.findViewProvider(file);
   }
 
-  @TestOnly
-  public void cleanupForNextTest() {
-    myFileManager.cleanupForNextTest();
-    LOG.assertTrue(ApplicationManager.getApplication().isUnitTestMode());
-  }
-
   @Override
   public PsiDirectory findDirectory(@NotNull VirtualFile file) {
     ProgressIndicatorProvider.checkCanceled();
-
     return myFileManager.findDirectory(file);
   }
 
@@ -533,4 +528,10 @@ public class PsiManagerImpl extends PsiManagerEx {
   public boolean isBatchFilesProcessingMode() {
     return myBatchFilesProcessingModeCount.get() > 0;
   }
+
+  @TestOnly
+  public void cleanupForNextTest() {
+    assert ApplicationManager.getApplication().isUnitTestMode();
+    myFileManager.cleanupForNextTest();
+  }
 }