summaryrefslogtreecommitdiffstats
path: root/ui/views
diff options
context:
space:
mode:
authorvollick@google.com <vollick@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2012-01-24 15:01:06 +0000
committervollick@google.com <vollick@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2012-01-24 15:01:06 +0000
commit5fc6e67abec594943b321e373858441d3c60972c (patch)
tree86b72efeb13ecd644cd0f505e7364c7def38d968 /ui/views
parent6baec9fddef0cff918a0815e3af0e9d7a85a4e97 (diff)
downloadchromium_src-5fc6e67abec594943b321e373858441d3c60972c.zip
chromium_src-5fc6e67abec594943b321e373858441d3c60972c.tar.gz
chromium_src-5fc6e67abec594943b321e373858441d3c60972c.tar.bz2
Revert 118834 - Support touch scroll gestures in menus.
Depends on http://codereview.chromium.org/8364039/ BUG=None TEST=None Review URL: https://chromiumcodereview.appspot.com/8508024 TBR=vollick@google.com Review URL: https://chromiumcodereview.appspot.com/9284024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118835 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui/views')
-rw-r--r--ui/views/animation/scroll_animator.cc82
-rw-r--r--ui/views/animation/scroll_animator.h63
-rw-r--r--ui/views/controls/menu/menu_controller.cc8
-rw-r--r--ui/views/controls/menu/menu_controller.h2
-rw-r--r--ui/views/controls/menu/menu_host_root_view.cc10
-rw-r--r--ui/views/controls/menu/menu_host_root_view.h3
-rw-r--r--ui/views/controls/menu/submenu_view.cc42
-rw-r--r--ui/views/controls/menu/submenu_view.h15
-rw-r--r--ui/views/views.gyp2
9 files changed, 6 insertions, 221 deletions
diff --git a/ui/views/animation/scroll_animator.cc b/ui/views/animation/scroll_animator.cc
deleted file mode 100644
index 1e484a3..0000000
--- a/ui/views/animation/scroll_animator.cc
+++ /dev/null
@@ -1,82 +0,0 @@
-// Copyright (c) 2012 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 "ui/views/animation/scroll_animator.h"
-
-#include <algorithm>
-#include <cmath>
-
-#include "base/logging.h"
-#include "ui/base/animation/slide_animation.h"
-
-namespace {
-const float kDefaultAcceleration = -1500.0f; // in pixels per second^2
-
-// Assumes that d0 == 0.0f
-float GetPosition(float v0, float a, float t) {
- float max_t = -v0 / a;
- if (t > max_t)
- t = max_t;
- return t * (v0 + 0.5f * a * t);
-}
-
-float GetDelta(float v0, float a, float t1, float t2) {
- return GetPosition(v0, a, t2) - GetPosition(v0, a, t1);
-}
-
-} // namespace
-
-namespace views {
-
-ScrollAnimator::ScrollAnimator(ScrollDelegate* delegate)
- : delegate_(delegate),
- velocity_x_(0.0f),
- velocity_y_(0.0f),
- last_t_(0.0f),
- duration_(0.0f),
- acceleration_(kDefaultAcceleration) {
- DCHECK(delegate);
-}
-
-ScrollAnimator::~ScrollAnimator() {
- Stop();
-}
-
-void ScrollAnimator::Start(float velocity_x, float velocity_y) {
- if (acceleration_ >= 0.0f)
- acceleration_ = kDefaultAcceleration;
- float v = std::max(fabs(velocity_x), fabs(velocity_y));
- last_t_ = 0.0f;
- velocity_x_ = velocity_x;
- velocity_y_ = velocity_y;
- duration_ = -v / acceleration_; // in seconds
- animation_.reset(new ui::SlideAnimation(this));
- animation_->SetSlideDuration(static_cast<int>(duration_ * 1000));
- animation_->Show();
-}
-
-void ScrollAnimator::Stop() {
- velocity_x_ = velocity_y_ = last_t_ = duration_ = 0.0f;
- animation_.reset();
-}
-
-void ScrollAnimator::AnimationEnded(const ui::Animation* animation) {
- Stop();
-}
-
-void ScrollAnimator::AnimationProgressed(const ui::Animation* animation) {
- float t = static_cast<float>(animation->GetCurrentValue()) * duration_;
- float a_x = velocity_x_ > 0 ? acceleration_ : -acceleration_;
- float a_y = velocity_y_ > 0 ? acceleration_ : -acceleration_;
- float dx = GetDelta(velocity_x_, a_x, last_t_, t);
- float dy = GetDelta(velocity_y_, a_y, last_t_, t);
- last_t_ = t;
- delegate_->OnScroll(dx, dy);
-}
-
-void ScrollAnimator::AnimationCanceled(const ui::Animation* animation) {
- Stop();
-}
-
-} // namespace views
diff --git a/ui/views/animation/scroll_animator.h b/ui/views/animation/scroll_animator.h
deleted file mode 100644
index 6c3eb6d..0000000
--- a/ui/views/animation/scroll_animator.h
+++ /dev/null
@@ -1,63 +0,0 @@
-// Copyright (c) 2012 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.
-
-#ifndef UI_VIEWS_ANIMATION_SCROLL_ANIMATOR_H_
-#define UI_VIEWS_ANIMATION_SCROLL_ANIMATOR_H_
-#pragma once
-
-#include "base/basictypes.h"
-#include "base/memory/scoped_ptr.h"
-#include "ui/base/animation/animation_delegate.h"
-#include "ui/views/views_export.h"
-
-namespace ui {
-class SlideAnimation;
-}
-
-namespace views {
-
-class VIEWS_EXPORT ScrollDelegate {
- public:
- virtual void OnScroll(float dx, float dy) = 0;
-
- protected:
- ~ScrollDelegate() {}
-};
-
-class VIEWS_EXPORT ScrollAnimator : public ui::AnimationDelegate {
- public:
- // The ScrollAnimator does not own the delegate. Uses default acceleration.
- explicit ScrollAnimator(ScrollDelegate* delegate);
- virtual ~ScrollAnimator();
-
- // Use this if you would prefer different acceleration than the default.
- void set_acceleration(float acceleration) { acceleration_ = acceleration; }
-
- void Start(float velocity_x, float velocity_y);
- void Stop();
-
- bool is_scrolling() const { return animation_.get(); }
-
- private:
- // Implementation of ui::AnimationDelegate.
- virtual void AnimationEnded(const ui::Animation* animation) OVERRIDE;
- virtual void AnimationProgressed(const ui::Animation* animation) OVERRIDE;
- virtual void AnimationCanceled(const ui::Animation* animation) OVERRIDE;
-
- ScrollDelegate* delegate_;
-
- float velocity_x_;
- float velocity_y_;
- float last_t_;
- float duration_;
- float acceleration_;
-
- scoped_ptr<ui::SlideAnimation> animation_;
-
- DISALLOW_COPY_AND_ASSIGN(ScrollAnimator);
-};
-
-} // namespace views
-
-#endif // UI_VIEWS_ANIMATION_SCROLL_ANIMATOR_H_
diff --git a/ui/views/controls/menu/menu_controller.cc b/ui/views/controls/menu/menu_controller.cc
index 2e26a6e..3408d2f 100644
--- a/ui/views/controls/menu/menu_controller.cc
+++ b/ui/views/controls/menu/menu_controller.cc
@@ -623,14 +623,6 @@ bool MenuController::OnMouseWheel(SubmenuView* source,
}
#endif
-ui::GestureStatus MenuController::OnGestureEvent(SubmenuView* source,
- const GestureEvent& event) {
- MenuPart part = GetMenuPart(source, event.location());
- if (!part.submenu)
- return ui::GESTURE_STATUS_UNKNOWN;
- return part.submenu->OnGestureEvent(event);
-}
-
bool MenuController::GetDropFormats(
SubmenuView* source,
int* formats,
diff --git a/ui/views/controls/menu/menu_controller.h b/ui/views/controls/menu/menu_controller.h
index e467bcd..ceabd3b 100644
--- a/ui/views/controls/menu/menu_controller.h
+++ b/ui/views/controls/menu/menu_controller.h
@@ -104,8 +104,6 @@ class VIEWS_EXPORT MenuController : public MessageLoop::Dispatcher {
#if defined(OS_LINUX)
bool OnMouseWheel(SubmenuView* source, const MouseWheelEvent& event);
#endif
- ui::GestureStatus OnGestureEvent(SubmenuView* source,
- const GestureEvent& event);
bool GetDropFormats(
SubmenuView* source,
diff --git a/ui/views/controls/menu/menu_host_root_view.cc b/ui/views/controls/menu/menu_host_root_view.cc
index 56086ea..44ed97f 100644
--- a/ui/views/controls/menu/menu_host_root_view.cc
+++ b/ui/views/controls/menu/menu_host_root_view.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
@@ -62,14 +62,6 @@ bool MenuHostRootView::OnMouseWheel(const MouseWheelEvent& event) {
#endif
}
-ui::GestureStatus MenuHostRootView::OnGestureEvent(const GestureEvent& event) {
- // ChromeOS uses MenuController to forward events like other
- // mouse events.
- if (!GetMenuController())
- return ui::GESTURE_STATUS_UNKNOWN;
- return GetMenuController()->OnGestureEvent(submenu_, event);
-}
-
MenuController* MenuHostRootView::GetMenuController() {
return submenu_ ? submenu_->GetMenuItem()->GetMenuController() : NULL;
}
diff --git a/ui/views/controls/menu/menu_host_root_view.h b/ui/views/controls/menu/menu_host_root_view.h
index 1af721c..15f61bb 100644
--- a/ui/views/controls/menu/menu_host_root_view.h
+++ b/ui/views/controls/menu/menu_host_root_view.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
@@ -31,7 +31,6 @@ class MenuHostRootView : public internal::RootView {
virtual void OnMouseReleased(const MouseEvent& event) OVERRIDE;
virtual void OnMouseMoved(const MouseEvent& event) OVERRIDE;
virtual bool OnMouseWheel(const MouseWheelEvent& event) OVERRIDE;
- virtual ui::GestureStatus OnGestureEvent(const GestureEvent& event) OVERRIDE;
private:
// Returns the MenuController for this MenuHostRootView.
diff --git a/ui/views/controls/menu/submenu_view.cc b/ui/views/controls/menu/submenu_view.cc
index 3839ac7..37f199b 100644
--- a/ui/views/controls/menu/submenu_view.cc
+++ b/ui/views/controls/menu/submenu_view.cc
@@ -1,12 +1,9 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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 "ui/views/controls/menu/submenu_view.h"
-#include <algorithm>
-
-#include "base/compiler_specific.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/gfx/canvas.h"
#include "ui/views/controls/menu/menu_config.h"
@@ -42,9 +39,7 @@ SubmenuView::SubmenuView(MenuItemView* parent)
scroll_view_container_(NULL),
max_accelerator_width_(0),
minimum_preferred_width_(0),
- resize_open_menu_(false),
- ALLOW_THIS_IN_INITIALIZER_LIST(
- scroll_animator_(new ScrollAnimator(this))) {
+ resize_open_menu_(false) {
DCHECK(parent);
// We'll delete ourselves, otherwise the ScrollView would delete us on close.
set_parent_owned(false);
@@ -247,26 +242,6 @@ bool SubmenuView::OnMouseWheel(const MouseWheelEvent& e) {
return true;
}
-ui::GestureStatus SubmenuView::OnGestureEvent(const GestureEvent& e) {
- ui::GestureStatus to_return = ui::GESTURE_STATUS_CONSUMED;
- switch (e.type()) {
- case ui::ET_GESTURE_SCROLL_BEGIN:
- scroll_animator_->Stop();
- break;
- case ui::ET_GESTURE_SCROLL_UPDATE:
- OnScroll(0, e.delta_y());
- break;
- case ui::ET_GESTURE_SCROLL_END:
- if (e.delta_y() != 0.0f)
- scroll_animator_->Start(0, e.delta_y());
- break;
- default:
- to_return = ui::GESTURE_STATUS_UNKNOWN;
- break;
- }
- return to_return;
-}
-
bool SubmenuView::IsShowing() {
return host_ && host_->IsMenuHostVisible();
}
@@ -420,17 +395,4 @@ gfx::Rect SubmenuView::CalculateDropIndicatorBounds(
}
}
-void SubmenuView::OnScroll(float dx, float dy) {
- const gfx::Rect& vis_bounds = GetVisibleBounds();
- const gfx::Rect& full_bounds = bounds();
- int x = vis_bounds.x();
- int y = vis_bounds.y() - static_cast<int>(dy);
- // clamp y to [0, full_height - vis_height)
- y = std::max(y, 0);
- y = std::min(y, full_bounds.height() - vis_bounds.height() - 1);
- gfx::Rect new_vis_bounds(x, y, vis_bounds.width(), vis_bounds.height());
- if (new_vis_bounds != vis_bounds)
- ScrollRectToVisible(new_vis_bounds);
-}
-
} // namespace views
diff --git a/ui/views/controls/menu/submenu_view.h b/ui/views/controls/menu/submenu_view.h
index 40981a2..d4d0497 100644
--- a/ui/views/controls/menu/submenu_view.h
+++ b/ui/views/controls/menu/submenu_view.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
@@ -9,7 +9,6 @@
#include <string>
#include "base/compiler_specific.h"
-#include "ui/views/animation/scroll_animator.h"
#include "ui/views/controls/menu/menu_delegate.h"
#include "ui/views/view.h"
@@ -34,8 +33,7 @@ class MenuScrollViewContainer;
// MenuScrollViewContainer handles showing as much of the SubmenuView as the
// screen allows. If the SubmenuView is taller than the screen, scroll buttons
// are provided that allow the user to see all the menu items.
-class VIEWS_EXPORT SubmenuView : public View,
- public ScrollDelegate {
+class VIEWS_EXPORT SubmenuView : public View {
public:
// The submenu's class name.
static const char kViewClassName[];
@@ -76,9 +74,6 @@ class VIEWS_EXPORT SubmenuView : public View,
// Scrolls on menu item boundaries.
virtual bool OnMouseWheel(const MouseWheelEvent& e) OVERRIDE;
- // Scrolls on menu item boundaries.
- virtual ui::GestureStatus OnGestureEvent(const GestureEvent& e) OVERRIDE;
-
// Returns true if the menu is showing.
bool IsShowing();
@@ -175,9 +170,6 @@ class VIEWS_EXPORT SubmenuView : public View,
gfx::Rect CalculateDropIndicatorBounds(MenuItemView* item,
MenuDelegate::DropPosition position);
- // Implementation of ScrollDelegate
- virtual void OnScroll(float dx, float dy) OVERRIDE;
-
// Parent menu item.
MenuItemView* parent_menu_item_;
@@ -204,9 +196,6 @@ class VIEWS_EXPORT SubmenuView : public View,
// Reposition open menu when contained views change size.
bool resize_open_menu_;
- // The submenu's scroll animator
- scoped_ptr<ScrollAnimator> scroll_animator_;
-
DISALLOW_COPY_AND_ASSIGN(SubmenuView);
};
diff --git a/ui/views/views.gyp b/ui/views/views.gyp
index 409d773..7768c60 100644
--- a/ui/views/views.gyp
+++ b/ui/views/views.gyp
@@ -51,8 +51,6 @@
'accessible_pane_view.h',
'animation/bounds_animator.cc',
'animation/bounds_animator.h',
- 'animation/scroll_animator.cc',
- 'animation/scroll_animator.h',
'background.cc',
'background.h',
'border.cc',