Adds support for new-tab targeted loads initiated from the renderer.

Adds support for new-tab targeted loads initiated from the renderer.

- My WCD impl in navigation::ViewImpl didn't provide an impl of "WCD::OpenURLInTab".
- Fix some crashes in browser.

R=jam@chromium.org

Committed: https://crrev.com/a6e9b7f997ca56310ec4ca851069b8e1a606c682
Cr-Commit-Position: refs/heads/master@{#399340}

[Ben Goodger (Google), 2016-06-10 21:11:44]

Description was changed from

==========
Fix some crashes in browser

BUG=
==========

to

==========
Adds support for new-tab targeted loads initiated from the renderer.

- My WCD impl in navigation::ViewImpl didn't provide an impl of
"WCD::OpenURLInTab".
- Fix some crashes in browser.

R=jam@chromium.org
==========

[Ben Goodger (Google), 2016-06-10 21:11:44]

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

[jam, 2016-06-10 23:19:10]

lgtm

https://codereview.chromium.org/2057023002/diff/60001/mash/browser/browser.cc
File mash/browser/browser.cc (right):

https://codereview.chromium.org/2057023002/diff/60001/mash/browser/browser.cc...
mash/browser/browser.cc:737: case
navigation::mojom::WindowOpenDisposition::CURRENT_TAB:
nit:indentation is off

[Ben Goodger (Google), 2016-06-10 23:21:55]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-06-10 23:21:56]

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

[commit-bot: I haz the power, 2016-06-10 23:22:52]

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

[commit-bot: I haz the power, 2016-06-11 00:58:56]

Description was changed from

==========
Adds support for new-tab targeted loads initiated from the renderer.

- My WCD impl in navigation::ViewImpl didn't provide an impl of
"WCD::OpenURLInTab".
- Fix some crashes in browser.

R=jam@chromium.org
==========

to

==========
Adds support for new-tab targeted loads initiated from the renderer.

- My WCD impl in navigation::ViewImpl didn't provide an impl of
"WCD::OpenURLInTab".
- Fix some crashes in browser.

R=jam@chromium.org
==========

[commit-bot: I haz the power, 2016-06-11 00:58:57]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-06-11 00:59:08]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-11 01:00:32]

Description was changed from

==========
Adds support for new-tab targeted loads initiated from the renderer.

- My WCD impl in navigation::ViewImpl didn't provide an impl of
"WCD::OpenURLInTab".
- Fix some crashes in browser.

R=jam@chromium.org
==========

to

==========
Adds support for new-tab targeted loads initiated from the renderer.

- My WCD impl in navigation::ViewImpl didn't provide an impl of
"WCD::OpenURLInTab".
- Fix some crashes in browser.

R=jam@chromium.org

Committed: https://crrev.com/a6e9b7f997ca56310ec4ca851069b8e1a606c682
Cr-Commit-Position: refs/heads/master@{#399340}
==========

[commit-bot: I haz the power, 2016-06-11 01:00:33]

Patchset 5 (id:??) landed as
https://crrev.com/a6e9b7f997ca56310ec4ca851069b8e1a606c682
Cr-Commit-Position: refs/heads/master@{#399340}