correctly handle inner classes removal

java/java-tests/testData/compileServer/incremental/common/deleteInnerClass.log

@@ -0,0 +1,13 @@
+Cleaning output files:
+out/production/DeleteInnerClass/A$Inner.class
+out/production/DeleteInnerClass/A.class
+End of files
+Compiling files:
+src/A.java
+End of files
+Cleaning output files:
+out/production/DeleteInnerClass/B.class
+End of files
+Compiling files:
+src/B.java
+End of files

java/java-tests/testData/compileServer/incremental/common/deleteInnerClass/src/A.java

@@ -0,0 +1,5 @@
+public class A {
+  public static class Inner {
+    public static final Integer field = new Integer(10);
+  }
+}

java/java-tests/testData/compileServer/incremental/common/deleteInnerClass/src/A.java.new

@@ -0,0 +1,2 @@
+public class A {
+}

java/java-tests/testData/compileServer/incremental/common/deleteInnerClass/src/B.java

@@ -0,0 +1,5 @@
+public class B {
+  Integer foo () {
+    return A.Inner.field;
+  }
+}

java/java-tests/testData/compileServer/incremental/common/deleteInnerClass1.log

@@ -0,0 +1,13 @@
+Cleaning output files:
+out/production/DeleteInnerClass1/A$Inner.class
+out/production/DeleteInnerClass1/A.class
+End of files
+Compiling files:
+src/A.java
+End of files
+Cleaning output files:
+out/production/DeleteInnerClass1/B.class
+End of files
+Compiling files:
+src/B.java
+End of files

java/java-tests/testData/compileServer/incremental/common/deleteInnerClass1/src/A.java

@@ -0,0 +1,6 @@
+public class A {
+  public static class Inner {
+    public static void method(){
+    }
+  }
+}

java/java-tests/testData/compileServer/incremental/common/deleteInnerClass1/src/A.java.new

@@ -0,0 +1,2 @@
+public class A {
+}

java/java-tests/testData/compileServer/incremental/common/deleteInnerClass1/src/B.java

@@ -0,0 +1,5 @@
+public class B {
+  void foo () {
+    A.Inner.method();
+  }
+}

jps/jps-builders/src/org/jetbrains/jps/builders/java/dependencyView/Mappings.java

@@ -1752,6 +1752,8 @@ public class Mappings {
           myPresent.appendDependents(c, state.myDependants);
           debug("Adding usages of class ", c.name);
           state.myAffectedUsages.add(c.createUsage());
+          debug("Affecting usages of removed class ", c.name);
+          affectAll(c.name, myAffectedFiles, myFilter);
         }
       }
       debug("End of removed classes processing.");

jps/jps-builders/testSrc/org/jetbrains/ether/CommonTest.java

@@ -50,7 +50,7 @@ public class CommonTest extends IncrementalTestCase {
 
   public void testDeleteClassAfterCompileErrors() throws Exception {
     setupInitialProject();
-  
+
     doTestBuild(2);
   }
 
@@ -91,18 +91,26 @@ public class CommonTest extends IncrementalTestCase {
   }
 
   public void testDeleteClass3() throws Exception {
-      doTest();
+    doTest();
   }
 
   public void testDeleteClass4() throws Exception {
-      doTest();
+    doTest();
+  }
+
+  public void testDeleteInnerClass() throws Exception {
+    doTest();
+  }
+
+  public void testDeleteInnerClass1() throws Exception {
+    doTest();
   }
 
   public void testAddClass() throws Exception {
-      doTest();
+    doTest();
   }
 
   public void testAddDuplicateClass() throws Exception {
-      doTest();
+    doTest();
   }
 }