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

Unified Diff: chrome/browser/ui/views/toolbar/toolbar_view.cc

Issue 2618403002: Reland crrev.com/8283cad74e0cad4840d1f with fix for static initializers. (Closed)
Patch Set: re-up 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/ui/views/toolbar/reload_button.cc ('k') | chrome/browser/upgrade_detector.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/toolbar/toolbar_view.cc
diff --git a/chrome/browser/ui/views/toolbar/toolbar_view.cc b/chrome/browser/ui/views/toolbar/toolbar_view.cc
index 96191078ce655869c0b1360798d915037b4072c9..63d495991caf12f8bb9b7bbfb513e845f3d26f89 100644
--- a/chrome/browser/ui/views/toolbar/toolbar_view.cc
+++ b/chrome/browser/ui/views/toolbar/toolbar_view.cc
@@ -12,6 +12,7 @@
#include "base/trace_event/trace_event.h"
#include "build/build_config.h"
#include "chrome/app/chrome_command_ids.h"
+#include "chrome/app/vector_icons/vector_icons.h"
#include "chrome/browser/chrome_notification_types.h"
#include "chrome/browser/command_updater.h"
#include "chrome/browser/extensions/extension_util.h"
@@ -68,7 +69,6 @@
#include "ui/gfx/canvas.h"
#include "ui/gfx/image/canvas_image_source.h"
#include "ui/gfx/paint_vector_icon.h"
-#include "ui/gfx/vector_icons_public.h"
#include "ui/keyboard/keyboard_controller.h"
#include "ui/native_theme/native_theme_aura.h"
#include "ui/views/focus/view_storage.h"
@@ -748,19 +748,18 @@ void ToolbarView::LoadImages() {
back_->SetImage(
views::Button::STATE_NORMAL,
- gfx::CreateVectorIcon(gfx::VectorIconId::NAVIGATE_BACK, normal_color));
+ gfx::CreateVectorIcon(kNavigateBackIcon, normal_color));
back_->SetImage(
views::Button::STATE_DISABLED,
- gfx::CreateVectorIcon(gfx::VectorIconId::NAVIGATE_BACK, disabled_color));
Evan Stade 2017/01/11 02:25:42 oops
+ gfx::CreateVectorIcon(kNavigateBackIcon, normal_color));
forward_->SetImage(
views::Button::STATE_NORMAL,
- gfx::CreateVectorIcon(gfx::VectorIconId::NAVIGATE_FORWARD, normal_color));
- forward_->SetImage(views::Button::STATE_DISABLED,
- gfx::CreateVectorIcon(gfx::VectorIconId::NAVIGATE_FORWARD,
- disabled_color));
- home_->SetImage(
- views::Button::STATE_NORMAL,
- gfx::CreateVectorIcon(gfx::VectorIconId::NAVIGATE_HOME, normal_color));
+ gfx::CreateVectorIcon(kNavigateForwardIcon, normal_color));
+ forward_->SetImage(
+ views::Button::STATE_DISABLED,
+ gfx::CreateVectorIcon(kNavigateForwardIcon, disabled_color));
+ home_->SetImage(views::Button::STATE_NORMAL,
+ gfx::CreateVectorIcon(kNavigateHomeIcon, normal_color));
app_menu_button_->UpdateIcon();
back_->set_ink_drop_base_color(normal_color);
« no previous file with comments | « chrome/browser/ui/views/toolbar/reload_button.cc ('k') | chrome/browser/upgrade_detector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698