Change mojo url to always use mojo:foo instead of mojo://foo

Change mojo url to always use mojo:foo instead of mojo://foo

R=abarth@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/de93437e5974558a19a12e4763d2a14a5ef27e0f

[qsr, 2014-11-05 16:54:12]

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

[Aaron Boodman, 2014-11-05 17:07:21]

Calls to AddStandardScheme need to be changed too. Otherwise, GURL will convert
foo:bar to foo://bar.

https://github.com/domokit/mojo/search?utf8=%E2%9C%93&q=AddStandardScheme

Also note that this is entirely decided yet, so we might change it yet again.
But consistent one way or the other is better for now.

[qsr, 2014-11-05 18:13:18]

On 2014/11/05 17:07:21, Aaron Boodman wrote:
> Calls to AddStandardScheme need to be changed too. Otherwise, GURL will
convert
> foo:bar to foo://bar.

 Done

> Also note that this is entirely decided yet, so we might change it yet again.
> But consistent one way or the other is better for now.

 I agree that it is not decided, but having 2 different convention makes using
url mapping quite hard. I choose a convention to have one, and as you say we
will be able to change if we decide to.

[abarth-chromium, 2014-11-05 18:15:01]

https://codereview.chromium.org/701933004/diff/20001/examples/window_manager/...
File examples/window_manager/debug_panel.cc (right):

https://codereview.chromium.org/701933004/diff/20001/examples/window_manager/...
examples/window_manager/debug_panel.cc:123: Navigate("mojo:embedded_app/");
Why the trailing / ?

https://codereview.chromium.org/701933004/diff/20001/sky/framework/xmlhttpreq...
File sky/framework/xmlhttprequest.sky (right):

https://codereview.chromium.org/701933004/diff/20001/sky/framework/xmlhttpreq...
sky/framework/xmlhttprequest.sky:9: "mojo:network_service/",
net.NetworkService);
Why the trailing / ?

[qsr, 2014-11-05 18:22:21]

https://codereview.chromium.org/701933004/diff/20001/examples/window_manager/...
File examples/window_manager/debug_panel.cc (right):

https://codereview.chromium.org/701933004/diff/20001/examples/window_manager/...
examples/window_manager/debug_panel.cc:123: Navigate("mojo:embedded_app/");
On 2014/11/05 18:15:01, abarth wrote:
> Why the trailing / ?

 Existing beforehands. Removed. Thanks.

https://codereview.chromium.org/701933004/diff/20001/sky/framework/xmlhttpreq...
File sky/framework/xmlhttprequest.sky (right):

https://codereview.chromium.org/701933004/diff/20001/sky/framework/xmlhttpreq...
sky/framework/xmlhttprequest.sky:9: "mojo:network_service/",
net.NetworkService);
On 2014/11/05 18:15:01, abarth wrote:
> Why the trailing / ?

Done.

[qsr, 2014-11-06 15:38:29]

gentle ping?

[abarth-chromium, 2014-11-06 16:10:24]

lgtm

[qsr, 2014-11-06 16:23:21]

Committed patchset #3 (id:40001) manually as
de93437e5974558a19a12e4763d2a14a5ef27e0f (presubmit successful).