chromeos: Makes ~QuickLaunch delete widgets

mash/quick_launch/quick_launch.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 "mash/quick_launch/quick_launch.h"
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/string16.h"
@@ skipping 135 matching lines @@
   views::Textfield* prompt_;
   catalog::mojom::CatalogPtr catalog_;
   std::set<base::string16> app_names_;
   bool suggestion_rejected_ = false;
 
   DISALLOW_COPY_AND_ASSIGN(QuickLaunchUI);
 };
 
 QuickLaunch::QuickLaunch() {
   registry_.AddInterface<::mash::mojom::Launchable>(this);
 }

[Elliot Glaysher, 2017/02/15 20:08:30]

nit: newline between ctor and dtor?

[sky, 2017/02/15 20:10:01]

Done.

-QuickLaunch::~QuickLaunch() {}
+QuickLaunch::~QuickLaunch() {
+  while (!windows_.empty())
+    windows_.front()->CloseNow();
+}
 
 void QuickLaunch::RemoveWindow(views::Widget* window) {
   auto it = std::find(windows_.begin(), windows_.end(), window);
   DCHECK(it != windows_.end());
   windows_.erase(it);
   if (windows_.empty() && base::MessageLoop::current()->is_running())
     base::MessageLoop::current()->QuitWhenIdle();
 }
 
 void QuickLaunch::OnStart() {
@@ skipping 31 matching lines @@
   windows_.push_back(window);
 }
 
 void QuickLaunch::Create(const service_manager::Identity& remote_identity,
                          ::mash::mojom::LaunchableRequest request) {
   bindings_.AddBinding(this, std::move(request));
 }
 
 }  // namespace quick_launch
 }  // namespace mash