Implements the dialog view for logout button tray in public sessions.

ash/system/logout_button/logout_button_tray.cc

 // Copyright 2013 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/system/logout_button/logout_button_tray.h"
 
 #include "ash/shelf/shelf_types.h"
 #include "ash/shell.h"
+#include "ash/system/logout_button/logout_confirmation_dialog_view.h"
 #include "ash/system/status_area_widget.h"
-#include "ash/system/tray/system_tray_delegate.h"

[bartfab (slow), 2013/10/30 11:17:28]

This is still used. system_tray_delegate() returns a SystemTrayDelegate.

[binjin, 2013/11/19 14:43:46]

Done.

 #include "ash/system/tray/system_tray_notifier.h"
 #include "ash/system/tray/tray_constants.h"
 #include "ash/system/tray/tray_utils.h"
 #include "base/logging.h"
 #include "grit/ash_resources.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/events/event.h"
 #include "ui/gfx/font.h"
 #include "ui/gfx/insets.h"
 #include "ui/gfx/size.h"
@@ skipping 68 matching lines @@
   set_animate_on_state_change(false);
 
   set_min_size(gfx::Size(0, GetShelfItemHeight()));
 }
 
 LogoutButton::~LogoutButton() {
 }
 
 LogoutButtonTray::LogoutButtonTray(StatusAreaWidget* status_area_widget)
     : TrayBackgroundView(status_area_widget),
-      button_(NULL),
       login_status_(user::LOGGED_IN_NONE),
       show_logout_button_in_tray_(false) {
   button_ = new LogoutButton(this);
   tray_container()->AddChildView(button_);
   tray_container()->set_border(NULL);
   Shell::GetInstance()->system_tray_notifier()->AddLogoutButtonObserver(this);
 }
 
 LogoutButtonTray::~LogoutButtonTray() {
   Shell::GetInstance()->system_tray_notifier()->
       RemoveLogoutButtonObserver(this);
+  if (confirmation_dialog_) {
+    confirmation_dialog_->DeleteDelegate();

[bartfab (slow), 2013/10/30 11:17:28]

This is the wrong method. DeleteDelegate() is how a widget notifies its delegate
that the widget is going away. You will need to do something different here.
From a cursory look at the code GetWidget()->Close() followed by a DeleteSoon
might be the correct soluton.

Also, no curly braces for single-line conditionals.

[binjin, 2013/11/19 14:43:46]

Done.

+  }
+}
+
+void LogoutButtonTray::EnsureConfirmationDialogIsShowing() {
+  if (!confirmation_dialog_) {
+    confirmation_dialog_ = (new LogoutConfirmationDialogView())->AsWeakPtr();
+    confirmation_dialog_->Show(dialog_duration_);
+  }
+}
+
+void LogoutButtonTray::EnsureConfirmationDialogIsClosed() {
+  if (confirmation_dialog_)
+    confirmation_dialog_->GetWidget()->Close();

[bartfab (slow), 2013/10/30 11:17:28]

Can you destroy the dialog here? Otherwise, we are using memory for no good
reason - the dialog is no longer showing, might as well destroy it.

[binjin, 2013/11/19 14:43:46]

Done.

 }
 
 void LogoutButtonTray::SetShelfAlignment(ShelfAlignment alignment) {
   TrayBackgroundView::SetShelfAlignment(alignment);
   tray_container()->set_border(NULL);
 }
 
 base::string16 LogoutButtonTray::GetAccessibleNameForTray() {
   return button_->GetText();
 }
 
 void LogoutButtonTray::HideBubbleWithView(
     const views::TrayBubbleView* bubble_view) {
 }
 
 bool LogoutButtonTray::ClickedOutsideBubble() {
   return false;
 }
 
 void LogoutButtonTray::OnShowLogoutButtonInTrayChanged(bool show) {
   show_logout_button_in_tray_ = show;
   UpdateVisibility();
 }
 
+void LogoutButtonTray::OnLogoutDialogDurationChanged(int duration) {
+  dialog_duration_ = base::TimeDelta::FromMilliseconds(duration);
+  if (confirmation_dialog_)
+    confirmation_dialog_->UpdateDialogDuration(dialog_duration_);
+}
+
 void LogoutButtonTray::ButtonPressed(views::Button* sender,
                                      const ui::Event& event) {
   DCHECK_EQ(sender, button_);
-  Shell::GetInstance()->system_tray_delegate()->SignOut();
+  // Sign out immediately if kLogoutDialogDurationMs is non-positive.
+  if (dialog_duration_ <= base::TimeDelta::FromSeconds(0)) {

[bartfab (slow), 2013/10/30 11:17:28]

Nit: No curly braces for single-line conditionals.

[binjin, 2013/11/19 14:43:46]

Done.

+    Shell::GetInstance()->system_tray_delegate()->SignOut();
+  } else {
+    EnsureConfirmationDialogIsShowing();
+  }
 }
 
 void LogoutButtonTray::UpdateAfterLoginStatusChange(
     user::LoginStatus login_status) {
   login_status_ = login_status;
   const base::string16 title =
       GetLocalizedSignOutStringForStatus(login_status, false);
   button_->SetText(title);
   button_->SetAccessibleName(title);
   UpdateVisibility();
 }
 
 void LogoutButtonTray::UpdateVisibility() {
   SetVisible(show_logout_button_in_tray_ &&
              login_status_ != user::LOGGED_IN_NONE &&
              login_status_ != user::LOGGED_IN_LOCKED);
+  if (!show_logout_button_in_tray_)
+    EnsureConfirmationDialogIsClosed();
 }
 
 }  // namespace internal
 }  // namespace ash