Close app list widget when users click outside of the widget

ui/app_list/presenter/app_list_presenter_impl.cc

 // Copyright 2016 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/presenter/app_list_presenter_impl.h"
 
 #include "ui/app_list/app_list_constants.h"
 #include "ui/app_list/app_list_switches.h"
 #include "ui/app_list/pagination_model.h"
 #include "ui/app_list/presenter/app_list_presenter_delegate_factory.h"
@@ skipping 19 matching lines @@
 
 }  // namespace
 
 AppListPresenterImpl::AppListPresenterImpl(
     AppListPresenterDelegateFactory* factory)
     : factory_(factory) {
   DCHECK(factory);
 }
 
 AppListPresenterImpl::~AppListPresenterImpl() {
+  Dismiss();

[sadrul, 2016/10/21 15:12:48]

Looks like this is a virtual function, and calling this from the dtor feels
icky/risky. Perhaps AppListPresenterImpl should be a final class? Or just make
the Dismiss() override final?

[thanhph, 2016/10/21 16:09:40]

Thank Sadrul! I make Dismiss override final by changing .h file.

   presenter_delegate_.reset();
   // Ensures app list view goes before the controller since pagination model
   // lives in the controller and app list view would access it on destruction.
   if (view_) {
     view_->GetAppsPaginationModel()->RemoveObserver(this);
     if (view_->GetWidget())
       view_->GetWidget()->CloseNow();
   }
 }
 
@@ skipping 214 matching lines @@
   } else if (should_snap_back_) {
     should_snap_back_ = false;
     ui::ScopedLayerAnimationSettings animation(widget_animator);
     animation.SetTransitionDuration(
         base::TimeDelta::FromMilliseconds(kOverscrollPageTransitionDurationMs));
     widget->SetBounds(view_bounds_);
   }
 }
 
 }  // namespace app_list