Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(565)

Unified Diff: mojo/dart/apptests/dart_apptests/lib/src/service_describer_apptests.dart

Issue 1753013002: Mojom runtime type info: New implementation for Dart. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/dart/apptests/dart_apptests/lib/src/service_describer_apptests.dart
diff --git a/mojo/dart/apptests/dart_apptests/lib/src/service_describer_apptests.dart b/mojo/dart/apptests/dart_apptests/lib/src/service_describer_apptests.dart
index ccafe587a1ebc3b73dfd978727c9a250073a79f9..7eb5a4b6e3131c65f3650e13dd58652c14507be1 100644
--- a/mojo/dart/apptests/dart_apptests/lib/src/service_describer_apptests.dart
+++ b/mojo/dart/apptests/dart_apptests/lib/src/service_describer_apptests.dart
@@ -48,14 +48,6 @@ tests(Application application, String url) {
mojom_types.MojomInterface b = ed.getTopLevelInterface(identity);
_compare(a, b);
- String interfaceID = "echo_service_EchoService__";
- mojom_types.MojomInterface c =
- (await sd.getTypeDefinition(interfaceID)).type.interfaceType;
- mojom_types.MojomInterface d =
- ed.getTypeDefinition(interfaceID, identity).interfaceType;
- _compare(a, c);
- _compare(c, d);
alexfandrianto 2016/03/02 05:27:12 Does this part of the test need to be removed? If
rudominer 2016/03/03 01:09:36 OK here is what I did: I switched this test to que
-
// Check that the mojom type definitions match between mappings.
// For simplicity, check in a shallow manner.
var actualDescriptions = (await sd.getAllTypeDefinitions()).definitions;

Powered by Google App Engine
This is Rietveld 408576698