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

Unified Diff: chrome/browser/ui/views/frame/browser_header_painter_ash.cc

Issue 2620613005: 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
« no previous file with comments | « chrome/browser/chromeos/options/wimax_config_view.cc ('k') | ui/gfx/vector_icons/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/frame/browser_header_painter_ash.cc
diff --git a/chrome/browser/ui/views/frame/browser_header_painter_ash.cc b/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
index 45d2a5591a403b26047eb89804db2c7e09b24e2b..c61cb3f2a5dfdf8887c3eb55399de76a97f5795b 100644
--- a/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
+++ b/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
@@ -7,6 +7,7 @@
#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/logging.h" // DCHECK
#include "chrome/browser/themes/theme_properties.h"
#include "chrome/browser/ui/browser.h"
@@ -22,7 +23,6 @@
#include "ui/gfx/geometry/rect.h"
#include "ui/gfx/image/image_skia.h"
#include "ui/gfx/skia_util.h"
-#include "ui/gfx/vector_icons_public.h"
#include "ui/views/view.h"
#include "ui/views/widget/widget.h"
#include "ui/views/widget/widget_delegate.h"
@@ -285,29 +285,27 @@ void BrowserHeaderPainterAsh::PaintTitleBar(gfx::Canvas* canvas) {
}
void BrowserHeaderPainterAsh::UpdateCaptionButtons() {
- caption_button_container_->SetButtonImage(
- ash::CAPTION_BUTTON_ICON_MINIMIZE,
- gfx::VectorIconId::WINDOW_CONTROL_MINIMIZE);
- caption_button_container_->SetButtonImage(
- ash::CAPTION_BUTTON_ICON_CLOSE, gfx::VectorIconId::WINDOW_CONTROL_CLOSE);
+ caption_button_container_->SetButtonImage(ash::CAPTION_BUTTON_ICON_MINIMIZE,
+ ash::kWindowControlMinimizeIcon);
+ caption_button_container_->SetButtonImage(ash::CAPTION_BUTTON_ICON_CLOSE,
+ ash::kWindowControlCloseIcon);
caption_button_container_->SetButtonImage(
ash::CAPTION_BUTTON_ICON_LEFT_SNAPPED,
- gfx::VectorIconId::WINDOW_CONTROL_LEFT_SNAPPED);
+ ash::kWindowControlLeftSnappedIcon);
caption_button_container_->SetButtonImage(
ash::CAPTION_BUTTON_ICON_RIGHT_SNAPPED,
- gfx::VectorIconId::WINDOW_CONTROL_RIGHT_SNAPPED);
+ ash::kWindowControlRightSnappedIcon);
- gfx::VectorIconId size_icon_id = gfx::VectorIconId::WINDOW_CONTROL_MAXIMIZE;
+ const gfx::VectorIcon* size_icon = &ash::kWindowControlMaximizeIcon;
gfx::Size button_size(
GetAshLayoutSize(AshLayoutSize::BROWSER_RESTORED_CAPTION_BUTTON));
if (frame_->IsMaximized() || frame_->IsFullscreen()) {
- size_icon_id = gfx::VectorIconId::WINDOW_CONTROL_RESTORE;
+ size_icon = &ash::kWindowControlRestoreIcon;
button_size =
GetAshLayoutSize(AshLayoutSize::BROWSER_MAXIMIZED_CAPTION_BUTTON);
}
caption_button_container_->SetButtonImage(
- ash::CAPTION_BUTTON_ICON_MAXIMIZE_RESTORE,
- size_icon_id);
+ ash::CAPTION_BUTTON_ICON_MAXIMIZE_RESTORE, *size_icon);
caption_button_container_->SetButtonSize(button_size);
}
« no previous file with comments | « chrome/browser/chromeos/options/wimax_config_view.cc ('k') | ui/gfx/vector_icons/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698