[Media Router WebUI] Open links in a new tab.

[Media Router WebUI] Open links in a new tab.

Open all links from the dialog (JS side) in a new tab. This prevents navigating the user away from whatever they're currently doing on the current active tab. The dialog remains open on the original tab.

URLs that are opened on the C++ side already open in a new tab.

BUG=594653
Committed: https://crrev.com/55669d8f6ed1ad73e0991c0285a1e3947d3e785e
Cr-Commit-Position: refs/heads/master@{#381822}

[apacible, 2016-03-15 21:27:35]

Description was changed from

==========
asdf

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

to

==========
[Media Router WebUI] Open links in a new tab.

Open all links from the dialog (JS side) in a new tab. This prevents navigating
the user away from whatever they're currently doing on the current active tab.

URLs that are opened on the C++ side already open in a new tab.

BUG=594653
==========

[apacible, 2016-03-15 21:27:35]

apacible@chromium.org changed reviewers:
+ imcheng@chromium.org

[apacible, 2016-03-15 21:28:58]

Description was changed from

==========
[Media Router WebUI] Open links in a new tab.

Open all links from the dialog (JS side) in a new tab. This prevents navigating
the user away from whatever they're currently doing on the current active tab.

URLs that are opened on the C++ side already open in a new tab.

BUG=594653
==========

to

==========
[Media Router WebUI] Open links in a new tab.

Open all links from the dialog (JS side) in a new tab. This prevents navigating
the user away from whatever they're currently doing on the current active tab.
The dialog remains open on the original tab.

URLs that are opened on the C++ side already open in a new tab.

BUG=594653
==========

[apacible, 2016-03-15 21:29:14]

PTAL, thanks!

Tested on mac and linux.

[imcheng, 2016-03-17 18:36:38]

lgtm

[apacible, 2016-03-17 22:09:15]

The CQ bit was checked by apacible@chromium.org

[apacible, 2016-03-17 22:09:16]

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

[commit-bot: I haz the power, 2016-03-17 22:09:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1809463002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1809463002/20001

[commit-bot: I haz the power, 2016-03-17 22:52:50]

Description was changed from

==========
[Media Router WebUI] Open links in a new tab.

Open all links from the dialog (JS side) in a new tab. This prevents navigating
the user away from whatever they're currently doing on the current active tab.
The dialog remains open on the original tab.

URLs that are opened on the C++ side already open in a new tab.

BUG=594653
==========

to

==========
[Media Router WebUI] Open links in a new tab.

Open all links from the dialog (JS side) in a new tab. This prevents navigating
the user away from whatever they're currently doing on the current active tab.
The dialog remains open on the original tab.

URLs that are opened on the C++ side already open in a new tab.

BUG=594653
==========

[commit-bot: I haz the power, 2016-03-17 22:52:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-17 22:53:55]

Description was changed from

==========
[Media Router WebUI] Open links in a new tab.

Open all links from the dialog (JS side) in a new tab. This prevents navigating
the user away from whatever they're currently doing on the current active tab.
The dialog remains open on the original tab.

URLs that are opened on the C++ side already open in a new tab.

BUG=594653
==========

to

==========
[Media Router WebUI] Open links in a new tab.

Open all links from the dialog (JS side) in a new tab. This prevents navigating
the user away from whatever they're currently doing on the current active tab.
The dialog remains open on the original tab.

URLs that are opened on the C++ side already open in a new tab.

BUG=594653

Committed: https://crrev.com/55669d8f6ed1ad73e0991c0285a1e3947d3e785e
Cr-Commit-Position: refs/heads/master@{#381822}
==========

[commit-bot: I haz the power, 2016-03-17 22:53:56]

Patchset 2 (id:??) landed as
https://crrev.com/55669d8f6ed1ad73e0991c0285a1e3947d3e785e
Cr-Commit-Position: refs/heads/master@{#381822}