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

Unified Diff: ui/wm/core/shadow.cc

Issue 2596743002: Replace WM shadow types (on/off) and styles (small/inactive/active) (Closed)
Patch Set: one more mechanical change Created 3 years, 11 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
« no previous file with comments | « ui/wm/core/shadow.h ('k') | ui/wm/core/shadow_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/wm/core/shadow.cc
diff --git a/ui/wm/core/shadow.cc b/ui/wm/core/shadow.cc
index 22e12576f7234ec7f65f32219c5ad0a6835d2719..31737aa37e20b0b12516720e981d7eb3972da0c0 100644
--- a/ui/wm/core/shadow.cc
+++ b/ui/wm/core/shadow.cc
@@ -10,6 +10,7 @@
#include "ui/compositor/scoped_layer_animation_settings.h"
#include "ui/gfx/geometry/insets.h"
#include "ui/gfx/image/image_skia_operations.h"
+#include "ui/wm/core/shadow_types.h"
namespace wm {
@@ -63,12 +64,12 @@ const ShadowDetails& GetDetailsForElevation(int elevation) {
} // namespace
-Shadow::Shadow() {}
+Shadow::Shadow() : desired_elevation_(ShadowElevation::NONE) {}
Shadow::~Shadow() {}
-void Shadow::Init(Style style) {
- style_ = style;
+void Shadow::Init(ShadowElevation elevation) {
+ desired_elevation_ = elevation;
layer_.reset(new ui::Layer(ui::LAYER_NOT_DRAWN));
RecreateShadowLayer();
}
@@ -83,11 +84,11 @@ void Shadow::SetContentBounds(const gfx::Rect& content_bounds) {
UpdateLayerBounds();
}
-void Shadow::SetStyle(Style style) {
- if (style_ == style)
+void Shadow::SetElevation(ShadowElevation elevation) {
+ if (desired_elevation_ == elevation)
return;
- style_ = style;
+ desired_elevation_ = elevation;
// Stop waiting for any as yet unfinished implicit animations.
StopObservingImplicitAnimations();
@@ -137,14 +138,14 @@ void Shadow::UpdateLayerBounds() {
if (content_bounds_.IsEmpty())
return;
- // The elevation depends on the style, but the ninebox assumption breaks down
- // when the window is too small. The height/width of |blur_region| will be
- // 4 * elevation (see GetDetailsForElevation), so cap elevation at the most we
- // can handle.
+ // The ninebox assumption breaks down when the window is too small for the
+ // desired elevation. The height/width of |blur_region| will be 4 * elevation
+ // (see GetDetailsForElevation), so cap elevation at the most we can handle.
const int smaller_dimension =
std::min(content_bounds_.width(), content_bounds_.height());
- const int size_adjusted_elevation = std::min(
- (smaller_dimension - 2 * kRoundedCornerRadius) / 4, ElevationForStyle());
+ const int size_adjusted_elevation =
+ std::min((smaller_dimension - 2 * kRoundedCornerRadius) / 4,
+ static_cast<int>(desired_elevation_));
const ShadowDetails& details =
GetDetailsForElevation(size_adjusted_elevation);
gfx::Insets blur_region = gfx::ShadowValue::GetBlurRegion(details.values) +
@@ -204,17 +205,4 @@ void Shadow::UpdateLayerBounds() {
blur_region.height()));
}
-int Shadow::ElevationForStyle() {
- switch (style_) {
- case STYLE_ACTIVE:
- return 24;
- case STYLE_INACTIVE:
- return 8;
- case STYLE_SMALL:
- return 6;
- }
- NOTREACHED();
- return 0;
-}
-
} // namespace wm
« no previous file with comments | « ui/wm/core/shadow.h ('k') | ui/wm/core/shadow_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698