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

Unified Diff: chrome/browser/extensions/extension_action.cc

Issue 2447553003: More pre-MD odds and ends. (Closed)
Patch Set: make test work Created 4 years, 1 month 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/extensions/extension_action.cc
diff --git a/chrome/browser/extensions/extension_action.cc b/chrome/browser/extensions/extension_action.cc
index 4eeab31b8d24956295364a277fffd8c1c5b3ab1b..b10c48e16526381bc632a7a9502ac9c13c9fd34e 100644
--- a/chrome/browser/extensions/extension_action.cc
+++ b/chrome/browser/extensions/extension_action.cc
@@ -23,7 +23,6 @@
#include "third_party/skia/include/core/SkCanvas.h"
#include "third_party/skia/include/core/SkPaint.h"
#include "third_party/skia/include/effects/SkGradientShader.h"
-#include "ui/base/material_design/material_design_controller.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/gfx/animation/animation_delegate.h"
#include "ui/gfx/canvas.h"
@@ -79,9 +78,7 @@ bool HasValue(const std::map<int, T>& map, int tab_id) {
} // namespace
extension_misc::ExtensionIcons ExtensionAction::ActionIconSize() {
- return ui::MaterialDesignController::IsModeMaterial()
- ? extension_misc::EXTENSION_ICON_BITTY
- : extension_misc::EXTENSION_ICON_ACTION;
+ return extension_misc::EXTENSION_ICON_BITTY;
}
const int ExtensionAction::kDefaultTabId = -1;
@@ -148,7 +145,7 @@ bool ExtensionAction::ParseIconFromCanvasDictionary(
CHECK(!bitmap.isNull());
// Chrome helpfully scales the provided icon(s), but let's not go overboard.
- const int kActionIconMaxSize = 10 * extension_misc::EXTENSION_ICON_ACTION;
+ const int kActionIconMaxSize = 10 * ActionIconSize();
if (bitmap.drawsNothing() || bitmap.width() > kActionIconMaxSize)
continue;

Powered by Google App Engine
This is Rietveld 408576698