Close app list widget when users click outside of the widget

ui/app_list/presenter/app_list_presenter_impl.cc

Index: ui/app_list/presenter/app_list_presenter_impl.cc
diff --git a/ui/app_list/presenter/app_list_presenter_impl.cc b/ui/app_list/presenter/app_list_presenter_impl.cc
index cd181d1657ed606e57ce605f0824b4b262a8eb1d..e6db16a76d1a001b89936e5dee210b80e819e52d 100644
--- a/ui/app_list/presenter/app_list_presenter_impl.cc
+++ b/ui/app_list/presenter/app_list_presenter_impl.cc
@@ -37,6 +37,7 @@ AppListPresenterImpl::AppListPresenterImpl(
 }
 
 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.