Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1705)

Unified Diff: mojo/shell/application_manager.h

Issue 1228743002: Mandoline: Move ContentHandlerConnection into separate file (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed jam's comments Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/shell/application_manager.h
diff --git a/mojo/shell/application_manager.h b/mojo/shell/application_manager.h
index 727b9e6e8c523abe187cd9a786da6d69bcf4c802..aee1e095140b362c2966159486dcce798a3613e7 100644
--- a/mojo/shell/application_manager.h
+++ b/mojo/shell/application_manager.h
@@ -31,6 +31,7 @@ class SequencedWorkerPool;
namespace mojo {
namespace shell {
+class ContentHandlerConnection;
class ShellImpl;
class ApplicationManager {
@@ -143,9 +144,11 @@ class ApplicationManager {
// Removes a ShellImpl when it encounters an error.
void OnShellImplError(ShellImpl* shell_impl);
- private:
- class ContentHandlerConnection;
+ // Removes a ContentHandler when its connection is closed.
+ void OnContentHandlerConnectionClosed(
+ ContentHandlerConnection* content_handler);
+ private:
using ApplicationPackagedAlias = std::map<GURL, std::pair<GURL, std::string>>;
using IdentityToShellImplMap = std::map<Identity, ShellImpl*>;
using MimeTypeToURLMap = std::map<std::string, GURL>;
@@ -223,9 +226,6 @@ class ApplicationManager {
// configured for the URL.
ApplicationLoader* GetLoaderForURL(const GURL& url);
- // Removes a ContentHandler when it encounters an error.
- void OnContentHandlerError(ContentHandlerConnection* content_handler);
-
void CleanupRunner(NativeRunner* runner);
Delegate* const delegate_;

Powered by Google App Engine
This is Rietveld 408576698