ApplicationManager: Use callback to get notified on application shutdown.

shell/native_runner_unittest.cc

 // Copyright 2014 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 "base/files/scoped_temp_dir.h"
 #include "shell/application_manager/application_manager.h"
 #include "shell/context.h"
 #include "shell/filename_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 60 matching lines @@
   void TearDown() override { context_.Shutdown(); }
 
  protected:
   shell::Context context_;
   base::MessageLoop loop_;
   ApplicationManager application_manager_;
   TestState state_;
 
  private:
   // ApplicationManager::Delegate
-  void OnApplicationError(const GURL& url) override {}
-
   GURL ResolveURL(const GURL& url) override { return url; }
 
   GURL ResolveMappings(const GURL& url) override { return url; }
 };
 
 TEST_F(NativeApplicationLoaderTest, DoesNotExist) {
   base::ScopedTempDir temp_dir;
   ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
   base::FilePath nonexistent_file(FILE_PATH_LITERAL("nonexistent.txt"));
   GURL url(FilePathToFileURL(temp_dir.path().Append(nonexistent_file)));
   InterfaceRequest<ServiceProvider> services;
   ServiceProviderPtr service_provider;
-  application_manager_.ConnectToApplication(url, GURL(), services.Pass(),
-                                            service_provider.Pass());
+  application_manager_.ConnectToApplication(
+      url, GURL(), services.Pass(), service_provider.Pass(), base::Closure());
   EXPECT_FALSE(state_.runner_was_created);
   EXPECT_FALSE(state_.runner_was_started);
   EXPECT_FALSE(state_.runner_was_destroyed);
 }
 
 }  // namespace
 }  // namespace shell
 }  // namespace mojo