Make struct members nullable so demo_launcher no longer crashes.

Make struct members nullable so demo_launcher no longer crashes.

This change fixes a few mojom declarations that were causing the
mojo_demo_launcher to crash when fetching a simple Apache "It works!"
web page.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=291029

[darin (slow to review), 2014-08-20 20:26:48]

LGTM, thanks!

[acolwell GONE FROM CHROMIUM, 2014-08-20 20:32:58]

The CQ bit was checked by acolwell@chromium.org

[yzshen1, 2014-08-20 20:34:15]

Thanks! LGTM

One nit

https://codereview.chromium.org/488263002/diff/1/mojo/services/public/interfa...
File mojo/services/public/interfaces/network/url_loader.mojom (right):

https://codereview.chromium.org/488263002/diff/1/mojo/services/public/interfa...
mojo/services/public/interfaces/network/url_loader.mojom:79: // set.
The comment sounds like this field can be null, right?

[commit-bot: I haz the power, 2014-08-20 20:34:17]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/acolwell@chromium.org/488263002/1

[acolwell GONE FROM CHROMIUM, 2014-08-20 20:36:30]

The CQ bit was unchecked by acolwell@chromium.org

[acolwell GONE FROM CHROMIUM, 2014-08-20 20:38:11]

https://codereview.chromium.org/488263002/diff/1/mojo/services/public/interfa...
File mojo/services/public/interfaces/network/url_loader.mojom (right):

https://codereview.chromium.org/488263002/diff/1/mojo/services/public/interfa...
mojo/services/public/interfaces/network/url_loader.mojom:79: // set.
On 2014/08/20 20:34:15, yzshen1 wrote:
> The comment sounds like this field can be null, right?
Yes. I just fixed this one as well.

[acolwell GONE FROM CHROMIUM, 2014-08-20 20:39:06]

The CQ bit was checked by acolwell@chromium.org

[commit-bot: I haz the power, 2014-08-20 20:41:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/acolwell@chromium.org/488263002/20001

[commit-bot: I haz the power, 2014-08-20 22:50:14]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  android_clang_dbg on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-20 22:54:23]

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

[commit-bot: I haz the power, 2014-08-20 22:54:25]

Try jobs failed on following builders:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  android_clang_dbg on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)

[yzshen1, 2014-08-21 04:16:58]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2014-08-21 04:21:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/acolwell@chromium.org/488263002/20001

[commit-bot: I haz the power, 2014-08-21 09:39:19]

Committed patchset #2 (20001) as 291029