Mojo: Set a default value for the --origin command line switch.

Mojo: Set a default value for the --origin command line switch.

(Also fixes a regression in the wget app.)

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

[darin (slow to review), 2014-07-23 03:27:11]

Tim, I know this CL conflicts with https://codereview.chromium.org/397733003/,
and I'm happy to rebase this change after you land yours.

[tim (not reviewing), 2014-07-23 22:22:32]

LGTM, it wouldn't be the end of the world if I had to rebase on this :)  I'm
just waiting for jamesr's InterfaceFactory to go in and then land mine on top.

https://codereview.chromium.org/414633003/diff/1/mojo/shell/mojo_url_resolver.cc
File mojo/shell/mojo_url_resolver.cc (right):

https://codereview.chromium.org/414633003/diff/1/mojo/shell/mojo_url_resolver...
mojo/shell/mojo_url_resolver.cc:40: origin_ =
net::FilePathToFileURL(path).spec();
As we were talking about, probably makes sense to unify the local_file_set code
with this, since it is also providing the same "default value" for local files. 
Then, local_file_set would be there to force looking in DIR_MODULE for
registered mojo: urls, even if a BaseURL is set on the resolver.

[tim (not reviewing), 2014-07-23 22:23:53]

On 2014/07/23 22:22:32, timsteele wrote:
> LGTM, it wouldn't be the end of the world if I had to rebase on this :)  I'm
> just waiting for jamesr's InterfaceFactory to go in and then land mine on top.
> 
>
https://codereview.chromium.org/414633003/diff/1/mojo/shell/mojo_url_resolver.cc
> File mojo/shell/mojo_url_resolver.cc (right):
> 
>
https://codereview.chromium.org/414633003/diff/1/mojo/shell/mojo_url_resolver...
> mojo/shell/mojo_url_resolver.cc:40: origin_ =
> net::FilePathToFileURL(path).spec();
> As we were talking about, probably makes sense to unify the local_file_set
code
> with this, since it is also providing the same "default value" for local
files. 
> Then, local_file_set would be there to force looking in DIR_MODULE for
> registered mojo: urls, even if a BaseURL is set on the resolver.

er, to be clear, my LGTM was assuming we'd do this as part of this CL or in a
follow up.

[darin (slow to review), 2014-07-25 17:12:59]

@tim - PTAL

This version is like what we discussed.

[tim (not reviewing), 2014-07-25 17:28:51]

LGTM

[darin (slow to review), 2014-07-25 17:45:38]

The CQ bit was checked by darin@chromium.org

[commit-bot: I haz the power, 2014-07-25 17:47:47]

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

[commit-bot: I haz the power, 2014-07-25 20:41:57]

Change committed as 285668