Documents what ApplicationManager::Delegate do
And renames one to better indicate what it is expected to do.
This is a port of https://codereview.chromium.org/1052103005/.
R=davemoore@chromium.org
Review URL: https://codereview.chromium.org/1105063002
diff --git a/shell/application_manager/application_manager.h b/shell/application_manager/application_manager.h
index 5d1e1c9..52b9050 100644
--- a/shell/application_manager/application_manager.h
+++ b/shell/application_manager/application_manager.h
@@ -35,9 +35,16 @@
public:
class Delegate {
public:
- virtual ~Delegate();
- virtual GURL ResolveURL(const GURL& url);
- virtual GURL ResolveMappings(const GURL& url);
+ // Gives the delegate a chance to apply any mappings for the specified url.
+ // This should not resolve 'mojo' urls, that is done by ResolveMojoURL().
+ virtual GURL ResolveMappings(const GURL& url) = 0;
+
+ // Used to map a url with the scheme 'mojo' to the appropriate url. Return
+ // |url| if the scheme is not 'mojo'.
+ virtual GURL ResolveMojoURL(const GURL& url) = 0;
+
+ protected:
+ virtual ~Delegate() {}
};
// API for testing.
