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

Unified Diff: mojo/android/javatests/src/org/chromium/mojo/bindings/ValidationTest.java

Issue 2339413004: Allow Mojo structs as map keys (Closed)
Patch Set: Address sammc's comments and improve Blink tests Created 4 years, 3 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/android/javatests/src/org/chromium/mojo/bindings/ValidationTest.java
diff --git a/mojo/android/javatests/src/org/chromium/mojo/bindings/ValidationTest.java b/mojo/android/javatests/src/org/chromium/mojo/bindings/ValidationTest.java
index b62fe6da79ab176fddcd10c5edfe78c534a42958..d184448ddb6c8c385f544e6df2b7b2b6a62ad922 100644
--- a/mojo/android/javatests/src/org/chromium/mojo/bindings/ValidationTest.java
+++ b/mojo/android/javatests/src/org/chromium/mojo/bindings/ValidationTest.java
@@ -61,7 +61,11 @@ public class ValidationTest extends MojoTestCase {
return false;
}
// TODO(crbug/640298): Implement max recursion depth for Java.
- if (pathname.getName().startsWith("conformance_mthd19_good")) {
+ if (pathname.getName().startsWith("conformance_mthd19_exceed_recursion_limit")) {
+ return false;
+ }
+ // TODO(crbug/628104): Support struct map keys for Java.
+ if (pathname.getName().startsWith("conformance_mthd20_good")) {
return false;
}
return pathname.isFile() && pathname.getName().startsWith(mPrefix)

Powered by Google App Engine
This is Rietveld 408576698