mash: Run pre-unlock animation via SessionController

chrome/browser/chromeos/power/login_lock_state_notifier.cc

 // 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.
 
 #include "chrome/browser/chromeos/power/login_lock_state_notifier.h"
 
 #include "ash/shell.h"
 #include "base/logging.h"
 #include "chrome/browser/chrome_notification_types.h"
-#include "chrome/browser/chromeos/login/lock/screen_locker.h"
 #include "content/public/browser/notification_service.h"
 
 namespace ash {
 class LockStateControllerDelegate;
 }
 
 namespace chromeos {
 
 LoginLockStateNotifier::LoginLockStateNotifier() {

[James Cook, 2017/04/07 20:26:55]

This isn't really a LockStateNotifier any more. It should probably be renamed
(or maybe folded into some other class that observes notifications).

[xiyuan, 2017/04/07 21:50:01]

Yes, my plan is to retire this class completely. I would probably move
NOTIFICATION_APP_TERMINATING into SessionControllerClient and add a mojo call to
forward it to ash.

   registrar_.Add(this, chrome::NOTIFICATION_APP_TERMINATING,
                  content::NotificationService::AllSources());
-  registrar_.Add(this, chrome::NOTIFICATION_SCREEN_LOCK_STATE_CHANGED,
-                 content::NotificationService::AllSources());
-
-  const ScreenLocker* locker = ScreenLocker::default_screen_locker();
-  bool locked = locker && locker->locked();
-  ash::Shell::Get()->OnLockStateChanged(locked);
 }
 
 LoginLockStateNotifier::~LoginLockStateNotifier() {}
 
 void LoginLockStateNotifier::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   switch (type) {
     case chrome::NOTIFICATION_APP_TERMINATING:
       ash::Shell::Get()->OnAppTerminating();
       break;
-    case chrome::NOTIFICATION_SCREEN_LOCK_STATE_CHANGED: {
-      bool locked = *content::Details<bool>(details).ptr();
-      ash::Shell::Get()->OnLockStateChanged(locked);
-      break;
-    }
     default:
       NOTREACHED() << "Unexpected notification " << type;
   }
 }
 
 }  // namespace chromeos