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

Unified Diff: chrome/browser/ui/views/frame/browser_header_painter_ash.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
« 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 c61cb3f2a5dfdf8887c3eb55399de76a97f5795b..45d2a5591a403b26047eb89804db2c7e09b24e2b 100644
--- a/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
+++ b/chrome/browser/ui/views/frame/browser_header_painter_ash.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/logging.h" // DCHECK
#include "chrome/browser/themes/theme_properties.h"
#include "chrome/browser/ui/browser.h"
@@ -23,6 +22,7 @@
#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,27 +285,29 @@
}
void BrowserHeaderPainterAsh::UpdateCaptionButtons() {
- 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_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_LEFT_SNAPPED,
- ash::kWindowControlLeftSnappedIcon);
+ gfx::VectorIconId::WINDOW_CONTROL_LEFT_SNAPPED);
caption_button_container_->SetButtonImage(
ash::CAPTION_BUTTON_ICON_RIGHT_SNAPPED,
- ash::kWindowControlRightSnappedIcon);
-
- const gfx::VectorIcon* size_icon = &ash::kWindowControlMaximizeIcon;
+ gfx::VectorIconId::WINDOW_CONTROL_RIGHT_SNAPPED);
+
+ gfx::VectorIconId size_icon_id = gfx::VectorIconId::WINDOW_CONTROL_MAXIMIZE;
gfx::Size button_size(
GetAshLayoutSize(AshLayoutSize::BROWSER_RESTORED_CAPTION_BUTTON));
if (frame_->IsMaximized() || frame_->IsFullscreen()) {
- size_icon = &ash::kWindowControlRestoreIcon;
+ size_icon_id = gfx::VectorIconId::WINDOW_CONTROL_RESTORE;
button_size =
GetAshLayoutSize(AshLayoutSize::BROWSER_MAXIMIZED_CAPTION_BUTTON);
}
caption_button_container_->SetButtonImage(
- ash::CAPTION_BUTTON_ICON_MAXIMIZE_RESTORE, *size_icon);
+ ash::CAPTION_BUTTON_ICON_MAXIMIZE_RESTORE,
+ size_icon_id);
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