Geolocation exceptions window, Windows....

chrome/browser/gtk/options/geolocation_content_exceptions_window.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/gtk/options/geolocation_content_exceptions_window.h"
 
 #include "app/l10n_util.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/geolocation/geolocation_content_settings_map.h"
@@ skipping 125 matching lines @@
 
   std::wstring action = model_->GetText(row, IDS_EXCEPTIONS_ACTION_HEADER);
   gtk_list_store_set(list_store_, iter, COL_ACTION,
                      WideToUTF8(action).c_str(), -1);
 }
 
 void GeolocationContentExceptionsWindow::UpdateButtonState() {
   int row_count = gtk_tree_model_iter_n_children(
       GTK_TREE_MODEL(list_store_), NULL);
 
-  gtk_widget_set_sensitive(remove_button_, CountSelectedRemovable() >= 1);
+  GeolocationContentSettingsTableModel::Rows rows;
+  GetSelectedRows(&rows);
+  gtk_widget_set_sensitive(remove_button_, model_->CanRemoveExceptions(rows));
   gtk_widget_set_sensitive(remove_all_button_, row_count > 0);
 }
 
-void GeolocationContentExceptionsWindow::GetSelectedRemovableIndicies(
-    std::set<int>* indicies) {
-  gtk_tree::GetSelectedIndicies(treeview_selection_, indicies);
-
-  for (std::set<int>::iterator i = indicies->begin(); i != indicies->end(); ) {
-    std::set<int>::iterator prev = i;
-    ++i;
-    if (!model_->CanRemoveException(*prev))
-      indicies->erase(prev);
-  }
-}
-
-int GeolocationContentExceptionsWindow::CountSelectedRemovable() {
+void GeolocationContentExceptionsWindow::GetSelectedRows(
+    GeolocationContentSettingsTableModel::Rows* rows) {
   std::set<int> indicies;
-  GetSelectedRemovableIndicies(&indicies);
-  return indicies.size();
+  gtk_tree::GetSelectedIndicies(treeview_selection_, &indicies);
+  for (std::set<int>::iterator i = indicies.begin(); i != indicies.end(); ++i)
+    rows->insert(*i);
 }
 
 void GeolocationContentExceptionsWindow::Remove(GtkWidget* widget) {
-  std::set<int> indicies;
-  GetSelectedRemovableIndicies(&indicies);
-
-  for (std::set<int>::reverse_iterator i = indicies.rbegin();
-       i != indicies.rend(); ++i) {
-    model_->RemoveException(*i);
-  }
-
+  GeolocationContentSettingsTableModel::Rows rows;
+  GetSelectedRows(&rows);
+  model_->RemoveExceptions(rows);
   UpdateButtonState();
 }
 
 void GeolocationContentExceptionsWindow::RemoveAll(GtkWidget* widget) {
   model_->RemoveAll();
   UpdateButtonState();
 }
 
 void GeolocationContentExceptionsWindow::OnWindowDestroy(GtkWidget* widget) {
   instance = NULL;
   MessageLoop::current()->DeleteSoon(FROM_HERE, this);
 }
 
 void GeolocationContentExceptionsWindow::OnTreeSelectionChanged(
     GtkWidget* selection) {
   UpdateButtonState();
 }