Modify FullRestoreSaveHandler to override OnWindowDestroyed.

in CL:2556904, OnWindowDestroying is used by the WM to save the window
info. So FullRestoreSaveHandler should not use OnWindowDestroying to
delete the window from the restore data. So modify the interface to
use OnWindowDestroyed to delete the window from the restore data.

BUG=1146900

Change-Id: I824e216a898c6c28bf6cdae044e0a8fba7079a64
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2622354Reviewed-by: Sammie Quon <sammiequon@chromium.org>
Commit-Queue: Nancy Wang <nancylingwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#842296}

components/full_restore/full_restore_save_handler.cc

@@ -54,7 +54,7 @@ void FullRestoreSaveHandler::OnWindowInitialized(aura::Window* window) {
   observed_windows_.AddObservation(window);
 }
 
-void FullRestoreSaveHandler::OnWindowDestroying(aura::Window* window) {
+void FullRestoreSaveHandler::OnWindowDestroyed(aura::Window* window) {
   // TODO(crbug.com/1146900): Handle ARC app windows.
 
   DCHECK(observed_windows_.IsObservingSource(window));

components/full_restore/full_restore_save_handler.h

@@ -52,7 +52,7 @@ class COMPONENT_EXPORT(FULL_RESTORE) FullRestoreSaveHandler
   void OnWindowInitialized(aura::Window* window) override;
 
   // aura::WindowObserver:
-  void OnWindowDestroying(aura::Window* window) override;
+  void OnWindowDestroyed(aura::Window* window) override;
 
   // Save |app_launch_info| to the full restore file in |profile_path|.
   void SaveAppLaunchInfo(const base::FilePath& profile_path,