Don't show navigation menu items when URI isnt accepted

Don't show navigation menu items when URI isnt accepted

Don't show "open in new window", "open in new tab", or "open in incognito tab" if URI isn't accepted.
BUG=618700
Committed: https://crrev.com/fdd83683711c3bb501a693e9fd1281c794dee694
Cr-Commit-Position: refs/heads/master@{#399005}

[amaralp, 2016-06-09 19:12:36]

Description was changed from

==========
Don't show navigation menu items when URI isnt accepted

Don't show "open in new window", "open in new tab", or "open in incognito tab"
if URI isn't accepted.
BUG=618700
==========

to

==========
Don't show navigation menu items when URI isnt accepted

Don't show "open in new window", "open in new tab", or "open in incognito tab"
if URI isn't accepted.
BUG=618700
==========

[amaralp, 2016-06-09 19:12:36]

amaralp@chromium.org changed reviewers:
+ aelias@chromium.org

[amaralp, 2016-06-09 19:15:59]

On 2016/06/09 19:12:36, amaralp wrote:
> mailto:amaralp@chromium.org changed reviewers:
> + mailto:aelias@chromium.org

PTAL, I'm not sure if isAcceptedScheme is the correct way to check if we should
allow opening in a new tab.

[aelias_OOO_until_Jul13, 2016-06-09 20:28:10]

aelias@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[aelias_OOO_until_Jul13, 2016-06-09 20:28:23]

lgtm, adding dfalcantara@ for OWNERS review.

[gone, 2016-06-09 20:36:15]

lgtm

[amaralp, 2016-06-09 20:37:43]

The CQ bit was checked by amaralp@chromium.org

[commit-bot: I haz the power, 2016-06-09 20:38:17]

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

[commit-bot: I haz the power, 2016-06-09 20:48:32]

Description was changed from

==========
Don't show navigation menu items when URI isnt accepted

Don't show "open in new window", "open in new tab", or "open in incognito tab"
if URI isn't accepted.
BUG=618700
==========

to

==========
Don't show navigation menu items when URI isnt accepted

Don't show "open in new window", "open in new tab", or "open in incognito tab"
if URI isn't accepted.
BUG=618700
==========

[commit-bot: I haz the power, 2016-06-09 20:48:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-09 20:51:20]

Description was changed from

==========
Don't show navigation menu items when URI isnt accepted

Don't show "open in new window", "open in new tab", or "open in incognito tab"
if URI isn't accepted.
BUG=618700
==========

to

==========
Don't show navigation menu items when URI isnt accepted

Don't show "open in new window", "open in new tab", or "open in incognito tab"
if URI isn't accepted.
BUG=618700

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

[commit-bot: I haz the power, 2016-06-09 20:51:21]

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