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

Unified Diff: chrome/browser/ui/cocoa/tabs/tab_controller.mm

Issue 7565007: Clicking tab close with option key close the other tabs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Using [NSApp currentEvent]. Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/tabs/tab_controller.mm
diff --git a/chrome/browser/ui/cocoa/tabs/tab_controller.mm b/chrome/browser/ui/cocoa/tabs/tab_controller.mm
index 4ee4a99d95d959d2b14a514f56fe26881c525dbd..8b31687061582e2bda18b733bd984a65d1cc9dcb 100644
--- a/chrome/browser/ui/cocoa/tabs/tab_controller.mm
+++ b/chrome/browser/ui/cocoa/tabs/tab_controller.mm
@@ -144,6 +144,13 @@ class MenuDelegate : public ui::SimpleMenuModel::Delegate {
}
- (IBAction)closeTab:(id)sender {
+ if ([[NSApp currentEvent] modifierFlags] & NSAlternateKeyMask) {
Robert Sesek 2011/08/09 15:20:37 nit: extra spaces
bashi 2011/08/10 06:52:30 Done.
+ if ([[self target] respondsToSelector:@selector(closeOtherTabs:)]) {
Robert Sesek 2011/08/09 15:20:37 Combine these two if's with &&
bashi 2011/08/10 06:52:30 Done.
+ [[self target] performSelector:@selector(closeOtherTabs:)
+ withObject:[self view]];
+ return;
+ }
+ }
if ([[self target] respondsToSelector:@selector(closeTab:)]) {
[[self target] performSelector:@selector(closeTab:)
withObject:[self view]];

Powered by Google App Engine
This is Rietveld 408576698