Review nullability in the gpu and geometry interfaces.

mojo/services/gpu/public/interfaces/gpu.mojom

Index: mojo/services/gpu/public/interfaces/gpu.mojom
diff --git a/mojo/services/gpu/public/interfaces/gpu.mojom b/mojo/services/gpu/public/interfaces/gpu.mojom
index 4b74035ac6589adcfdb732a7465ff774687f9bd4..4e0ef5137350c469590ffd12fb927ad75bbf6bcc 100644
--- a/mojo/services/gpu/public/interfaces/gpu.mojom
+++ b/mojo/services/gpu/public/interfaces/gpu.mojom
@@ -10,8 +10,8 @@ import "mojo/services/gpu/public/interfaces/viewport_parameter_listener.mojom";
 
 interface Gpu {
   CreateOnscreenGLES2Context(uint64 native_viewport_id,
-                             Size? size,
-                             CommandBuffer&? gles2_client,
-                             ViewportParameterListener? listener);
-  CreateOffscreenGLES2Context(CommandBuffer&? gles2_client);
+                             Size size,
+                             CommandBuffer& gles2_client,
+                             ViewportParameterListener listener);

[jamesr, 2015/01/08 23:16:29]

this broke mojo:sample_app. This field is valid to be null

+  CreateOffscreenGLES2Context(CommandBuffer& gles2_client);
 };