Windows/Views: delete native bookmark manager code.

chrome/browser/views/bookmark_context_menu_controller_views.cc

Index: chrome/browser/views/bookmark_context_menu_controller_views.cc
diff --git a/chrome/browser/views/bookmark_context_menu_controller_views.cc b/chrome/browser/views/bookmark_context_menu_controller_views.cc
index dc7d090809be914f3d1dce59103c9b47e9598544..712df823a3340557637cf1397a34292eaa13e79c 100644
--- a/chrome/browser/views/bookmark_context_menu_controller_views.cc
+++ b/chrome/browser/views/bookmark_context_menu_controller_views.cc
@@ -8,7 +8,6 @@
 #include "base/compiler_specific.h"
 #include "chrome/browser/bookmarks/bookmark_editor.h"
 #include "chrome/browser/bookmarks/bookmark_folder_editor_controller.h"
-#include "chrome/browser/bookmarks/bookmark_manager.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/bookmarks/bookmark_utils.h"
 #include "chrome/browser/browser.h"
@@ -35,27 +34,6 @@ bool NodeHasURLs(const BookmarkNode* node) {
   return false;
 }
 
-// SelectOnCreationHandler ----------------------------------------------------
-
-// Used when adding a new bookmark. If a new bookmark is created it is selected
-// in the bookmark manager.
-class SelectOnCreationHandler : public BookmarkEditor::Handler {
- public:
-  explicit SelectOnCreationHandler(Profile* profile) : profile_(profile) {
-  }
-
-  virtual void NodeCreated(const BookmarkNode* new_node) {
-#if defined(OS_WIN)
-    BookmarkManager::SelectInTree(profile_, new_node);
-#endif
-  }
-
- private:
-  Profile* profile_;
-
-  DISALLOW_COPY_AND_ASSIGN(SelectOnCreationHandler);
-};
-
 }  // namespace
 
 BookmarkContextMenuControllerViews::BookmarkContextMenuControllerViews(
@@ -107,15 +85,6 @@ void BookmarkContextMenuControllerViews::BuildMenu() {
     delegate_->AddItem(IDS_BOOKMARK_BAR_EDIT);
   }
 
-#if defined(OS_WIN)
-  if (configuration_ == BOOKMARK_MANAGER_TABLE ||
-      configuration_ == BOOKMARK_MANAGER_TABLE_OTHER ||
-      configuration_ == BOOKMARK_MANAGER_ORGANIZE_MENU ||
-      configuration_ == BOOKMARK_MANAGER_ORGANIZE_MENU_OTHER) {
-    delegate_->AddItem(IDS_BOOKMARK_MANAGER_SHOW_IN_FOLDER);
-  }
-#endif
-
   delegate_->AddSeparator();
   delegate_->AddItem(IDS_CUT);
   delegate_->AddItem(IDS_COPY);
@@ -213,20 +182,14 @@ void BookmarkContextMenuControllerViews::ExecuteCommand(int id) {
       UserMetrics::RecordAction(
           UserMetricsAction("BookmarkBar_ContextMenu_Add"), profile_);
 
-      BookmarkEditor::Configuration editor_config;
-      BookmarkEditor::Handler* handler = NULL;
-      if (configuration_ == BOOKMARK_BAR) {
-        editor_config = BookmarkEditor::SHOW_TREE;
-      } else {
-        editor_config = BookmarkEditor::NO_TREE;
-        // This is owned by the BookmarkEditorView.
-        handler = new SelectOnCreationHandler(profile_);
-      }
+      BookmarkEditor::Configuration editor_config =
+          (configuration_ == BOOKMARK_BAR) ? BookmarkEditor::SHOW_TREE :
+                                             BookmarkEditor::NO_TREE;
       // TODO: this should honor the index from GetParentForNewNodes.
       BookmarkEditor::Show(
           parent_window_, profile_,
           bookmark_utils::GetParentForNewNodes(parent_, selection_, NULL),
-          BookmarkEditor::EditDetails(), editor_config, handler);
+          BookmarkEditor::EditDetails(), editor_config, NULL);
       break;
     }
 
@@ -249,21 +212,6 @@ void BookmarkContextMenuControllerViews::ExecuteCommand(int id) {
       bookmark_utils::ToggleWhenVisible(profile_);
       break;
 
-#if defined(OS_WIN)
-    case IDS_BOOKMARK_MANAGER_SHOW_IN_FOLDER:
-      UserMetrics::RecordAction(
-          UserMetricsAction("BookmarkBar_ContextMenu_ShowInFolder"),
-          profile_);
-
-      if (selection_.size() != 1) {
-        NOTREACHED();
-        return;
-      }
-
-      BookmarkManager::SelectInTree(profile_, selection_[0]);
-      break;
-#endif
-
     case IDS_BOOKMARK_MANAGER:
       UserMetrics::RecordAction(UserMetricsAction("ShowBookmarkManager"),
                                 profile_);
@@ -335,13 +283,6 @@ bool BookmarkContextMenuControllerViews::IsCommandEnabled(int id) const {
     case IDS_BOOKMARK_BAR_REMOVE:
       return !selection_.empty() && !is_root_node;
 
-#if defined(OS_WIN)
-    case IDS_BOOKMARK_MANAGER_SHOW_IN_FOLDER:
-      return (configuration_ == BOOKMARK_MANAGER_TABLE_OTHER ||
-              configuration_ == BOOKMARK_MANAGER_ORGANIZE_MENU_OTHER) &&
-             selection_.size() == 1;
-#endif
-
     case IDS_BOOKMARK_MANAGER_SORT:
       return parent_ && parent_ != model_->root_node();