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

Unified Diff: chrome/browser/ui/cocoa/browser_window_controller.mm

Issue 8314017: Fix minor WebUI dialog issues, mostly with bookmark management (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Tweaks from flackr CR Created 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/browser.cc ('k') | chrome/browser/ui/webui/bookmarks_ui.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/cocoa/browser_window_controller.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_controller.mm b/chrome/browser/ui/cocoa/browser_window_controller.mm
index 45a5e51f3843fd88ceae33acc820269830b9b068..304bbd37317ff4b8aaae95ba28a5ce00717e0306 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller.mm
@@ -1628,17 +1628,10 @@ enum {
DCHECK(responds);
if (responds) {
const BookmarkNode* node = [sender node];
- if (node) {
- // A BookmarkEditorController is a sheet that owns itself, and
- // deallocates itself when closed.
- [[[BookmarkEditorController alloc]
- initWithParentWindow:[self window]
- profile:browser_->profile()
- parent:node->parent()
- node:node
- configuration:BookmarkEditor::SHOW_TREE]
- runAsModalSheet];
- }
+ if (node)
+ BookmarkEditor::Show([self window], browser_->profile(),
+ BookmarkEditor::EditDetails::EditNode(node),
+ BookmarkEditor::SHOW_TREE);
}
}
« no previous file with comments | « chrome/browser/ui/browser.cc ('k') | chrome/browser/ui/webui/bookmarks_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698