diff --git a/java/java-tests/testSrc/com/intellij/codeInsight/daemon/ImplementationsViewTest.java b/java/java-tests/testSrc/com/intellij/codeInsight/daemon/ImplementationsViewTest.java index 6e3b0ccdff82..ecb14768b153 100644 --- a/java/java-tests/testSrc/com/intellij/codeInsight/daemon/ImplementationsViewTest.java +++ b/java/java-tests/testSrc/com/intellij/codeInsight/daemon/ImplementationsViewTest.java @@ -115,6 +115,9 @@ public class ImplementationsViewTest extends LightCodeInsightFixtureTestCase { new ImplementationViewComponent(all.toArray(new PsiElement[all.size()]), 0); try { final String[] visibleFiles = component.getVisibleFiles(); + assertTrue(visibleFiles.length > 0); + assertEquals(visibleFiles[0], "a.java (AFoo)"); + Arrays.sort(visibleFiles); Assert.assertArrayEquals(Arrays.toString(visibleFiles), new String[]{"a.java (AFoo)", "a.java (AFoo1 in AFoo)", "a.java (AFoo2 in AFoo)", "a.java (AFoo3 in AFoo)"}, visibleFiles); } diff --git a/platform/lang-impl/src/com/intellij/codeInsight/hint/ImplementationViewComponent.java b/platform/lang-impl/src/com/intellij/codeInsight/hint/ImplementationViewComponent.java index 867e4ac0b065..b83d832f8348 100644 --- a/platform/lang-impl/src/com/intellij/codeInsight/hint/ImplementationViewComponent.java +++ b/platform/lang-impl/src/com/intellij/codeInsight/hint/ImplementationViewComponent.java @@ -330,15 +330,6 @@ public class ImplementationViewComponent extends JPanel { names.add(((PsiNamedElement)element).getName()); } } - Arrays.sort(elements, new Comparator() { - @Override - public int compare(PsiElement e1, PsiElement e2) { - if (e1 instanceof PsiNamedElement && e2 instanceof PsiNamedElement) { - return Comparing.compare(((PsiNamedElement)e1).getName(), ((PsiNamedElement)e2).getName()); - } - return e1.hashCode() - e2.hashCode(); - } - }); for (PsiElement element : elements) { PsiFile file = getContainingFile(element); if (file == null) continue;