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

Unified Diff: ash/system/palette/palette_tray.cc

Issue 2826593002: Remove ash's MaterialDesignController. (Closed)
Patch Set: one more dead include Created 3 years, 8 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
« no previous file with comments | « ash/material_design/material_design_controller.cc ('k') | ash/system/status_area_widget.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/system/palette/palette_tray.cc
diff --git a/ash/system/palette/palette_tray.cc b/ash/system/palette/palette_tray.cc
index 73c97978ca2f9bf9211971fab0660f84121d4b02..d72986eca7d8827a9864b8942da4c1382cfcc0a9 100644
--- a/ash/system/palette/palette_tray.cc
+++ b/ash/system/palette/palette_tray.cc
@@ -4,7 +4,6 @@
#include "ash/system/palette/palette_tray.h"
-#include "ash/material_design/material_design_controller.h"
#include "ash/public/cpp/shell_window_ids.h"
#include "ash/resources/grit/ash_resources.h"
#include "ash/resources/vector_icons/vector_icons.h"
@@ -86,31 +85,12 @@ class TitleView : public views::View, public views::ButtonListener {
TrayPopupItemStyle style(TrayPopupItemStyle::FontStyle::TITLE);
style.SetupLabel(title_label);
box_layout->SetFlexForView(title_label, 1);
- if (MaterialDesignController::IsSystemTrayMenuMaterial()) {
- help_button_ =
- new SystemMenuButton(this, TrayPopupInkDropStyle::HOST_CENTERED,
- kSystemMenuHelpIcon, IDS_ASH_STATUS_TRAY_HELP);
- settings_button_ = new SystemMenuButton(
- this, TrayPopupInkDropStyle::HOST_CENTERED, kSystemMenuSettingsIcon,
- IDS_ASH_PALETTE_SETTINGS);
- } else {
- gfx::ImageSkia help_icon =
- gfx::CreateVectorIcon(kSystemMenuHelpIcon, kMenuIconColor);
- gfx::ImageSkia settings_icon =
- gfx::CreateVectorIcon(kSystemMenuSettingsIcon, kMenuIconColor);
-
- auto* help_button = new ash::TrayPopupHeaderButton(
- this, help_icon, IDS_ASH_STATUS_TRAY_HELP);
- help_button->SetTooltipText(
- l10n_util::GetStringUTF16(IDS_ASH_STATUS_TRAY_HELP));
- help_button_ = help_button;
-
- auto* settings_button = new ash::TrayPopupHeaderButton(
- this, settings_icon, IDS_ASH_STATUS_TRAY_SETTINGS);
- settings_button->SetTooltipText(
- l10n_util::GetStringUTF16(IDS_ASH_STATUS_TRAY_SETTINGS));
- settings_button_ = settings_button;
- }
+ help_button_ =
+ new SystemMenuButton(this, TrayPopupInkDropStyle::HOST_CENTERED,
+ kSystemMenuHelpIcon, IDS_ASH_STATUS_TRAY_HELP);
+ settings_button_ =
+ new SystemMenuButton(this, TrayPopupInkDropStyle::HOST_CENTERED,
+ kSystemMenuSettingsIcon, IDS_ASH_PALETTE_SETTINGS);
AddChildView(help_button_);
AddChildView(settings_button_);
@@ -153,9 +133,7 @@ PaletteTray::PaletteTray(WmShelf* wm_shelf)
weak_factory_(this) {
PaletteTool::RegisterToolInstances(palette_tool_manager_.get());
- if (MaterialDesignController::IsShelfMaterial())
- SetInkDropMode(InkDropMode::ON);
-
+ SetInkDropMode(InkDropMode::ON);
SetLayoutManager(new views::FillLayout());
icon_ = new views::ImageView();
UpdateTrayIcon();
« no previous file with comments | « ash/material_design/material_design_controller.cc ('k') | ash/system/status_area_widget.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698