The default value for NullableString should be null (was "")

mojo/public/interfaces/bindings/tests/test_structs.mojom

Index: mojo/public/interfaces/bindings/tests/test_structs.mojom
diff --git a/mojo/public/interfaces/bindings/tests/test_structs.mojom b/mojo/public/interfaces/bindings/tests/test_structs.mojom
index 66cd2e965efe8c3922c6484715bbc3cec468af0a..1237ee56e45d7e2f0bc3df19b9d8b241e6c671f4 100644
--- a/mojo/public/interfaces/bindings/tests/test_structs.mojom
+++ b/mojo/public/interfaces/bindings/tests/test_structs.mojom
@@ -20,4 +20,51 @@ struct RectPair {
 struct EmptyStruct {
 };
 
+struct DefaultFieldValues {
+  bool f0;
+  int8 f1;
+  uint8 f2;
+  int16 f3;

[yzshen1, 2014/09/24 22:49:25]

uint16 :)

[hansmuller, 2014/09/24 23:33:11]

Done.

+  int32 f4;
+  uint32 f5;
+  int64 f6;
+  uint64 f7;
+  float f8;
+  double f9;
+  string f10;
+  string? f11;
+  handle<message_pipe> f12;
+  handle<data_pipe_consumer> f13;
+  handle<data_pipe_producer> f14;
+  handle<message_pipe>? f15;
+  handle<data_pipe_consumer>? f16;
+  handle<data_pipe_producer>? f17;

[yzshen1, 2014/09/24 22:49:25]

(optional; if you want to)

handle, handle?, handle<shared_buffer>, handle<shared_buffer>?

[hansmuller, 2014/09/24 23:33:11]

I'd wanted to include all of the types, so yes. Thanks for pointing those out.

+  string[] f18;
+  string?[] f19;
+  string[]? f20;
+  string?[]? f21;
+  EmptyStruct f22;
+  EmptyStruct? f23;
+};
+
+struct ExplicitFieldValues {
+  const string kFoo = "foo";
+  bool f0 = true;
+  int8 f1 = 100;
+  uint8 f2 = 100;
+  int16 f3 = 100;

[yzshen1, 2014/09/24 22:49:25]

uint16 :)

[hansmuller, 2014/09/24 23:33:11]

Done.

+  int32 f4 = 100;
+  uint32 f5 = 100;
+  int64 f6 = 100;
+  uint64 f7 = 100;
+  float f8 = 100;
+  float f9 = 100.0;
+  double f10 = 100;
+  double f11 = 100.0;
+  string f12 = kFoo;
+  string? f13 = kFoo;
+  Rect f14 = default;
+  Rect? f15 = default;
+};
+
 }