Use bookmark manager to add/edit bookmark pages with webui_dialogs=1.

chrome/browser/ui/browser.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/browser.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #endif  // OS_WIN
@@ skipping 1937 matching lines @@
 }
 
 void Browser::OpenBookmarkManagerForNode(int64 node_id) {
   UserMetrics::RecordAction(UserMetricsAction("ShowBookmarkManager"));
   UserMetrics::RecordAction(UserMetricsAction("ShowBookmarks"));
   ShowSingletonTabOverwritingNTP(
     GetSingletonTabNavigateParams(GURL(chrome::kChromeUIBookmarksURL).Resolve(
       StringPrintf("/#%s", base::Int64ToString(node_id).c_str()))));
 }
 
+void Browser::OpenBookmarkManagerEditNode(int64 node_id) {
+  UserMetrics::RecordAction(UserMetricsAction("ShowBookmarkManager"));

[Rick Byers, 2011/08/04 18:26:07]

Avoid the duplicated code here - make all 3 functions delegate to some helper
function which takes the hash prefix as an extra argument.

[flackr, 2011/08/12 18:11:24]

Done.

+  UserMetrics::RecordAction(UserMetricsAction("ShowBookmarks"));
+  ShowSingletonTabOverwritingNTP(
+    GetSingletonTabNavigateParams(GURL(chrome::kChromeUIBookmarksURL).Resolve(

[arv (Not doing code reviews), 2011/08/11 22:32:22]

fix indentation

[flackr, 2011/08/12 18:11:24]

Done.

+      StringPrintf("/#e=%s", base::Int64ToString(node_id).c_str()))));
+}
+
+void Browser::OpenBookmarkManagerAddNodeIn(int64 node_id) {
+  UserMetrics::RecordAction(UserMetricsAction("ShowBookmarkManager"));
+  UserMetrics::RecordAction(UserMetricsAction("ShowBookmarks"));
+  ShowSingletonTabOverwritingNTP(
+    GetSingletonTabNavigateParams(GURL(chrome::kChromeUIBookmarksURL).Resolve(

[arv (Not doing code reviews), 2011/08/11 22:32:22]

here too

[flackr, 2011/08/12 18:11:24]

Done.

+      StringPrintf("/#a=%s", base::Int64ToString(node_id).c_str()))));
+}
+
 void Browser::ShowAppMenu() {
   // We record the user metric for this event in WrenchMenu::RunMenu.
   window_->ShowAppMenu();
 }
 
 void Browser::ShowHistoryTab() {
   UserMetrics::RecordAction(UserMetricsAction("ShowHistory"));
   ShowSingletonTabOverwritingNTP(
       GetSingletonTabNavigateParams(GURL(chrome::kChromeUIHistoryURL)));
 }
@@ skipping 2846 matching lines @@
 }
 
 void Browser::ShowSyncSetup() {
   ProfileSyncService* service =
       profile()->GetOriginalProfile()->GetProfileSyncService();
   if (service->HasSyncSetupCompleted())
     ShowOptionsTab(chrome::kSyncSetupSubPage);
   else
     service->ShowLoginDialog();
 }