Chromium Code Reviews| Index: pkg/analyzer/test/instrumentation/instrumentation_test.dart | 
| diff --git a/pkg/analyzer/test/instrumentation/instrumentation_test.dart b/pkg/analyzer/test/instrumentation/instrumentation_test.dart | 
| index 53a81914b72f8024fd4c9c7a9ed1a15d1e9c4101..de520de7fff63a21b8dbcf48e190e99ee2c14180 100644 | 
| --- a/pkg/analyzer/test/instrumentation/instrumentation_test.dart | 
| +++ b/pkg/analyzer/test/instrumentation/instrumentation_test.dart | 
| @@ -16,8 +16,8 @@ main() { | 
| }); | 
| } | 
| -@ReflectiveTestCase() | 
| -class InstrumentationServiceTest extends ReflectiveTestCase { | 
| +@reflectiveTest | 
| +class InstrumentationServiceTest extends ReflectiveTest { | 
| 
 
Brian Wilkerson
2015/01/13 17:06:22
This looks wrong. Why would we ever subclass the a
 
scheglov
2015/01/13 17:10:33
Good question.
It shouldn't.
Removed.
 
 | 
| void assertNormal(TestInstrumentationServer server, String tag, | 
| String message) { | 
| String sent = server.normalChannel.toString(); | 
| @@ -107,8 +107,8 @@ class InstrumentationServiceTest extends ReflectiveTestCase { | 
| } | 
| } | 
| -@ReflectiveTestCase() | 
| -class MulticastInstrumentationServerTest extends ReflectiveTestCase { | 
| +@reflectiveTest | 
| +class MulticastInstrumentationServerTest extends ReflectiveTest { | 
| TestInstrumentationServer serverA = new TestInstrumentationServer(); | 
| TestInstrumentationServer serverB = new TestInstrumentationServer(); | 
| MulticastInstrumentationServer server; |