Added new features tests to AppListPresenterDelegateUnittests.

ui/app_list/views/app_list_view.cc

 // 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/app_list/views/app_list_view.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/macros.h"
@@ skipping 503 matching lines @@
   }
   fullscreen_widget_->SetBounds(new_widget_bounds);
 }
 
 void AppListView::EndDrag(const gfx::Point& location) {
   // Change the app list state based on where the drag ended. If fling velocity
   // was over the threshold, snap to the next state in the direction of the
   // fling.
   int const new_y_position = location.y() - initial_drag_point_.y() +
                              fullscreen_widget_->GetWindowBoundsInScreen().y();
-  if (std::abs(last_fling_velocity_) > kAppListDragVelocityThreshold) {
+  if (std::abs(last_fling_velocity_) >= kAppListDragVelocityThreshold) {
     // If the user releases drag with velocity over the threshold, snap to
     // the next state, ignoring the drag release position.
 
     if (last_fling_velocity_ > 0) {
       switch (app_list_state_) {
         case PEEKING:
         case HALF:
         case FULLSCREEN_SEARCH:
           SetState(CLOSED);
           break;
@@ skipping 182 matching lines @@
     default:
       break;
   }
 }
 
 void AppListView::OnGestureEvent(ui::GestureEvent* event) {
   if (!is_fullscreen_app_list_enabled_)
     return;
 
   switch (event->type()) {
+    case ui::ET_SCROLL_FLING_START:
     case ui::ET_GESTURE_SCROLL_BEGIN:
       StartDrag(event->location());
       event->SetHandled();
       break;
     case ui::ET_GESTURE_SCROLL_UPDATE:
-      last_fling_velocity_ = event->details().velocity_y();
+      last_fling_velocity_ = event->details().scroll_y();
       UpdateDrag(event->location());
       event->SetHandled();
       break;
     case ui::ET_GESTURE_END:
       EndDrag(event->location());
       event->SetHandled();
       break;
     default:
       break;
   }
@@ skipping 105 matching lines @@
       break;
     case FULLSCREEN_ALL_APPS:
       new_widget_bounds.set_y(0);
       app_list_main_view_->contents_view()->SetActiveState(
           AppListModel::STATE_APPS);
       break;
     case FULLSCREEN_SEARCH:
       new_widget_bounds.set_y(0);
       break;
     case CLOSED:
+      GetWidget()->Close();

[xiyuan, 2017/06/21 18:16:18]

Why do we need to do this? delegate_->Dismiss() down below should close app list
eventually.

[newcomer, 2017/06/21 20:39:42]

I changed it so I could test it more easily. I agree, and I modified the tests
to check for this.app_list_state() == CLOSED instead of forcing the widget to
close.

       app_list_main_view_->Close();
       delegate_->Dismiss();
       break;
   }
   fullscreen_widget_->SetBounds(new_widget_bounds);
   app_list_state_ = new_state_override;
 }
 
 void AppListView::OnWidgetDestroying(views::Widget* widget) {
   BubbleDialogDelegateView::OnWidgetDestroying(widget);
@@ skipping 90 matching lines @@
                        .work_area()
                        .size();
   size.Enlarge(0, kShelfSize);
   fullscreen_widget_->SetSize(size);
 
   // Update the |fullscreen_widget_| bounds to accomodate the new work area.
   SetState(app_list_state_);
 }
 
 }  // namespace app_list