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

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: Tiny tweaks 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
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..9f24ff38df191036a73c004beff396e06848946e 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller.mm
@@ -1629,15 +1629,9 @@ enum {
if (responds) {
const BookmarkNode* node = [sender node];
if (node) {
flackr 2011/10/24 19:16:01 Remove {}
Rick Byers 2011/10/24 20:02:42 Done.
- // 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];
+ BookmarkEditor::Show([self window], browser_->profile(),
flackr 2011/10/24 19:16:01 Should only be indented 6 spaces.
Rick Byers 2011/10/24 20:02:42 Done.
+ BookmarkEditor::EditDetails::EditNode(node),
+ BookmarkEditor::SHOW_TREE);
}
}
}

Powered by Google App Engine
This is Rietveld 408576698