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

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

Issue 605443002: The default value for NullableString should be null (was "") (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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/public/js/bindings/codec_unittests.js
diff --git a/mojo/public/js/bindings/codec_unittests.js b/mojo/public/js/bindings/codec_unittests.js
index 80f1402eef2ac0c7df9f346f1f80ba17fe7fddbe..0276db21bae8c00cfa9c745b776fd5f28d691f67 100644
--- a/mojo/public/js/bindings/codec_unittests.js
+++ b/mojo/public/js/bindings/codec_unittests.js
@@ -188,6 +188,7 @@ define([
expect(result).toEqual(expectedResult);
}
encodeDecode(codec.String, "banana", "banana", 24);
+ encodeDecode(codec.NullableString, null, null, 8);
encodeDecode(codec.Int8, -1, -1);
encodeDecode(codec.Int8, 0xff, -1);
encodeDecode(codec.Int16, -1, -1);

Powered by Google App Engine
This is Rietveld 408576698