Review nullability in the gpu and geometry interfaces.

Review nullability in the gpu and geometry interfaces.

BUG=407680
R=jamesr@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/94eab58c3258e1238d4c9b97df5f209e91386848

[Elliot Glaysher, 2015-01-06 21:36:32]

erg@chromium.org changed reviewers:
+ jamesr@chromium.org

[Elliot Glaysher, 2015-01-06 21:36:33]

Prompted by our conversation earlier over coffee.

[jamesr, 2015-01-06 21:39:44]

lgtm

thanks, none of these can be usefully null

[Elliot Glaysher, 2015-01-06 21:46:50]

The CQ bit was checked by erg@chromium.org

[commit-bot: I haz the power, 2015-01-06 21:46:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-01-06 21:46:59]

Commit queue rejected this change because it did not recognize the base URL.
Please commit your change manually.

[Elliot Glaysher, 2015-01-06 21:52:54]

Committed patchset #1 (id:1) manually as
94eab58c3258e1238d4c9b97df5f209e91386848 (presubmit successful).

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

https://codereview.chromium.org/816913004/diff/1/mojo/services/gpu/public/int...
File mojo/services/gpu/public/interfaces/gpu.mojom (right):

https://codereview.chromium.org/816913004/diff/1/mojo/services/gpu/public/int...
mojo/services/gpu/public/interfaces/gpu.mojom:15: ViewportParameterListener
listener);
this broke mojo:sample_app. This field is valid to be null

[blundell, 2015-01-09 14:43:43]

blundell@chromium.org changed reviewers:
+ blundell@chromium.org

[blundell, 2015-01-09 14:43:43]

lgtm

[blundell, 2015-01-09 14:46:00]

On 2015/01/09 14:43:43, blundell wrote:
> lgtm

oops, that was meant for a different cl (exercise for the reader: guess which ;)