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

Unified Diff: ash/common/frame/default_header_painter.cc

Issue 2618383004: Revert of Move more vector icons to respective directories and update code to (Closed)
Patch Set: Created 3 years, 11 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
Index: ash/common/frame/default_header_painter.cc
diff --git a/ash/common/frame/default_header_painter.cc b/ash/common/frame/default_header_painter.cc
index 4856f7e98aafe9dc3174982f837782ae866afe46..f333bf1d4a47c41303bbb3c28f753a40f0660296 100644
--- a/ash/common/frame/default_header_painter.cc
+++ b/ash/common/frame/default_header_painter.cc
@@ -7,7 +7,6 @@
#include "ash/common/ash_layout_constants.h"
#include "ash/common/frame/caption_buttons/frame_caption_button_container_view.h"
#include "ash/common/frame/header_painter_util.h"
-#include "ash/resources/vector_icons/vector_icons.h"
#include "base/debug/leak_annotations.h"
#include "base/logging.h" // DCHECK
#include "grit/ash_resources.h"
@@ -22,6 +21,7 @@
#include "ui/gfx/image/image.h"
#include "ui/gfx/scoped_canvas.h"
#include "ui/gfx/skia_util.h"
+#include "ui/gfx/vector_icons_public.h"
#include "ui/views/view.h"
#include "ui/views/widget/native_widget_aura.h"
#include "ui/views/widget/widget.h"
@@ -283,27 +283,29 @@
void DefaultHeaderPainter::UpdateAllButtonImages() {
caption_button_container_->SetUseLightImages(ShouldUseLightImages());
- caption_button_container_->SetButtonImage(CAPTION_BUTTON_ICON_MINIMIZE,
- kWindowControlMinimizeIcon);
+ caption_button_container_->SetButtonImage(
+ CAPTION_BUTTON_ICON_MINIMIZE, gfx::VectorIconId::WINDOW_CONTROL_MINIMIZE);
UpdateSizeButtonImages();
- caption_button_container_->SetButtonImage(CAPTION_BUTTON_ICON_CLOSE,
- kWindowControlCloseIcon);
-
- caption_button_container_->SetButtonImage(CAPTION_BUTTON_ICON_LEFT_SNAPPED,
- kWindowControlLeftSnappedIcon);
-
- caption_button_container_->SetButtonImage(CAPTION_BUTTON_ICON_RIGHT_SNAPPED,
- kWindowControlRightSnappedIcon);
+ caption_button_container_->SetButtonImage(
+ CAPTION_BUTTON_ICON_CLOSE, gfx::VectorIconId::WINDOW_CONTROL_CLOSE);
+
+ caption_button_container_->SetButtonImage(
+ CAPTION_BUTTON_ICON_LEFT_SNAPPED,
+ gfx::VectorIconId::WINDOW_CONTROL_LEFT_SNAPPED);
+
+ caption_button_container_->SetButtonImage(
+ CAPTION_BUTTON_ICON_RIGHT_SNAPPED,
+ gfx::VectorIconId::WINDOW_CONTROL_RIGHT_SNAPPED);
}
void DefaultHeaderPainter::UpdateSizeButtonImages() {
- const gfx::VectorIcon& icon = frame_->IsMaximized() || frame_->IsFullscreen()
- ? kWindowControlRestoreIcon
- : kWindowControlMaximizeIcon;
- caption_button_container_->SetButtonImage(
- CAPTION_BUTTON_ICON_MAXIMIZE_RESTORE, icon);
+ gfx::VectorIconId icon_id = frame_->IsMaximized() || frame_->IsFullscreen()
+ ? gfx::VectorIconId::WINDOW_CONTROL_RESTORE
+ : gfx::VectorIconId::WINDOW_CONTROL_MAXIMIZE;
+ caption_button_container_->SetButtonImage(
+ CAPTION_BUTTON_ICON_MAXIMIZE_RESTORE, icon_id);
}
gfx::Rect DefaultHeaderPainter::GetLocalBounds() const {

Powered by Google App Engine
This is Rietveld 408576698