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

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

Issue 8497008: Implement Bookmark All Tabs Dialog with WebUI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 #pragma once 7 #pragma once
8 8
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 // existing node or creating a new bookmark node (as determined by 80 // existing node or creating a new bookmark node (as determined by
81 // |details.type|). If |configuration| is SHOW_TREE, a tree is shown allowing 81 // |details.type|). If |configuration| is SHOW_TREE, a tree is shown allowing
82 // the user to choose the parent of the node. 82 // the user to choose the parent of the node.
83 // |parent| gives the initial parent to select in the tree for the node. 83 // |parent| gives the initial parent to select in the tree for the node.
84 // |parent| is only used if |details.existing_node| is null. 84 // |parent| is only used if |details.existing_node| is null.
85 static void Show(gfx::NativeWindow parent_window, 85 static void Show(gfx::NativeWindow parent_window,
86 Profile* profile, 86 Profile* profile,
87 const EditDetails& details, 87 const EditDetails& details,
88 Configuration configuration); 88 Configuration configuration);
89 89
90 #if defined(OS_WIN) || defined(TOOLKIT_VIEWS)
mazda 2011/11/09 04:49:43 I think it's better to switch the implementation w
yoshiki 2011/11/09 15:06:39 Done.
91 static void ShowBookmarkAllTabsDialog(Profile* profile);
92 #endif
93
90 private: 94 private:
91 // Shows the native bookmark editor. 95 // Shows the native bookmark editor.
92 // TODO(flackr): Remove parent argument. 96 // TODO(flackr): Remove parent argument.
93 static void ShowNative(gfx::NativeWindow parent_window, 97 static void ShowNative(gfx::NativeWindow parent_window,
94 Profile* profile, 98 Profile* profile,
95 const BookmarkNode* parent, 99 const BookmarkNode* parent,
96 const EditDetails& details, 100 const EditDetails& details,
97 Configuration configuration); 101 Configuration configuration);
98 102
99 // Shows the WebUI bookmark editor. 103 // Shows the WebUI bookmark editor.
100 static void ShowWebUI(Profile* profile, 104 static void ShowWebUI(Profile* profile,
101 const EditDetails& details); 105 const EditDetails& details);
102 }; 106 };
103 107
104 #endif // CHROME_BROWSER_BOOKMARKS_BOOKMARK_EDITOR_H_ 108 #endif // CHROME_BROWSER_BOOKMARKS_BOOKMARK_EDITOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698