Commit e44ea732 authored by Timothy Loh's avatar Timothy Loh Committed by Commit Bot

Remove unused InternalAppContextMenu

InternalAppContextMenu was added to support a custom context menu for
Plugin VM but no longer appears to be used.

Change-Id: Ide138a0a22eeb1356ad723e533bbfd9f20e1aeec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2164136Reviewed-by: default avatarNancy Wang <nancylingwang@chromium.org>
Commit-Queue: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#762705}
parent 7ab4fae5
...@@ -3846,8 +3846,6 @@ jumbo_static_library("ui") { ...@@ -3846,8 +3846,6 @@ jumbo_static_library("ui") {
"app_list/extension_app_item.h", "app_list/extension_app_item.h",
"app_list/extension_app_utils.cc", "app_list/extension_app_utils.cc",
"app_list/extension_app_utils.h", "app_list/extension_app_utils.h",
"app_list/internal_app/internal_app_context_menu.cc",
"app_list/internal_app/internal_app_context_menu.h",
"app_list/internal_app/internal_app_metadata.cc", "app_list/internal_app/internal_app_metadata.cc",
"app_list/internal_app/internal_app_metadata.h", "app_list/internal_app/internal_app_metadata.h",
"app_list/md_icon_normalizer.cc", "app_list/md_icon_normalizer.cc",
......
// Copyright 2019 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/app_list/internal_app/internal_app_context_menu.h"
#include "ash/public/cpp/app_menu_constants.h"
#include "chrome/browser/apps/app_service/app_service_metrics.h"
#include "chrome/browser/chromeos/plugin_vm/plugin_vm_manager.h"
#include "chrome/browser/chromeos/plugin_vm/plugin_vm_util.h"
#include "chrome/browser/ui/app_list/internal_app/internal_app_metadata.h"
#include "chrome/grit/generated_resources.h"
InternalAppContextMenu::InternalAppContextMenu(
Profile* profile,
const std::string& app_id,
AppListControllerDelegate* controller)
: app_list::AppContextMenu(nullptr, profile, app_id, controller) {}
InternalAppContextMenu::~InternalAppContextMenu() = default;
bool InternalAppContextMenu::IsCommandIdEnabled(int command_id) const {
if (command_id == ash::SHUTDOWN_GUEST_OS) {
DCHECK_EQ(app_list::FindInternalApp(app_id())->internal_app_name,
apps::BuiltInAppName::kPluginVm);
return plugin_vm::IsPluginVmRunning(profile());
}
return app_list::AppContextMenu::IsCommandIdEnabled(command_id);
}
void InternalAppContextMenu::ExecuteCommand(int command_id, int event_flags) {
switch (command_id) {
case ash::SHUTDOWN_GUEST_OS:
DCHECK_EQ(app_list::FindInternalApp(app_id())->internal_app_name,
apps::BuiltInAppName::kPluginVm);
plugin_vm::PluginVmManager::GetForProfile(profile())->StopPluginVm(
plugin_vm::kPluginVmName, /*force=*/false);
return;
}
app_list::AppContextMenu::ExecuteCommand(command_id, event_flags);
}
void InternalAppContextMenu::BuildMenu(ui::SimpleMenuModel* menu_model) {
app_list::AppContextMenu::BuildMenu(menu_model);
const auto* internal_app = app_list::FindInternalApp(app_id());
DCHECK(internal_app);
if (internal_app->internal_app_name == apps::BuiltInAppName::kPluginVm) {
AddContextMenuOption(menu_model, ash::SHUTDOWN_GUEST_OS,
IDS_PLUGIN_VM_SHUT_DOWN_MENU_ITEM);
}
}
// Copyright 2019 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.
#ifndef CHROME_BROWSER_UI_APP_LIST_INTERNAL_APP_INTERNAL_APP_CONTEXT_MENU_H_
#define CHROME_BROWSER_UI_APP_LIST_INTERNAL_APP_INTERNAL_APP_CONTEXT_MENU_H_
#include <string>
#include "base/macros.h"
#include "chrome/browser/ui/app_list/app_context_menu.h"
class AppListControllerDelegate;
class Profile;
class InternalAppContextMenu : public app_list::AppContextMenu {
public:
InternalAppContextMenu(Profile* profile,
const std::string& app_id,
AppListControllerDelegate* controller);
~InternalAppContextMenu() override;
// app_list::AppContextMenu:
bool IsCommandIdEnabled(int command_id) const override;
void ExecuteCommand(int command_id, int event_flags) override;
void BuildMenu(ui::SimpleMenuModel* menu_model) override;
private:
DISALLOW_COPY_AND_ASSIGN(InternalAppContextMenu);
};
#endif // CHROME_BROWSER_UI_APP_LIST_INTERNAL_APP_INTERNAL_APP_CONTEXT_MENU_H_
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment