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

Unified Diff: chrome/browser/cocoa/geolocation_exceptions_window_controller.mm

Issue 2838037: Introduce RemoveRowTableModel interface, let GeolocationExceptionsTableModel derive from it. (Closed)
Patch Set: '' Created 10 years, 6 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/cocoa/geolocation_exceptions_window_controller.mm
diff --git a/chrome/browser/cocoa/geolocation_exceptions_window_controller.mm b/chrome/browser/cocoa/geolocation_exceptions_window_controller.mm
index fbb69d326656fb8508231b0a3713eb54b0231212..654d8e843c5308edc14b3244738a6169e5365914 100644
--- a/chrome/browser/cocoa/geolocation_exceptions_window_controller.mm
+++ b/chrome/browser/cocoa/geolocation_exceptions_window_controller.mm
@@ -127,7 +127,7 @@ GeolocationExceptionsWindowController* g_exceptionWindow = nil;
case NSDeleteFunctionKey:
// Delete deletes.
if ([[tableView_ selectedRowIndexes] count] > 0)
- [self removeException:self];
+ [self removeRow:self];
return;
}
}
@@ -153,13 +153,13 @@ GeolocationExceptionsWindowController* g_exceptionWindow = nil;
[NSApp endSheet:[self window]];
}
-- (IBAction)removeException:(id)sender {
+- (IBAction)removeRow:(id)sender {
GeolocationExceptionsTableModel::Rows rows;
[self selectedRows:&rows];
- model_->RemoveExceptions(rows);
+ model_->RemoveRows(rows);
}
-- (IBAction)removeAllExceptions:(id)sender {
+- (IBAction)removeAll:(id)sender {
model_->RemoveAll();
}
@@ -208,7 +208,7 @@ GeolocationExceptionsWindowController* g_exceptionWindow = nil;
- (void)adjustEditingButtons {
GeolocationExceptionsTableModel::Rows rows;
[self selectedRows:&rows];
- [removeButton_ setEnabled:model_->CanRemoveExceptions(rows)];
+ [removeButton_ setEnabled:model_->CanRemoveRows(rows)];
[removeAllButton_ setEnabled:([tableView_ numberOfRows] > 0)];
}

Powered by Google App Engine
This is Rietveld 408576698