Completion should not show overridden inherited methods

pkg/analysis_server/test/services/completion/imported_computer_test.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library test.services.completion.toplevel;
 
 import 'package:analysis_server/src/protocol.dart' as protocol
     show Element, ElementKind;
 import 'package:analysis_server/src/protocol.dart' hide Element, ElementKind;
 import 'package:analysis_server/src/services/completion/completion_manager.dart';
@@ skipping 575 matching lines @@
       expect(suggestion.parameterNames[0], 'x');
       expect(suggestion.parameterTypes[0], 'dynamic');
       expect(suggestion.parameterNames[1], 'y');
       expect(suggestion.parameterTypes[1], 'int');
       expect(suggestion.requiredParameterCount, 2);
       expect(suggestion.hasNamedParameters, false);
     });
   }
 
   test_mixin_ordering() {
-    // TODO(paulberry): The mixins are visited in the correct order, so we see
-    // M2.m() before M1.m(), as we should.  But the second (shadowed) result
-    // isn't being thrown out as it should.
     addSource('/libA.dart', '''
 class B {}
 class M1 {
   void m() {}
 }
 class M2 {
   void m() {}
 }
 ''');
     addTestSource('''
@@ skipping 118 matching lines @@
   }
 
   @override
   test_partFile_TypeName2() {
     return super.test_partFile_TypeName2().then((_) {
       expect(request.cache.importKey,
           'library libA;import "/testB.dart";part "/testA.dart";');
     });
   }
 }