Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(74)

Issue 218032: [Mac] Support option-return to open in new foreground tab.... (Closed)

Created:
11 years, 3 months ago by rohitrao (ping after 24h)
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com, John Grabowski, pam+watch_chromium.org, Ben Goodger (Google), tim (not reviewing), Paweł Hajdan Jr.
Visibility:
Public.

Description

[Mac] Support option-return to open in new foreground tab. BUG=http://crbug.com/22882 TEST=Type something into the omnibox. Option-return should open it in a new foreground tab. TEST=After using Option-return to open something in a new foreground tab, switch back to the original tab and verify that the url in the omnibox has reverted to the original page's url. TEST=Option-leftclick on a link in a webpage should still prompt you to "save as." Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27127

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M chrome/browser/autocomplete/autocomplete_edit_view_mac.mm View 1 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
rohitrao (ping after 24h)
11 years, 3 months ago (2009-09-24 18:29:39 UTC) #1
Peter Kasting
BTW, one test you should do: After doing option-enter in Tab A (to create Tab ...
11 years, 3 months ago (2009-09-24 18:32:22 UTC) #2
Peter Kasting
This approach is wrong, see comments on the bug.
11 years, 3 months ago (2009-09-24 18:35:11 UTC) #3
rohitrao (ping after 24h)
Looks like we pass. I've added another TEST= line, in case QA actually looks at ...
11 years, 3 months ago (2009-09-24 18:35:21 UTC) #4
rohitrao (ping after 24h)
Changed this behavior to apply only in the omnibox. Verified that option-click in a webpage ...
11 years, 3 months ago (2009-09-24 18:48:58 UTC) #5
Peter Kasting
On 2009/09/24 18:48:58, rohitrao wrote: > Changed this behavior to apply only in the omnibox. ...
11 years, 3 months ago (2009-09-24 19:16:02 UTC) #6
Scott Hess - ex-Googler
11 years, 3 months ago (2009-09-24 20:35:47 UTC) #7
LGTM.  A one-line code change which incidentally allows us to remove a TODO.  So
awesome.

Powered by Google App Engine
This is Rietveld 408576698