Move navigations with POST or referrer to the shell.

Move navigations with POST or referrer to the shell.

Previously we were doing them inside html_viewer because NavigatorHost was throwing out that extra data before it got to the network service. The fix is to pass through this url request information.

Also:
-add support for netlog to make debugging network issues easier. Use --log-net-log=file_path
-ensure that mojo shell uses the same user agent as html_viewer. since this UA is sent to websites, they need to match or else there are bugs with Gmail (since it sees different UAs when logging in)

BUG=481153
Committed: https://crrev.com/3cf73ae56fd339f5ce180ab39a0f6dd3e5aa4b1d
Cr-Commit-Position: refs/heads/master@{#330864}

[jam, 2015-05-02 23:27:14]

jam@chromium.org changed reviewers:
+ ben@chromium.org

[jam, 2015-05-04 05:56:02]

Patchset #4 (id:60001) has been deleted

[jam, 2015-05-04 16:19:11]

Patchset #4 (id:80001) has been deleted

[jam, 2015-05-04 16:19:18]

Patchset #3 (id:40001) has been deleted

[jam, 2015-05-11 05:42:25]

Patchset #4 (id:120001) has been deleted

[jam, 2015-05-20 18:30:40]

Patchset #5 (id:160001) has been deleted

[jam, 2015-05-20 19:21:54]

Patchset #5 (id:180001) has been deleted

[jam, 2015-05-20 19:57:31]

Patchset #5 (id:200001) has been deleted

[jam, 2015-05-20 20:26:26]

PTAL, this is now ready to review after the interfaces were forked.

[Ben Goodger (Google), 2015-05-20 21:17:06]

lgtm

https://codereview.chromium.org/1121783003/diff/260001/mojo/shell/shell_impl.cc
File mojo/shell/shell_impl.cc (right):

https://codereview.chromium.org/1121783003/diff/260001/mojo/shell/shell_impl....
mojo/shell/shell_impl.cc:45: void
ShellImpl::ConnectToApplication(mojo::URLRequestPtr app_url,
nit: app_request

[jam, 2015-05-20 22:02:17]

The CQ bit was checked by jam@chromium.org

[jam, 2015-05-20 22:02:18]

The patchset sent to the CQ was uploaded after l-g-t-m from ben@chromium.org
Link to the patchset: https://codereview.chromium.org/1121783003/#ps280001
(title: "review comments")

[commit-bot: I haz the power, 2015-05-20 22:04:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1121783003/280001

[commit-bot: I haz the power, 2015-05-20 22:17:44]

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

[commit-bot: I haz the power, 2015-05-20 22:17:46]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jam, 2015-05-20 23:02:32]

Patchset #9 (id:300001) has been deleted

[jam, 2015-05-20 23:02:46]

Patchset #9 (id:300001) has been deleted

[jam, 2015-05-21 00:41:09]

https://codereview.chromium.org/1121783003/diff/260001/mojo/shell/shell_impl.cc
File mojo/shell/shell_impl.cc (right):

https://codereview.chromium.org/1121783003/diff/260001/mojo/shell/shell_impl....
mojo/shell/shell_impl.cc:45: void
ShellImpl::ConnectToApplication(mojo::URLRequestPtr app_url,
On 2015/05/20 21:17:05, Ben Goodger (Google) wrote:
> nit: app_request

Done.

[jam, 2015-05-21 00:41:23]

The CQ bit was checked by jam@chromium.org

[jam, 2015-05-21 00:41:24]

The patchset sent to the CQ was uploaded after l-g-t-m from ben@chromium.org
Link to the patchset: https://codereview.chromium.org/1121783003/#ps320001
(title: "merge")

[commit-bot: I haz the power, 2015-05-21 00:41:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1121783003/320001

[commit-bot: I haz the power, 2015-05-21 00:48:23]

Committed patchset #9 (id:320001)

[commit-bot: I haz the power, 2015-05-21 00:49:13]

Patchset 9 (id:??) landed as
https://crrev.com/3cf73ae56fd339f5ce180ab39a0f6dd3e5aa4b1d
Cr-Commit-Position: refs/heads/master@{#330864}