Add LogoutConfirmationController to show LogoutConfirmationDialogs

ash/ash.gyp

 # Copyright (c) 2012 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.
 
 {
   'variables': {
     'chromium_code': 1,
     'grit_out_dir': '<(SHARED_INTERMEDIATE_DIR)/chrome',
   },
   'includes': [
@@ skipping 350 matching lines @@
         'system/drive/tray_drive.h',
         'system/ime/ime_observer.h',
         'system/ime/tray_ime.cc',
         'system/ime/tray_ime.h',
         'system/keyboard_brightness/keyboard_brightness_control_delegate.h',
         'system/locale/locale_notification_controller.cc',
         'system/locale/locale_notification_controller.h',
         'system/logout_button/logout_button_observer.h',
         'system/logout_button/logout_button_tray.cc',
         'system/logout_button/logout_button_tray.h',
-        'system/logout_button/logout_confirmation_dialog_view.cc',
-        'system/logout_button/logout_confirmation_dialog_view.h',
+        'system/logout_confirmation/logout_confirmation_controller.cc',
+        'system/logout_confirmation/logout_confirmation_controller.h',
+        'system/logout_confirmation/logout_confirmation_dialog.cc',
+        'system/logout_confirmation/logout_confirmation_dialog.h',

[stevenjb, 2014/02/27 17:58:44]

Are logout_button and logout_conformation related at this point? It seems a bit
confusing to have two separate directories, maybe we can put them all in a
'logout' directory?

[bartfab (slow), 2014/02/28 12:13:49]

As of this CL, the logout button is the only class that uses the logout
confirmation dialog. But I am doing this whole refactoring only because my next
CL is adding another consumer of the dialog. If you want me to, I can still move
all the classes into a joint "logout" directory but that would imply a stronger
relationship between dialog and button than there really is.

[stevenjb, 2014/03/04 23:33:29]

I wouldn't say that implies a strong relationship, I would just prefer to keep
the number of subfolders under ash/system reasonable; it doens't make sense to
have a folder for every class. Maybe a better grouping would be
ash/system/session and everything logout and session_length_limit related could
be moved there. That can be in a separate CL however.

[bartfab (slow), 2014/03/05 18:19:44]

Moving everything under ash/system/session makes sense. I modified the CL do to
this move straight away.

         'system/monitor/tray_monitor.cc',
         'system/monitor/tray_monitor.h',
         'system/session_length_limit/session_length_limit_observer.h',
         'system/session_length_limit/tray_session_length_limit.cc',
         'system/session_length_limit/tray_session_length_limit.h',
         'system/status_area_widget.cc',
         'system/status_area_widget.h',
         'system/status_area_widget_delegate.cc',
         'system/status_area_widget_delegate.h',
         'system/system_notifier.cc',
@@ skipping 533 matching lines @@
         'sticky_keys/sticky_keys_overlay_unittest.cc',
         'sticky_keys/sticky_keys_unittest.cc',
         'system/chromeos/managed/tray_locally_managed_user_unittest.cc',
         'system/chromeos/network/network_state_notifier_unittest.cc',
         'system/chromeos/power/power_event_observer_unittest.cc',
         'system/chromeos/power/power_status_unittest.cc',
         'system/chromeos/power/tray_power_unittest.cc',
         'system/chromeos/screen_security/screen_tray_item_unittest.cc',
         'system/chromeos/tray_display_unittest.cc',
         'system/date/date_view_unittest.cc',
-        'system/logout_button/logout_button_tray_unittest.cc',
+        'system/logout_confirmation/logout_confirmation_controller_unittest.cc',
         'system/session_length_limit/tray_session_length_limit_unittest.cc',
         'system/tray/system_tray_unittest.cc',
         'system/tray/tray_details_view_unittest.cc',
         'system/user/tray_user_unittest.cc',
         'system/web_notification/web_notification_tray_unittest.cc',
         'test/ash_test_helper_unittest.cc',
         'test/ash_unittests.cc',
         'tooltips/tooltip_controller_unittest.cc',
         'touch/touch_observer_hud_unittest.cc',
         'wm/app_list_controller_unittest.cc',
@@ skipping 176 matching lines @@
             },
           },
           'dependencies': [
             '../sandbox/sandbox.gyp:sandbox',
           ],
         }],
       ],
     },
   ],
 }