New issue
Advanced search Search tips

Issue 619791 link

Starred by 2 users
Status: Fixed
Owner:
dpa...@chromium.org
Closed: Jul 2016
Cc:
tbuck...@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Feature



Sign in to add a comment

Search Engine Lists: Clicking "Manage" for search engine extension overwrites current tab

Project Member Reported by mgreenwald@google.com, Jun 14 2016 
To repeat:
(1) Have a search engine extension
(2) in your search engine lists scroll to bottom and click "Manage"
(3) will open in same tab instead opening in new tab

Please make it open in a new tab
Project Member

Comment 1 by sheriffbot@chromium.org, Jun 14 2016

Labels: Hotlist-Google

Comment 2 by dpa...@chromium.org, Jul 12 2016

Owner: tbuck...@chromium.org
Status: Untriaged (was: Available)
@tbuckley: Could you triage this bug? Current behavior is consistent across old Options and new Settings.

Comment 3 by mgreenwald@google.com, Jul 12 2016

Labels: -Type-Bug Type-Feature
Changing to feature request unless Tom thinks otherwise

Comment 4 by tbuck...@chromium.org, Jul 12 2016

Owner: dpa...@chromium.org
Status: Assigned (was: Untriaged)
Yep, regardless of the old behavior this should open in a new tab instead of overriding the current one.

Comment 5 by dpa...@chromium.org, Jul 12 2016

Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Jul 13 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/157635fc46d91566cff9cc44575d732936a9622e

commit 157635fc46d91566cff9cc44575d732936a9622e
Author: dpapad <dpapad@chromium.org>
Date: Wed Jul 13 00:17:50 2016

MD Settings: Search engines page, open "manage extension" in new tab.

BUG= 619791 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2145643005
Cr-Commit-Position: refs/heads/master@{#404863}

[modify] https://crrev.com/157635fc46d91566cff9cc44575d732936a9622e/chrome/browser/resources/settings/search_engines_page/search_engines_browser_proxy.js

Comment 7 by dpa...@chromium.org, Jul 13 2016

Status: Fixed (was: Started)
►

Sign in to add a comment