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

Issue 12703022: Inconsistent use of [x] close panel icon. Adding binaries. (Closed)

Created:
7 years, 9 months ago by varkha
Modified:
7 years, 9 months ago
Reviewers:
flackr, sky
CC:
chromium-reviews, oshima+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@issue_173251
Visibility:
Public.

Description

Adding binaries for new close panel icon names Adding the binaries for https://codereview.chromium.org/12742013/ TBR=sky BUG=217094 TEST=None Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=190485

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -0 lines) Patch
A + chrome/app/theme/default_100_percent/common/close_1.png View Binary file 0 comments Download
A + chrome/app/theme/default_100_percent/common/close_1_hover.png View Binary file 0 comments Download
A + chrome/app/theme/default_100_percent/common/close_1_mask.png View Binary file 0 comments Download
A + chrome/app/theme/default_100_percent/common/close_1_pressed.png View Binary file 0 comments Download
A + chrome/app/theme/default_200_percent/common/close_1.png View Binary file 0 comments Download
A + chrome/app/theme/default_200_percent/common/close_1_hover.png View Binary file 0 comments Download
A + chrome/app/theme/default_200_percent/common/close_1_mask.png View Binary file 0 comments Download
A + chrome/app/theme/default_200_percent/common/close_1_pressed.png View Binary file 0 comments Download
A + chrome/app/theme/touch_100_percent/common/close_1.png View Binary file 0 comments Download
A + chrome/app/theme/touch_100_percent/common/close_1_hover.png View Binary file 0 comments Download
A + chrome/app/theme/touch_100_percent/common/close_1_mask.png View Binary file 0 comments Download
A + chrome/app/theme/touch_100_percent/common/close_1_pressed.png View Binary file 0 comments Download
A + chrome/app/theme/touch_140_percent/common/close_1.png View Binary file 0 comments Download
A + chrome/app/theme/touch_140_percent/common/close_1_hover.png View Binary file 0 comments Download
A + chrome/app/theme/touch_140_percent/common/close_1_mask.png View Binary file 0 comments Download
A + chrome/app/theme/touch_140_percent/common/close_1_pressed.png View Binary file 0 comments Download
A + chrome/app/theme/touch_180_percent/common/close_1.png View Binary file 0 comments Download
A + chrome/app/theme/touch_180_percent/common/close_1_hover.png View Binary file 0 comments Download
A + chrome/app/theme/touch_180_percent/common/close_1_mask.png View Binary file 0 comments Download
A + chrome/app/theme/touch_180_percent/common/close_1_pressed.png View Binary file 0 comments Download
A + ui/resources/default_100_percent/close_2.png View Binary file 0 comments Download
A + ui/resources/default_100_percent/close_2_hover.png View Binary file 0 comments Download
A + ui/resources/default_100_percent/close_2_mask.png View Binary file 0 comments Download
A + ui/resources/default_100_percent/close_2_pressed.png View Binary file 0 comments Download
A + ui/resources/default_200_percent/close_2.png View Binary file 0 comments Download
A + ui/resources/default_200_percent/close_2_hover.png View Binary file 0 comments Download
A + ui/resources/default_200_percent/close_2_mask.png View Binary file 0 comments Download
A + ui/resources/default_200_percent/close_2_pressed.png View Binary file 0 comments Download

Messages

Total messages: 6 (0 generated)
varkha
This is the first part of the 3-step commit needed for https://codereview.chromium.org/12634025
7 years, 9 months ago (2013-03-25 20:23:39 UTC) #1
flackr
Since this CL is just landing the resources for the main one and is untestable, ...
7 years, 9 months ago (2013-03-25 21:18:20 UTC) #2
varkha
Done
7 years, 9 months ago (2013-03-25 21:21:16 UTC) #3
flackr
Adding TBR=sky and landing new binaries so that the main change can be landed through ...
7 years, 9 months ago (2013-03-25 21:34:21 UTC) #4
flackr
lgtm
7 years, 9 months ago (2013-03-25 21:35:01 UTC) #5
flackr
7 years, 9 months ago (2013-03-25 21:38:09 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r190485 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698