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

Side by Side Diff: chrome/browser/gtk/bookmark_manager_gtk.cc

Issue 271115: Makes canceling 'bookmark all tabs' delete the folder. Or rather,... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 2 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/gtk/bookmark_manager_gtk.h" 5 #include "chrome/browser/gtk/bookmark_manager_gtk.h"
6 6
7 #include <gdk/gdkkeysyms.h> 7 #include <gdk/gdkkeysyms.h>
8 #include <vector> 8 #include <vector>
9 9
10 #include "app/gtk_dnd_util.h" 10 #include "app/gtk_dnd_util.h"
(...skipping 206 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 BookmarkNodeAdded(model, new_parent, new_index); 217 BookmarkNodeAdded(model, new_parent, new_index);
218 } 218 }
219 219
220 void BookmarkManagerGtk::BookmarkNodeAdded(BookmarkModel* model, 220 void BookmarkManagerGtk::BookmarkNodeAdded(BookmarkModel* model,
221 const BookmarkNode* parent, 221 const BookmarkNode* parent,
222 int index) { 222 int index) {
223 const BookmarkNode* node = parent->GetChild(index); 223 const BookmarkNode* node = parent->GetChild(index);
224 if (node->is_folder()) { 224 if (node->is_folder()) {
225 GtkTreeIter iter = { 0, }; 225 GtkTreeIter iter = { 0, };
226 if (RecursiveFind(GTK_TREE_MODEL(left_store_), &iter, parent->id())) 226 if (RecursiveFind(GTK_TREE_MODEL(left_store_), &iter, parent->id()))
227 bookmark_utils::AddToTreeStoreAt(node, 0, NULL, left_store_, NULL, &iter); 227 bookmark_utils::AddToTreeStoreAt(node, 0, left_store_, NULL, &iter);
228 } 228 }
229 } 229 }
230 230
231 void BookmarkManagerGtk::BookmarkNodeRemoved(BookmarkModel* model, 231 void BookmarkManagerGtk::BookmarkNodeRemoved(BookmarkModel* model,
232 const BookmarkNode* parent, 232 const BookmarkNode* parent,
233 int old_index, 233 int old_index,
234 const BookmarkNode* node) { 234 const BookmarkNode* node) {
235 if (node->is_folder()) { 235 if (node->is_folder()) {
236 GtkTreeIter iter = { 0, }; 236 GtkTreeIter iter = { 0, };
237 if (RecursiveFind(GTK_TREE_MODEL(left_store_), &iter, node->id())) { 237 if (RecursiveFind(GTK_TREE_MODEL(left_store_), &iter, node->id())) {
(...skipping 370 matching lines...) Expand 10 before | Expand all | Expand 10 after
608 608
609 organize_menu_.reset(new BookmarkContextMenuGtk(GTK_WINDOW(window_), profile_, 609 organize_menu_.reset(new BookmarkContextMenuGtk(GTK_WINDOW(window_), profile_,
610 NULL, NULL, parent, nodes, 610 NULL, NULL, parent, nodes,
611 BookmarkContextMenuGtk::BOOKMARK_MANAGER_ORGANIZE_MENU, NULL)); 611 BookmarkContextMenuGtk::BOOKMARK_MANAGER_ORGANIZE_MENU, NULL));
612 gtk_menu_item_set_submenu(GTK_MENU_ITEM(organize_), organize_menu_->menu()); 612 gtk_menu_item_set_submenu(GTK_MENU_ITEM(organize_), organize_menu_->menu());
613 } 613 }
614 614
615 void BookmarkManagerGtk::BuildLeftStore() { 615 void BookmarkManagerGtk::BuildLeftStore() {
616 GtkTreeIter select_iter; 616 GtkTreeIter select_iter;
617 bookmark_utils::AddToTreeStore(model_, 617 bookmark_utils::AddToTreeStore(model_,
618 model_->GetBookmarkBarNode()->id(), NULL, left_store_, &select_iter); 618 model_->GetBookmarkBarNode()->id(), left_store_, &select_iter);
619 gtk_tree_selection_select_iter(left_selection(), &select_iter); 619 gtk_tree_selection_select_iter(left_selection(), &select_iter);
620 620
621 // TODO(estade): is there a decent stock icon we can use here? 621 // TODO(estade): is there a decent stock icon we can use here?
622 ResourceBundle& rb = ResourceBundle::GetSharedInstance(); 622 ResourceBundle& rb = ResourceBundle::GetSharedInstance();
623 gtk_tree_store_append(left_store_, &select_iter, NULL); 623 gtk_tree_store_append(left_store_, &select_iter, NULL);
624 gtk_tree_store_set(left_store_, &select_iter, 624 gtk_tree_store_set(left_store_, &select_iter,
625 bookmark_utils::FOLDER_ICON, 625 bookmark_utils::FOLDER_ICON,
626 rb.GetPixbufNamed(IDR_BOOKMARK_MANAGER_RECENT_ICON), 626 rb.GetPixbufNamed(IDR_BOOKMARK_MANAGER_RECENT_ICON),
627 bookmark_utils::FOLDER_NAME, 627 bookmark_utils::FOLDER_NAME,
628 l10n_util::GetStringUTF8( 628 l10n_util::GetStringUTF8(
(...skipping 799 matching lines...) Expand 10 before | Expand all | Expand 10 after
1428 modifier & gtk_accelerator_get_default_mod_mask()); 1428 modifier & gtk_accelerator_get_default_mod_mask());
1429 // The only accelerator we have registered is ctrl+w, so any other value is a 1429 // The only accelerator we have registered is ctrl+w, so any other value is a
1430 // non-fatal error. 1430 // non-fatal error.
1431 DCHECK_EQ(keyval, static_cast<guint>(GDK_w)); 1431 DCHECK_EQ(keyval, static_cast<guint>(GDK_w));
1432 DCHECK_EQ(modifier, GDK_CONTROL_MASK); 1432 DCHECK_EQ(modifier, GDK_CONTROL_MASK);
1433 1433
1434 gtk_widget_destroy(bookmark_manager->window_); 1434 gtk_widget_destroy(bookmark_manager->window_);
1435 1435
1436 return TRUE; 1436 return TRUE;
1437 } 1437 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698