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

Unified Diff: chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm

Issue 10660021: HiDPI assets for omnibox icons. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: cros Created 8 years, 6 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: chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
diff --git a/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm b/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
index 1aaf42190e881e70210ac4c60875c7e3ae630146..6f9010f2f6f23beccfc8a2423eaba31b88ac3163 100644
--- a/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
+++ b/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
@@ -130,35 +130,6 @@ NSRange ComponentToNSRange(const url_parse::Component& component) {
// static
NSImage* OmniboxViewMac::ImageForResource(int resource_id) {
- NSString* image_name = nil;
-
- switch(resource_id) {
- // Values from |AutocompleteMatch::TypeToIcon()|.
- case IDR_OMNIBOX_SEARCH:
- image_name = @"omnibox_search.pdf"; break;
- case IDR_OMNIBOX_HTTP:
- image_name = @"omnibox_http.pdf"; break;
- case IDR_OMNIBOX_EXTENSION_APP:
- image_name = @"omnibox_extension_app.pdf"; break;
-
- // Values from |ToolbarModel::GetIcon()|.
- case IDR_OMNIBOX_HTTPS_VALID:
- image_name = @"omnibox_https_valid.pdf"; break;
- case IDR_OMNIBOX_HTTPS_WARNING:
- image_name = @"omnibox_https_warning.pdf"; break;
- case IDR_OMNIBOX_HTTPS_INVALID:
- image_name = @"omnibox_https_invalid.pdf"; break;
- }
-
- if (image_name) {
- if (NSImage* image = gfx::GetCachedImageWithName(image_name)) {
- return image;
- } else {
- NOTREACHED()
- << "Missing image for " << base::SysNSStringToUTF8(image_name);
- }
- }
-
ResourceBundle& rb = ResourceBundle::GetSharedInstance();
return rb.GetNativeImageNamed(resource_id);
}
« no previous file with comments | « chrome/browser/ui/cocoa/omnibox/omnibox_popup_view_mac.mm ('k') | chrome/browser/ui/gtk/omnibox/omnibox_popup_view_gtk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698