Commit 5db22213 authored by Mark Cogan's avatar Mark Cogan Committed by Commit Bot

[iOS] Move infobar close image

Move infobar_close into downloads/, which is the only place where it's
used, and rename it.

Bug: 884722
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
Change-Id: I539058203790b7ba7b0e28e94e989ce32e0e95fd
Reviewed-on: https://chromium-review.googlesource.com/1248704
Commit-Queue: Mark Cogan <marq@chromium.org>
Reviewed-by: default avatarGauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#594708}
parent eca64c98
...@@ -26,6 +26,7 @@ source_set("download") { ...@@ -26,6 +26,7 @@ source_set("download") {
"resources:background_compact", "resources:background_compact",
"resources:background_regular", "resources:background_regular",
"resources:done_badge", "resources:done_badge",
"resources:download_close",
"resources:error_badge", "resources:error_badge",
"//base", "//base",
"//components/infobars/core", "//components/infobars/core",
......
...@@ -419,8 +419,7 @@ NSString* GetSizeString(long long size_in_bytes) { ...@@ -419,8 +419,7 @@ NSString* GetSizeString(long long size_in_bytes) {
_closeButton.exclusiveTouch = YES; _closeButton.exclusiveTouch = YES;
_closeButton.accessibilityLabel = l10n_util::GetNSString(IDS_CLOSE); _closeButton.accessibilityLabel = l10n_util::GetNSString(IDS_CLOSE);
// TODO(crbug.com/228611): Add IDR_ constant and use GetNativeImageNamed(). UIImage* image = [UIImage imageNamed:@"download_close"];
UIImage* image = [UIImage imageNamed:@"infobar_close"];
[_closeButton setImage:image forState:UIControlStateNormal]; [_closeButton setImage:image forState:UIControlStateNormal];
[_closeButton addTarget:self [_closeButton addTarget:self
......
...@@ -30,7 +30,14 @@ imageset("done_badge") { ...@@ -30,7 +30,14 @@ imageset("done_badge") {
"done_badge.imageset/done_badge@3x.png", "done_badge.imageset/done_badge@3x.png",
] ]
} }
imageset("download_close") {
sources = [
"download_close.imageset/Contents.json",
"download_close.imageset/download_close.png",
"download_close.imageset/download_close@2x.png",
"download_close.imageset/download_close@3x.png",
]
}
imageset("error_badge") { imageset("error_badge") {
sources = [ sources = [
"error_badge.imageset/Contents.json", "error_badge.imageset/Contents.json",
......
...@@ -3,17 +3,17 @@ ...@@ -3,17 +3,17 @@
{ {
"idiom": "universal", "idiom": "universal",
"scale": "1x", "scale": "1x",
"filename": "infobar_close.png" "filename": "download_close.png"
}, },
{ {
"idiom": "universal", "idiom": "universal",
"scale": "2x", "scale": "2x",
"filename": "infobar_close@2x.png" "filename": "download_close@2x.png"
}, },
{ {
"idiom": "universal", "idiom": "universal",
"scale": "3x", "scale": "3x",
"filename": "infobar_close@3x.png" "filename": "download_close@3x.png"
} }
], ],
"info": { "info": {
......
...@@ -13,7 +13,6 @@ source_set("infobars") { ...@@ -13,7 +13,6 @@ source_set("infobars") {
"infobar_view_sizing_delegate.h", "infobar_view_sizing_delegate.h",
] ]
deps = [ deps = [
"resources:infobar_close",
"resources:infobar_downloading", "resources:infobar_downloading",
"resources:infobar_popup_blocker", "resources:infobar_popup_blocker",
"resources:infobar_warning", "resources:infobar_warning",
......
...@@ -22,15 +22,6 @@ imageset("infobar_downloading") { ...@@ -22,15 +22,6 @@ imageset("infobar_downloading") {
] ]
} }
imageset("infobar_close") {
sources = [
"infobar_close.imageset/Contents.json",
"infobar_close.imageset/infobar_close.png",
"infobar_close.imageset/infobar_close@2x.png",
"infobar_close.imageset/infobar_close@3x.png",
]
}
imageset("infobar_popup_blocker") { imageset("infobar_popup_blocker") {
sources = [ sources = [
"infobar_popup_blocker.imageset/Contents.json", "infobar_popup_blocker.imageset/Contents.json",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment