Remove wrench button from panel.

chrome/browser/ui/panels/panel_browsertest.cc

 // Copyright (c) 2012 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 "base/bind.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/download/download_service.h"
 #include "chrome/browser/download/download_service_factory.h"
 #include "chrome/browser/net/url_request_mock_util.h"
@@ skipping 10 matching lines @@
 #include "chrome/browser/ui/app_modal_dialogs/native_app_modal_dialog.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/find_bar/find_bar.h"
 #include "chrome/browser/ui/find_bar/find_bar_controller.h"
 #include "chrome/browser/ui/panels/base_panel_browser_test.h"
 #include "chrome/browser/ui/panels/docked_panel_strip.h"
 #include "chrome/browser/ui/panels/native_panel.h"
 #include "chrome/browser/ui/panels/panel.h"
 #include "chrome/browser/ui/panels/panel_manager.h"
-#include "chrome/browser/ui/panels/panel_settings_menu_model.h"
 #include "chrome/browser/ui/panels/test_panel_mouse_watcher.h"
 #include "chrome/browser/web_applications/web_app.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/extensions/extension_manifest_constants.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/download_manager.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/web_contents.h"
@@ skipping 84 matching lines @@
     DCHECK(old_states.size() == new_states.size());
     std::vector<Panel*> panels = PanelManager::GetInstance()->panels();
     for (size_t i = 0; i < old_states.size(); i++) {
       if (old_states[i] != new_states[i]){
         WaitForPanelActiveState(
             panels[i], new_states[i] ? SHOW_AS_ACTIVE : SHOW_AS_INACTIVE);
       }
     }
   }
 
-  struct MenuItem {
-    int id;
-    bool enabled;
-  };
-
-  void ValidateSettingsMenuItems(ui::SimpleMenuModel* settings_menu_contents,
-                                 size_t num_expected_menu_items,
-                                 const MenuItem* expected_menu_items) {
-    ASSERT_TRUE(settings_menu_contents);
-    EXPECT_EQ(static_cast<int>(num_expected_menu_items),
-              settings_menu_contents->GetItemCount());
-    for (size_t i = 0; i < num_expected_menu_items; ++i) {
-      if (expected_menu_items[i].id == -1) {
-        EXPECT_EQ(ui::MenuModel::TYPE_SEPARATOR,
-                  settings_menu_contents->GetTypeAt(i));
-      } else {
-        EXPECT_EQ(expected_menu_items[i].id,
-                  settings_menu_contents->GetCommandIdAt(i));
-        EXPECT_EQ(expected_menu_items[i].enabled,
-                  settings_menu_contents->IsEnabledAt(i));
-      }
-    }
-  }
-
-  void TestCreateSettingsMenuForExtension(const FilePath::StringType& path,
-                                          Extension::Location location,
-                                          const std::string& homepage_url,
-                                          const std::string& options_page) {
-    // Creates a testing extension.
-    DictionaryValue extra_value;
-    if (!homepage_url.empty()) {
-      extra_value.SetString(extension_manifest_keys::kHomepageURL,
-                            homepage_url);
-    }
-    if (!options_page.empty()) {
-      extra_value.SetString(extension_manifest_keys::kOptionsPage,
-                            options_page);
-    }
-    scoped_refptr<Extension> extension = CreateExtension(
-        path, location, extra_value);
-
-    // Creates a panel with the app name that comes from the extension ID.
-    Panel* panel = CreatePanel(
-        web_app::GenerateApplicationNameFromExtensionId(extension->id()));
-
-    scoped_ptr<PanelSettingsMenuModel> settings_menu_model(
-        new PanelSettingsMenuModel(panel));
-
-    // Validates the settings menu items.
-    MenuItem expected_panel_menu_items[] = {
-        { PanelSettingsMenuModel::COMMAND_NAME, false },
-        { -1, false },  // Separator
-        { PanelSettingsMenuModel::COMMAND_CONFIGURE, false },
-        { PanelSettingsMenuModel::COMMAND_DISABLE, false },
-        { PanelSettingsMenuModel::COMMAND_UNINSTALL, false },
-        { -1, false },  // Separator
-        { PanelSettingsMenuModel::COMMAND_MANAGE, true }
-    };
-    if (!homepage_url.empty())
-      expected_panel_menu_items[0].enabled = true;
-    if (!options_page.empty())
-      expected_panel_menu_items[2].enabled = true;
-    if (location != Extension::EXTERNAL_POLICY_DOWNLOAD) {
-      expected_panel_menu_items[3].enabled = true;
-      expected_panel_menu_items[4].enabled = true;
-    }
-    ValidateSettingsMenuItems(settings_menu_model.get(),
-                              arraysize(expected_panel_menu_items),
-                              expected_panel_menu_items);
-
-    panel->Close();
-  }
-
   void TestMinimizeRestore() {
     // This constant is used to generate a point 'sufficiently higher then
     // top edge of the panel'. On some platforms (Mac) we extend hover area
     // a bit above the minimized panel as well, so it takes significant
     // distance to 'move mouse out' of the hover-sensitive area.
     const int kFarEnoughFromHoverArea = 153;
 
     PanelManager* panel_manager = PanelManager::GetInstance();
     std::vector<Panel*> panels = panel_manager->panels();
     std::vector<gfx::Rect> test_begin_bounds = GetAllPanelBounds();
@@ skipping 152 matching lines @@
 IN_PROC_BROWSER_TEST_F(PanelBrowserTest, FindBar) {
   Panel* panel = CreatePanelWithBounds("PanelTest", gfx::Rect(0, 0, 400, 400));
   Browser* browser = panel->browser();
   // FindBar needs tab contents.
   CreateTestTabContents(browser);
   browser->ShowFindBar();
   ASSERT_TRUE(browser->GetFindBarController()->find_bar()->IsFindBarVisible());
   panel->Close();
 }
 
-IN_PROC_BROWSER_TEST_F(PanelBrowserTest, CreateSettingsMenu) {
-  TestCreateSettingsMenuForExtension(
-      FILE_PATH_LITERAL("extension1"), Extension::EXTERNAL_POLICY_DOWNLOAD,
-      "", "");
-  TestCreateSettingsMenuForExtension(
-      FILE_PATH_LITERAL("extension2"), Extension::INVALID,
-      "http://home", "options.html");
-}
-
 // Flaky: http://crbug.com/105445
 IN_PROC_BROWSER_TEST_F(PanelBrowserTest, DISABLED_AutoResize) {
   PanelManager* panel_manager = PanelManager::GetInstance();
   panel_manager->enable_auto_sizing(true);
   // Bigger space is needed by this test.
   SetTestingWorkArea(gfx::Rect(0, 0, 1200, 900));
 
   // Create a test panel with tab contents loaded.
   CreatePanelParams params("PanelTest1", gfx::Rect(), SHOW_AS_ACTIVE);
   GURL url(ui_test_utils::GetTestUrl(
@@ skipping 1351 matching lines @@
   // position when tall panel brings up its titlebar.
   CloseWindowAndWait(panel1->browser());
   EXPECT_EQ(balloon_bottom_after_tall_panel_titlebar_up,
             GetBalloonBottomPosition(balloon));
 
   // Closing the remaining tall panel should move the notification balloon back
   // to its original position.
   CloseWindowAndWait(panel2->browser());
   EXPECT_EQ(original_balloon_bottom, GetBalloonBottomPosition(balloon));
 }