Rename OnServiceError to OnApplicationError

BUG=

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

Cr-Commit-Position: refs/heads/master@{#291951}

mojo/application_manager/application_loader.h

@@ -75,8 +75,9 @@ class MOJO_APPLICATION_MANAGER_EXPORT ApplicationLoader {
                     const GURL& url,
                     scoped_refptr<LoadCallbacks> callbacks) = 0;
 
-  // TODO(davemoore): Change name to OnApplicationError.
-  virtual void OnServiceError(ApplicationManager* manager, const GURL& url) = 0;
+  // Called when the Application exits.
+  virtual void OnApplicationError(ApplicationManager* manager,
+                                  const GURL& url) = 0;
 
  protected:
   ApplicationLoader() {}

mojo/application_manager/application_manager.cc

@@ -288,7 +288,7 @@ void ApplicationManager::OnShellImplError(ShellImpl* shell_impl) {
   url_to_shell_impl_.erase(it);
   ApplicationLoader* loader = GetLoaderForURL(url);
   if (loader)
-    loader->OnServiceError(this, url);
+    loader->OnApplicationError(this, url);
   if (delegate_)
     delegate_->OnApplicationError(url);
 }

mojo/application_manager/application_manager_unittest.cc

@@ -118,7 +118,7 @@ class TestApplicationLoader : public ApplicationLoader,
         new ApplicationImpl(this, callbacks->RegisterApplication().Pass()));
   }
 
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE {}
 
   // ApplicationDelegate implementation.
@@ -329,7 +329,7 @@ class Tester : public ApplicationDelegate,
         new ApplicationImpl(this, callbacks->RegisterApplication().Pass()));
   }
 
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE {}
 
   virtual bool ConfigureIncomingConnection(

mojo/application_manager/background_shell_application_loader.cc

@@ -23,8 +23,8 @@ class BackgroundShellApplicationLoader::BackgroundLoader {
     loader_->Load(manager, url, callbacks);
   }
 
-  void OnServiceError(ApplicationManager* manager, const GURL& url) {
-    loader_->OnServiceError(manager, url);
+  void OnApplicationError(ApplicationManager* manager, const GURL& url) {
+    loader_->OnApplicationError(manager, url);
   }
 
  private:
@@ -80,13 +80,12 @@ void BackgroundShellApplicationLoader::Load(
           base::Owned(new ScopedMessagePipeHandle(shell_handle.Pass()))));
 }
 
-void BackgroundShellApplicationLoader::OnServiceError(
+void BackgroundShellApplicationLoader::OnApplicationError(
     ApplicationManager* manager,
     const GURL& url) {
-  task_runner_->PostTask(
-      FROM_HERE,
-      base::Bind(
-          &BackgroundShellApplicationLoader::OnServiceErrorOnBackgroundThread,
+  task_runner_->PostTask(FROM_HERE,
+                         base::Bind(&BackgroundShellApplicationLoader::
+                                        OnApplicationErrorOnBackgroundThread,
                                     base::Unretained(this),
                                     manager,
                                     url));
@@ -116,13 +115,13 @@ void BackgroundShellApplicationLoader::LoadOnBackgroundThread(
   background_loader_->Load(manager, url, shell_handle->Pass());
 }
 
-void BackgroundShellApplicationLoader::OnServiceErrorOnBackgroundThread(
+void BackgroundShellApplicationLoader::OnApplicationErrorOnBackgroundThread(
     ApplicationManager* manager,
     const GURL& url) {
   DCHECK(task_runner_->RunsTasksOnCurrentThread());
   if (!background_loader_)
     background_loader_ = new BackgroundLoader(loader_.get());
-  background_loader_->OnServiceError(manager, url);
+  background_loader_->OnApplicationError(manager, url);
 }
 
 }  // namespace mojo

mojo/application_manager/background_shell_application_loader.h

@@ -29,7 +29,7 @@ class MOJO_APPLICATION_MANAGER_EXPORT BackgroundShellApplicationLoader
   virtual void Load(ApplicationManager* manager,
                     const GURL& url,
                     scoped_refptr<LoadCallbacks> callbacks) OVERRIDE;
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE;
 
  private:
@@ -45,7 +45,7 @@ class MOJO_APPLICATION_MANAGER_EXPORT BackgroundShellApplicationLoader
   void LoadOnBackgroundThread(ApplicationManager* manager,
                               const GURL& url,
                               ScopedMessagePipeHandle* shell_handle);
-  void OnServiceErrorOnBackgroundThread(ApplicationManager* manager,
+  void OnApplicationErrorOnBackgroundThread(ApplicationManager* manager,
                                             const GURL& url);
   bool quit_on_shutdown_;
   scoped_ptr<ApplicationLoader> loader_;

mojo/application_manager/background_shell_application_loader_unittest.cc

@@ -23,7 +23,7 @@ class DummyLoader : public ApplicationLoader {
       base::MessageLoop::current()->Quit();
   }
 
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE {}
 
   void DontSimulateAppQuit() { simulate_app_quit_ = false; }

mojo/services/public/cpp/view_manager/tests/view_manager_unittest.cc

@@ -64,7 +64,7 @@ class ConnectApplicationLoader : public ApplicationLoader,
     apps_.push_back(app.release());
   }
 
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE {}
 
   virtual bool ConfigureIncomingConnection(ApplicationConnection* connection)

mojo/services/view_manager/view_manager_unittest.cc

@@ -360,7 +360,7 @@ class EmbedApplicationLoader : public ApplicationLoader,
                                                         shell_handle.Pass()));
     apps_.push_back(app.release());
   }
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE {}
 
   // ApplicationDelegate implementation:

mojo/services/window_manager/window_manager_api_unittest.cc

@@ -115,7 +115,7 @@ class TestApplicationLoader : public ApplicationLoader,
         new ApplicationImpl(this, shell_handle.Pass()));
     apps_.push_back(app.release());
   }
-  virtual void OnServiceError(ApplicationManager* application_manager,
+  virtual void OnApplicationError(ApplicationManager* application_manager,
                                   const GURL& url) MOJO_OVERRIDE {}
 
   // Overridden from ApplicationDelegate:

mojo/shell/context.cc

@@ -127,7 +127,7 @@ class Context::NativeViewportApplicationLoader
       app_.reset(new ApplicationImpl(this, shell_handle.Pass()));
   }
 
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE {}
 
   // ApplicationDelegate implementation.

mojo/shell/dbus_application_loader_linux.cc

@@ -162,7 +162,7 @@ void DBusApplicationLoader::Load(ApplicationManager* manager,
       new LoadContext(this, bus_, url, shell_handle.Pass());
 }
 
-void DBusApplicationLoader::OnServiceError(ApplicationManager* manager,
+void DBusApplicationLoader::OnApplicationError(ApplicationManager* manager,
                                                const GURL& url) {
   // TODO(cmasone): Anything at all in this method here.
 }

mojo/shell/dbus_application_loader_linux.h

@@ -64,7 +64,7 @@ class DBusApplicationLoader : public ApplicationLoader {
                     const GURL& url,
                     scoped_refptr<LoadCallbacks> callbacks) OVERRIDE;
 
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE;
 
  private:

mojo/shell/dynamic_application_loader.cc

@@ -167,7 +167,7 @@ void DynamicApplicationLoader::OnRunLibraryComplete(
   }
 }
 
-void DynamicApplicationLoader::OnServiceError(ApplicationManager* manager,
+void DynamicApplicationLoader::OnApplicationError(ApplicationManager* manager,
                                                   const GURL& url) {
   // TODO(darin): What should we do about service errors? This implies that
   // the app closed its handle to the service manager. Maybe we don't care?

mojo/shell/dynamic_application_loader.h

@@ -40,7 +40,7 @@ class DynamicApplicationLoader : public ApplicationLoader {
   virtual void Load(ApplicationManager* manager,
                     const GURL& url,
                     scoped_refptr<LoadCallbacks> callbacks) OVERRIDE;
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE;
 
  private:

mojo/shell/network_application_loader.cc

@@ -44,7 +44,7 @@ void NetworkApplicationLoader::Load(ApplicationManager* manager,
   }
 }
 
-void NetworkApplicationLoader::OnServiceError(ApplicationManager* manager,
+void NetworkApplicationLoader::OnApplicationError(ApplicationManager* manager,
                                                   const GURL& url) {
   apps_.erase(reinterpret_cast<uintptr_t>(manager));
 }

mojo/shell/network_application_loader.h

@@ -34,7 +34,7 @@ class NetworkApplicationLoader : public ApplicationLoader,
   virtual void Load(ApplicationManager* manager,
                     const GURL& url,
                     scoped_refptr<LoadCallbacks> callbacks) OVERRIDE;
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE;
 
   // ApplicationDelegate overrides.

mojo/shell/ui_application_loader_android.cc

@@ -23,8 +23,8 @@ class UIApplicationLoader::UILoader {
     loader_->Load(manager, url, callbacks);
   }
 
-  void OnServiceError(ApplicationManager* manager, const GURL& url) {
-    loader_->OnServiceError(manager, url);
+  void OnApplicationError(ApplicationManager* manager, const GURL& url) {
+    loader_->OnApplicationError(manager, url);
   }
 
  private:
@@ -62,11 +62,11 @@ void UIApplicationLoader::Load(ApplicationManager* manager,
           base::Owned(new ScopedMessagePipeHandle(shell_handle.Pass()))));
 }
 
-void UIApplicationLoader::OnServiceError(ApplicationManager* manager,
+void UIApplicationLoader::OnApplicationError(ApplicationManager* manager,
                                              const GURL& url) {
   context_->ui_loop()->PostTask(
       FROM_HERE,
-      base::Bind(&UIApplicationLoader::OnServiceErrorOnUIThread,
+      base::Bind(&UIApplicationLoader::OnApplicationErrorOnUIThread,
                  base::Unretained(this),
                  manager,
                  url));
@@ -81,11 +81,12 @@ void UIApplicationLoader::LoadOnUIThread(
   ui_loader_->Load(manager, url, shell_handle->Pass());
 }
 
-void UIApplicationLoader::OnServiceErrorOnUIThread(ApplicationManager* manager,
+void UIApplicationLoader::OnApplicationErrorOnUIThread(
+    ApplicationManager* manager,
     const GURL& url) {
   if (!ui_loader_)
     ui_loader_ = new UILoader(loader_.get());
-  ui_loader_->OnServiceError(manager, url);
+  ui_loader_->OnApplicationError(manager, url);
 }
 
 void UIApplicationLoader::ShutdownOnUIThread() {

mojo/shell/ui_application_loader_android.h

@@ -29,7 +29,7 @@ class UIApplicationLoader : public ApplicationLoader {
   virtual void Load(ApplicationManager* manager,
                     const GURL& url,
                     scoped_refptr<LoadCallbacks> callbacks) OVERRIDE;
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE;
 
  private:
@@ -42,7 +42,8 @@ class UIApplicationLoader : public ApplicationLoader {
   void LoadOnUIThread(ApplicationManager* manager,
                       const GURL& url,
                       ScopedMessagePipeHandle* shell_handle);
-  void OnServiceErrorOnUIThread(ApplicationManager* manager, const GURL& url);
+  void OnApplicationErrorOnUIThread(ApplicationManager* manager,
+                                    const GURL& url);
   void ShutdownOnUIThread();
 
   scoped_ptr<ApplicationLoader> loader_;

mojo/shell/view_manager_loader.cc

@@ -34,7 +34,7 @@ void ViewManagerLoader::Load(ApplicationManager* manager,
   apps_.push_back(app.release());
 }
 
-void ViewManagerLoader::OnServiceError(ApplicationManager* manager,
+void ViewManagerLoader::OnApplicationError(ApplicationManager* manager,
                                            const GURL& url) {
 }
 

mojo/shell/view_manager_loader.h

@@ -32,7 +32,7 @@ class ViewManagerLoader : public ApplicationLoader,
   virtual void Load(ApplicationManager* manager,
                     const GURL& url,
                     scoped_refptr<LoadCallbacks> callbacks) OVERRIDE;
-  virtual void OnServiceError(ApplicationManager* manager,
+  virtual void OnApplicationError(ApplicationManager* manager,
                                   const GURL& url) OVERRIDE;
 
   // ApplicationDelegate overrides.