aura: Use the WindowSlider for gesture-nav while the page is reloading.

content/browser/web_contents/aura/window_slider.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/web_contents/aura/window_slider.h"
 
+#include <algorithm>
+
 #include "base/bind.h"
 #include "base/callback.h"
 #include "content/browser/web_contents/aura/shadow_layer_delegate.h"
 #include "content/public/browser/overscroll_configuration.h"
 #include "ui/aura/window.h"
 #include "ui/base/events/event.h"
 #include "ui/compositor/layer_animation_observer.h"
 #include "ui/compositor/scoped_layer_animation_settings.h"
 
 namespace content {
@@ skipping 62 matching lines @@
 void WindowSlider::ChangeOwner(aura::Window* new_owner) {
   if (owner_)
     owner_->RemoveObserver(this);
   owner_ = new_owner;
   if (owner_) {
     owner_->AddObserver(this);
     UpdateForScroll(0.f, 0.f);
   }
 }
 
+bool WindowSlider::SlideIsInProgress() const {
+  return fabs(delta_x_) >= min_start_threshold_ || weak_factory_.HasWeakPtrs();
+}
+
 void WindowSlider::SetupSliderLayer() {
   ui::Layer* parent = owner_->layer()->parent();
   parent->Add(slider_.get());
   if (delta_x_ < 0)
     parent->StackAbove(slider_.get(), owner_->layer());
   else
     parent->StackBelow(slider_.get(), owner_->layer());
   slider_->SetBounds(owner_->layer()->bounds());
   slider_->SetVisible(true);
 }
 
 void WindowSlider::UpdateForScroll(float x_offset, float y_offset) {
   float old_delta = delta_x_;
   delta_x_ += x_offset;
-  if (fabs(delta_x_) < min_start_threshold_) {
-    ResetScroll();
+  if (fabs(delta_x_) < min_start_threshold_ && !slider_.get())
     return;
-  }
 
   if ((old_delta < 0 && delta_x_ > 0) ||
       (old_delta > 0 && delta_x_ < 0)) {
     slider_.reset();
     shadow_.reset();
   }
 
   float translate = 0.f;
   ui::Layer* translate_layer = NULL;
 
   if (delta_x_ <= -min_start_threshold_) {
     if (!slider_.get()) {
       slider_.reset(delegate_->CreateFrontLayer());
+      if (!slider_.get())
+        return;
       SetupSliderLayer();
     }
-    translate = event_window_->bounds().width() -
-        fabs(delta_x_ - min_start_threshold_);
+    translate = owner_->bounds().width() +
+        std::max(delta_x_ + min_start_threshold_,
+                 static_cast<float>(-owner_->bounds().width()));
     translate_layer = slider_.get();
   } else if (delta_x_ >= min_start_threshold_) {
     if (!slider_.get()) {
       slider_.reset(delegate_->CreateBackLayer());
+      if (!slider_.get())
+        return;
       SetupSliderLayer();
     }
-    translate = delta_x_ - min_start_threshold_;
+    translate = std::min(delta_x_ - min_start_threshold_,
+                         static_cast<float>(owner_->bounds().width()));
     translate_layer = owner_->layer();
   } else {
     NOTREACHED();
   }
 
   if (!shadow_.get())
     shadow_.reset(new ShadowLayerDelegate(translate_layer));
 
   gfx::Transform transform;
   transform.Translate(translate, 0);
@@ skipping 66 matching lines @@
   }
 
   delta_x_ = 0.f;
 }
 
 void WindowSlider::CancelScroll() {
   ResetScroll();
 }
 
 void WindowSlider::CompleteWindowSlideAfterAnimation() {
+  // The delegate may delete the |owner_| from the |OnWindowSlideComplete()|
+  // callback, which would trigger the
+  // |WindowSlider::OnWindowRemovingFromRootWindow()| callback, which would try
+  // to delete itself again. So avoid that by resetting |owner_| before calling
+  // the |OnWindowSlideComplete()| callback on the delegate.
+  owner_->RemoveObserver(this);
+  owner_ = NULL;
+
   delegate_->OnWindowSlideComplete();
   delete this;
 }
 
 void WindowSlider::AbortWindowSlideAfterAnimation() {
   delegate_->OnWindowSlideAborted();
 }
 
 void WindowSlider::OnKeyEvent(ui::KeyEvent* event) {
   CancelScroll();
@@ skipping 55 matching lines @@
     window->RemoveObserver(this);
     owner_ = NULL;
     if (!slider_.get())
       delete this;
   } else {
     NOTREACHED();
   }
 }
 
 }  // namespace content