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

Unified Diff: ash/frame/caption_buttons/frame_caption_button_container_view.cc

Issue 934503002: Ash frames: dark frames now use white caption buttons. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 10 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/frame/caption_buttons/frame_caption_button_container_view.cc
diff --git a/ash/frame/caption_buttons/frame_caption_button_container_view.cc b/ash/frame/caption_buttons/frame_caption_button_container_view.cc
index 089090702e1f1b0bc77b0048b3433e51b061973e..686fa08afac402186b3025302e154c57b22c5c5f 100644
--- a/ash/frame/caption_buttons/frame_caption_button_container_view.cc
+++ b/ash/frame/caption_buttons/frame_caption_button_container_view.cc
@@ -151,11 +151,9 @@ void FrameCaptionButtonContainerView::TestApi::EndAnimations() {
void FrameCaptionButtonContainerView::SetButtonImages(
CaptionButtonIcon icon,
int icon_image_id,
- int inactive_icon_image_id,
int hovered_background_image_id,
int pressed_background_image_id) {
button_icon_id_map_[icon] = ButtonIconIds(icon_image_id,
- inactive_icon_image_id,
hovered_background_image_id,
pressed_background_image_id);
FrameCaptionButton* buttons[] = {
@@ -166,7 +164,6 @@ void FrameCaptionButtonContainerView::SetButtonImages(
buttons[i]->SetImages(icon,
FrameCaptionButton::ANIMATE_NO,
icon_image_id,
- inactive_icon_image_id,
hovered_background_image_id,
pressed_background_image_id);
}
@@ -307,7 +304,6 @@ void FrameCaptionButtonContainerView::SetButtonIcon(FrameCaptionButton* button,
button->SetImages(icon,
fcb_animate,
it->second.icon_image_id,
- it->second.inactive_icon_image_id,
it->second.hovered_background_image_id,
it->second.pressed_background_image_id);
}
@@ -418,18 +414,15 @@ void FrameCaptionButtonContainerView::SetHoveredAndPressedButtons(
FrameCaptionButtonContainerView::ButtonIconIds::ButtonIconIds()
: icon_image_id(-1),
- inactive_icon_image_id(-1),
hovered_background_image_id(-1),
pressed_background_image_id(-1) {
}
FrameCaptionButtonContainerView::ButtonIconIds::ButtonIconIds(
int icon_id,
- int inactive_icon_id,
int hovered_background_id,
int pressed_background_id)
: icon_image_id(icon_id),
- inactive_icon_image_id(inactive_icon_id),
hovered_background_image_id(hovered_background_id),
pressed_background_image_id(pressed_background_id) {
}

Powered by Google App Engine
This is Rietveld 408576698