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

Unified Diff: ash/wm/window_animations.cc

Issue 10869066: Attempt 2 at Fixes crash introduced @ 153047 (you can hit crash by maximizing a (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Better fix Created 8 years, 3 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: ash/wm/window_animations.cc
diff --git a/ash/wm/window_animations.cc b/ash/wm/window_animations.cc
index 387d44777d2bc07cb73a57688c9472d6ebb256ad..48afe8d8ee2de16e5199dad83bef89c9a869c1bd 100644
--- a/ash/wm/window_animations.cc
+++ b/ash/wm/window_animations.cc
@@ -50,7 +50,6 @@ namespace internal {
namespace {
const float kWindowAnimation_Vertical_TranslateY = 15.f;
-bool delayed_old_layer_deletion_in_cross_fade_for_test_ = false;
}
DEFINE_WINDOW_PROPERTY_KEY(WindowVisibilityAnimationType,
@@ -661,13 +660,7 @@ class CrossFadeObserver : public ui::CompositorObserver,
// ui::ImplicitAnimationObserver overrides:
virtual void OnImplicitAnimationsCompleted() OVERRIDE {
- // ImplicitAnimationObserver's base class uses the object after
- // calling this function, so we cannot delete |this|. The |layer_|
- // may be gone by the next message loop run when shutting down, so
- // clean them up now.
- if (!delayed_old_layer_deletion_in_cross_fade_for_test_)
- Cleanup();
- MessageLoop::current()->DeleteSoon(FROM_HERE, this);
+ delete this;
}
private:
@@ -974,9 +967,5 @@ bool AnimateOnChildWindowVisibilityChanged(aura::Window* window, bool visible) {
}
}
-void SetDelayedOldLayerDeletionInCrossFadeForTest(bool value) {
- delayed_old_layer_deletion_in_cross_fade_for_test_ = value;
-}
-
} // namespace internal
} // namespace ash

Powered by Google App Engine
This is Rietveld 408576698