Revert 199970 "Restart apps that don't listen to onRestarted() b..."

This breaks unit_tests:BrowserCommandControllerTest.IncognitoModeOnSigninAllowedPrefChange
in debug builds.

The waterfall seems to not be running with DCHECKs on so this passed by unnoticed, but
it causes failures in the CQ which is using DCHECKs. See also crbug.com/240718

Here's a failure build: http://build.chromium.org/p/tryserver.chromium/builders/linux_rel/builds/127560

Local bisect found this test breaks after this CL but not before.

> Restart apps that don't listen to onRestarted() by sending them onLaunched().
> 
> We only do this if the app had windows opened the last time it was running,
> otherwise apps that were running but had no windows may restart by opening
> windows, which manifests as apps coming back on a restart after the user
> already closed them.
> 
> BUG=230667,167740,162057
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=199406
> Reverted: https://src.chromium.org/viewvc/chrome?view=rev&revision=199409
> 
> Review URL: https://chromiumcodereview.appspot.com/14878008

TBR=koz@chromium.org

Review URL: https://codereview.chromium.org/15031005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@200016 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 1 insertions, 13 deletions

diff --git a/apps/app_restore_service.h b/apps/app_restore_service.h
index 287a21c..f27c83c 100644
--- a/apps/app_restore_service.h
+++ b/apps/app_restore_service.h
@@ -8,7 +8,6 @@
 #include <string>
 #include <vector>
 
-#include "chrome/browser/extensions/shell_window_registry.h"
 #include "chrome/browser/profiles/profile_keyed_service.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
@@ -30,8 +29,7 @@ namespace apps {
 
 // Tracks what apps need to be restarted when the browser restarts.
 class AppRestoreService : public ProfileKeyedService,
-                          public content::NotificationObserver,
-                          public extensions::ShellWindowRegistry::Observer {
+                          public content::NotificationObserver {
  public:
   // Returns true if apps should be restored on the current platform, given
   // whether this new browser process launched due to a restart.
@@ -49,18 +47,8 @@ class AppRestoreService : public ProfileKeyedService,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
-  // extensions::ShellWindowRegistry::Observer.
-  virtual void OnShellWindowAdded(ShellWindow* shell_window) OVERRIDE;
-  virtual void OnShellWindowIconChanged(ShellWindow* shell_window) OVERRIDE;
-  virtual void OnShellWindowRemoved(ShellWindow* shell_window) OVERRIDE;
-
-  // ProfileKeyedService.
-  virtual void Shutdown() OVERRIDE;
-
   void RecordAppStart(const std::string& extension_id);
   void RecordAppStop(const std::string& extension_id);
-  void RecordIfAppHasWindows(const std::string& id);
-
   void RestoreApp(
       const extensions::Extension* extension,
       const std::vector<SavedFileEntry>& file_entries);