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

Unified Diff: extensions/common/constants.cc

Issue 1492073003: Handle more scale factors for extension Browser Action icons (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: test catches real bug! Created 5 years 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 | « extensions/common/constants.h ('k') | extensions/common/manifest_handler_helpers.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/common/constants.cc
diff --git a/extensions/common/constants.cc b/extensions/common/constants.cc
index ff10df9a9119c42088b2d884d582f67cf13ca40f..8d079b2e5a864afdf7683c60f82be1d5f236fd64 100644
--- a/extensions/common/constants.cc
+++ b/extensions/common/constants.cc
@@ -75,15 +75,6 @@ const int kExtensionIconSizes[] = {EXTENSION_ICON_GIGANTOR, // 512
const size_t kNumExtensionIconSizes = arraysize(kExtensionIconSizes);
-const IconRepresentationInfo kExtensionActionIconSizes[] = {
- { EXTENSION_ICON_ACTION, "19", ui::SCALE_FACTOR_100P },
- { 2 * EXTENSION_ICON_ACTION, "38", ui::SCALE_FACTOR_200P }
-};
-
-static_assert(kNumExtensionActionIconSizes ==
- arraysize(kExtensionActionIconSizes),
- "num action icon sizes must be in sync with action icon sizes");
-
const char kPdfExtensionId[] = "mhjfbmdgcfjbbpaeojofohoefgiehjai";
const char kQuickOfficeComponentExtensionId[] =
"bpmcpldpdmajfigpchkicefoigmkfalc";
« no previous file with comments | « extensions/common/constants.h ('k') | extensions/common/manifest_handler_helpers.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698