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

Unified Diff: mojo/public/js/bindings/validation_unittests.js

Issue 488173006: Integrate Mojo JS validation bindings with the Router (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed a template indentO Created 6 years, 4 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/public/js/bindings/validation_unittests.js
diff --git a/mojo/public/js/bindings/validation_unittests.js b/mojo/public/js/bindings/validation_unittests.js
index be07ac97a08121c014be3f77b51accecadd7519e..ca275ad8c68b63e6d66454bd00c522019fd1bfa1 100644
--- a/mojo/public/js/bindings/validation_unittests.js
+++ b/mojo/public/js/bindings/validation_unittests.js
@@ -226,20 +226,18 @@ define([
}
function testConformanceMessageValidation() {
- testMessageValidation("conformance_", [
- testInterface.validateConformanceTestInterfaceRequest,
- ]);
+ testMessageValidation("conformance_",
+ testInterface.ConformanceTestInterfaceStub.prototype.validators);
}
function testIntegrationMessageValidation() {
- testMessageValidation("integration_", [
- testInterface.validateIntegrationTestInterface1Request,
- testInterface.validateIntegrationTestInterface2Response
- ]);
+ testMessageValidation("integration_",
+ testInterface.IntegrationTestInterface1Stub.prototype.validators.concat(
+ testInterface.IntegrationTestInterface2Proxy.prototype.validators));
}
+ expect(checkTestMessageParser()).toBeNull();
testConformanceMessageValidation();
testIntegrationMessageValidation();
- expect(checkTestMessageParser()).toBeNull();
this.result = "PASS";
});

Powered by Google App Engine
This is Rietveld 408576698