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

Side by Side Diff: chrome/browser/bookmarks/bookmark_editor.h

Issue 1795007: Get rid of BookmarkEditor::Show()'s BookmarkEditor::Handler argument. (Closed)
Patch Set: baz Created 10 years, 7 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
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 #ifndef CHROME_BROWSER_BOOKMARKS_BOOKMARK_EDITOR_H_ 5 #ifndef CHROME_BROWSER_BOOKMARKS_BOOKMARK_EDITOR_H_
6 #define CHROME_BROWSER_BOOKMARKS_BOOKMARK_EDITOR_H_ 6 #define CHROME_BROWSER_BOOKMARKS_BOOKMARK_EDITOR_H_
7 7
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 std::vector<std::pair<GURL, std::wstring> > urls; 68 std::vector<std::pair<GURL, std::wstring> > urls;
69 }; 69 };
70 70
71 // Shows the bookmark editor. The bookmark editor allows editing an 71 // Shows the bookmark editor. The bookmark editor allows editing an
72 // existing node or creating a new bookmark node (as determined by 72 // existing node or creating a new bookmark node (as determined by
73 // |details.type|). If |configuration| is SHOW_TREE, a tree is shown allowing 73 // |details.type|). If |configuration| is SHOW_TREE, a tree is shown allowing
74 // the user to choose the parent of the node. 74 // the user to choose the parent of the node.
75 // |parent| gives the initial parent to select in the tree for the node. 75 // |parent| gives the initial parent to select in the tree for the node.
76 // |parent| is only used if |details.existing_node| is null. 76 // |parent| is only used if |details.existing_node| is null.
77 // BookmarkEditor takes ownership of |handler| and deletes it when done. 77 // BookmarkEditor takes ownership of |handler| and deletes it when done.
78 // |handler| may be null. See description of Handler for details.
79 static void Show(gfx::NativeWindow parent_window, 78 static void Show(gfx::NativeWindow parent_window,
80 Profile* profile, 79 Profile* profile,
81 const BookmarkNode* parent, 80 const BookmarkNode* parent,
82 const EditDetails& details, 81 const EditDetails& details,
83 Configuration configuration, 82 Configuration configuration);
84 Handler* handler);
85 }; 83 };
86 84
87 #endif // CHROME_BROWSER_BOOKMARKS_BOOKMARK_EDITOR_H_ 85 #endif // CHROME_BROWSER_BOOKMARKS_BOOKMARK_EDITOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698