Adding split view divider widget.

athena/wm/bezel_controller.cc

Index: athena/wm/bezel_controller.cc
diff --git a/athena/wm/bezel_controller.cc b/athena/wm/bezel_controller.cc
index a4e815602e7b32ecee1667e88caf0962f71199d6..b680bef52b7e8b67f09a7d434af0082b0f36fe13 100644
--- a/athena/wm/bezel_controller.cc
+++ b/athena/wm/bezel_controller.cc
@@ -46,6 +46,21 @@ float GetDistance(const gfx::PointF& location,
              : point_in_screen.x() - GetDisplay(window).bounds().width();
 }
 
+// Returns the bezel corresponding to the |location| in |window| or BEZEL_NONE
+// if the location is outside of the bezel area.
+// Only implemented for LEFT and RIGHT bezels ATM.

[pkotwicz, 2014/09/23 05:13:40]

Nit: Remove 'ATM'

[mfomitchev, 2014/09/23 15:46:02]

Done.

+BezelController::Bezel GetBezel(const gfx::PointF& location,
+                                aura::Window* window) {
+  int screen_width = GetDisplay(window).bounds().width();
+  gfx::Point point_in_screen(gfx::ToRoundedPoint(location));
+  wm::ConvertPointToScreen(window, &point_in_screen);
+  if (point_in_screen.x() < kBezelWidth)
+    return BezelController::BEZEL_LEFT;
+  if (point_in_screen.x() > screen_width - kBezelWidth)
+    return BezelController::BEZEL_RIGHT;
+  return BezelController::BEZEL_NONE;
+}
+
 }  // namespace
 
 BezelController::BezelController(aura::Window* container)
@@ -69,9 +84,9 @@ void BezelController::SetState(BezelController::State state,
     return;
 
   if (state == BEZEL_SCROLLING_TWO_FINGERS)
-    left_right_delegate_->ScrollBegin(scroll_bezel_, scroll_delta);
+    left_right_delegate_->BezelScrollBegin(scroll_bezel_, scroll_delta);
   else if (state_ == BEZEL_SCROLLING_TWO_FINGERS)
-    left_right_delegate_->ScrollEnd();
+    left_right_delegate_->BezelScrollEnd();
   state_ = state;
   if (state == NONE) {
     scroll_bezel_ = BEZEL_NONE;
@@ -79,18 +94,6 @@ void BezelController::SetState(BezelController::State state,
   }
 }
 
-// Only implemented for LEFT and RIGHT bezels ATM.
-BezelController::Bezel BezelController::GetBezel(const gfx::PointF& location) {
-  int screen_width = GetDisplay(container_).bounds().width();
-  if (location.x() < kBezelWidth) {
-    return BEZEL_LEFT;
-  } else if (location.x() > screen_width - kBezelWidth) {
-    return BEZEL_RIGHT;
-  } else {
-    return BEZEL_NONE;
-  }
-}
-
 void BezelController::OnGestureEvent(ui::GestureEvent* event) {
   // TODO(mfomitchev): Currently we aren't retargetting or consuming any of the
   // touch events. This means that content can prevent the generation of gesture
@@ -105,24 +108,32 @@ void BezelController::OnGestureEvent(ui::GestureEvent* event) {
   if (!ShouldProcessGesture(type))
     return;
 
+  const ui::GestureEventDetails& event_details = event->details();
+  int num_touch_points = event_details.touch_points();
+  if (num_touch_points == 1 && type == ui::ET_GESTURE_BEGIN) {
+    // Reset the state if needed.
+    // Normally, the state would be reset when ET_GESTURE_END is dispatched,
+    // however ET_GESTURE_END may not be dispatched if the gesture target is
+    // hidden or deleted before the gesture ends.
+    SetState(NONE);

[pkotwicz, 2014/09/23 05:13:40]

I am sure that I am missing something. Can you explain why this is needed? I
initially thought that this was to make the unit tests pass but that's not the
case.

Based on the description, it sounds like the changes that I am making in
BezelController in https://codereview.chromium.org/574113004/ accomplish the
same task (If the two do the same thing I think my implementation is slightly
better)

[mfomitchev, 2014/09/23 15:46:02]

When you drag the divider handle, it becomes the target of the gesture event
sequence. The handle is hidden on ET_GESTURE_SCROLL_END. This prevents
ET_GESTURE_END from being dispatched (because the target is hidden). Thus the
bezel controller never receives ET_GESTURE_END and it doesn't change its state
to NONE.

More generally, we can't rely on receiving ET_GESTURE_END for every
ET_GESTURE_BEGIN because the target can be hidden or even deleted mid-gesture.

Thus we do a state sanity check on every ET_GESTURE_BEGIN.

I've updated the comment to make it a bit more clear/detailed.

[pkotwicz, 2014/09/23 19:44:46]

From talking to tdresser@ we are supposed to get ET_GESTURE_END. However, I
cannot get ET_GESTURE_END to be sent

This is ok for now for the sake of this CL. However, it would be useful to
figure out why ET_GESTURE_END is not received

[mfomitchev, 2014/09/23 21:02:09]

I started a thread with sadrul on this, but it didn't get anywhere. I'll add
you. Also, if the target was deleted (which could happen)I understand it is
expected that ET_GESTURE_END won't be send.

+  }
+
   if (scroll_target_ && event->target() != scroll_target_)
     return;
 
   const gfx::PointF& event_location = event->location_f();
-  const ui::GestureEventDetails& event_details = event->details();
-  int num_touch_points = event_details.touch_points();
   float scroll_delta = kScrollDeltaNone;
-  if (scroll_bezel_ != BEZEL_NONE) {
-    aura::Window* target_window = static_cast<aura::Window*>(event->target());
+  aura::Window* target_window = static_cast<aura::Window*>(event->target());
+  if (scroll_bezel_ != BEZEL_NONE)
     scroll_delta = GetDistance(event_location, target_window, scroll_bezel_);
-  }
 
   if (type == ui::ET_GESTURE_BEGIN) {
     if (num_touch_points > 2) {
       SetState(IGNORE_CURRENT_SCROLL);
       return;
     }
-    BezelController::Bezel event_bezel = GetBezel(event->location_f());
+    BezelController::Bezel event_bezel =
+        GetBezel(event->location_f(), target_window);
     switch (state_) {
       case NONE:
         scroll_bezel_ = event_bezel;
@@ -174,14 +185,14 @@ void BezelController::OnGestureEvent(ui::GestureEvent* event) {
 
     DCHECK_EQ(num_touch_points, 2);
     SetState(BEZEL_SCROLLING_TWO_FINGERS, scroll_delta);
-    if (left_right_delegate_->CanScroll())
+    if (left_right_delegate_->BezelCanScroll())
       event->SetHandled();
   } else if (type == ui::ET_GESTURE_SCROLL_UPDATE) {
     if (state_ != BEZEL_SCROLLING_TWO_FINGERS)
       return;
 
-    left_right_delegate_->ScrollUpdate(scroll_delta);
-    if (left_right_delegate_->CanScroll())
+    left_right_delegate_->BezelScrollUpdate(scroll_delta);
+    if (left_right_delegate_->BezelCanScroll())
       event->SetHandled();
   }
 }