cros: Make sure views-based lock screen is destroyed after it is dismissed.

ash/login/ui/lock_screen.cc

 // Copyright 2017 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 "ash/login/ui/lock_screen.h"
 
 #include "ash/login/ui/lock_contents_view.h"
 #include "ash/login/ui/lock_window.h"
 #include "ui/display/display.h"
 #include "ui/display/screen.h"
 #include "ui/views/widget/widget.h"
 
 namespace ash {
 
+namespace {
+// Reference to global lock screen instance. There can only ever be one lock
+// screen display at the same time.
+LockWindow* window_ = nullptr;

[xiyuan, 2017/06/06 22:37:52]

Just call it |window| or |g_window|. No trailing "_".

[jdufault, 2017/06/07 18:58:35]

Done.

+}  // namespace
+
 bool ShowLockScreen() {
-  LockWindow* window = new LockWindow();
-  window->SetBounds(display::Screen::GetScreen()->GetPrimaryDisplay().bounds());
+  CHECK(!window_);
+  window_ = new LockWindow();
+  window_->SetBounds(
+      display::Screen::GetScreen()->GetPrimaryDisplay().bounds());
 
-  views::View* contents = new LockContentsView();
-  window->SetContentsView(contents);
-  window->Show();
+  auto* contents = new LockContentsView();
+  window_->SetContentsView(contents);
+  window_->Show();
 
   return true;
 }
 
+void DestroyLockScreen() {
+  CHECK(window_);
+  window_->Close();
+  window_ = nullptr;
+}
+
 }  // namespace ash