Move mojom definitions of URL{Request,Response} into SDK

Move mojom definitions of URL{Request,Response} into SDK

This moves the definition of the mojom URLRequest and URLResponse
objects into the SDK at //mojo/public/interfaces/network/. These are
used directly by the shell and have a special place in the system. This
patch does the minimal amount of import/include updating to get tests
passing but doesn't fully update everything - in particular it
doesn't update all the C++ code that #includes url_loader.mojom.h for
the URLRequest or URLResponse definitions to #include the more specific
mojom.h files, or the equivalent in other languages. I'll do that in a
follow-up to minimize churn.

R=viettrungluu@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/46b709be65425f3d3852b48e1cb3ebd85b762919

[jamesr, 2015-05-09 00:01:00]

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

[jamesr, 2015-05-09 00:01:00]

Green on the bots!

[viettrungluu, 2015-05-09 00:13:09]

LGTM w/extra blank lines removed.

https://codereview.chromium.org/1129063008/diff/40001/mojo/public/interfaces/...
File mojo/public/interfaces/network/network_error.mojom (right):

https://codereview.chromium.org/1129063008/diff/40001/mojo/public/interfaces/...
mojo/public/interfaces/network/network_error.mojom:11: 
Why oh why?

https://codereview.chromium.org/1129063008/diff/40001/mojo/public/interfaces/...
File mojo/public/interfaces/network/url_response.mojom (right):

https://codereview.chromium.org/1129063008/diff/40001/mojo/public/interfaces/...
mojo/public/interfaces/network/url_response.mojom:42: 
My eyes!

[qsr, 2015-05-11 11:39:29]

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

[qsr, 2015-05-11 11:39:30]

https://codereview.chromium.org/1129063008/diff/40001/mojo/services/url_respo...
File
mojo/services/url_response_disk_cache/public/interfaces/url_response_disk_cache.mojom
(right):

https://codereview.chromium.org/1129063008/diff/40001/mojo/services/url_respo...
mojo/services/url_response_disk_cache/public/interfaces/url_response_disk_cache.mojom:8:
import "mojo/services/network/public/interfaces/url_loader.mojom";
This can go away.

[jamesr, 2015-05-21 21:30:45]

On 2015/05/11 11:39:30, qsr wrote:
>
https://codereview.chromium.org/1129063008/diff/40001/mojo/services/url_respo...
> File
>
mojo/services/url_response_disk_cache/public/interfaces/url_response_disk_cache.mojom
> (right):
> 
>
https://codereview.chromium.org/1129063008/diff/40001/mojo/services/url_respo...
>
mojo/services/url_response_disk_cache/public/interfaces/url_response_disk_cache.mojom:8:
> import "mojo/services/network/public/interfaces/url_loader.mojom";
> This can go away.

Thanks, done

I need to resolve some issues with the network service but will land this as
soon as that's sorted.

[jamesr, 2015-05-27 23:32:55]

I've updated this on top of the monet update.  I'll land this which requires
that the HttpHeader and URL{Request,Response} struct definitions match exactly
what the network service is compiled against, which is currently true in
practice.  Then I'll roll the Mojo SDK into the monet repo and rebuild from
there using the SDK's version of these structs, then start fixing up the rest of
the mojoms.

[jamesr, 2015-05-28 00:07:56]

Committed patchset #4 (id:60001) manually as
46b709be65425f3d3852b48e1cb3ebd85b762919 (presubmit successful).