Register application connections under their post-redirect URL.

shell/application_manager/fetcher.h

Index: shell/application_manager/fetcher.h
diff --git a/shell/application_manager/fetcher.h b/shell/application_manager/fetcher.h
index aaff96529e1ee0607ef1bfb471d9d38e1b9cb017..b9e9de5b830d7fdc56d81d1c20cf65aa24092b8a 100644
--- a/shell/application_manager/fetcher.h
+++ b/shell/application_manager/fetcher.h
@@ -40,6 +40,13 @@ class Fetcher {
   Fetcher(const FetchCallback& fetch_callback);
   virtual ~Fetcher();
 
+  // Returns the original URL that was fetched.

[DaveMoore, 2015/02/24 20:42:59]

Wouldn't it be better to just have one method on fetcher called GetURL()? The
local one would always return the requested one and the network implementation
would return the redirect url if there was one, otherwise the requested.

[Aaron Boodman, 2015/02/24 21:48:25]

Whether a redirect happened is important information -- it affects what
ApplicationManager does. Current CL encodes this information in whether
GetRedirectURL() returns a non-empty URL. It could also be encoded in a separate
bit ... but I kinda prefer it as it is.

+  virtual const GURL& GetURL() const = 0;
+
+  // If the fetch resulted in a redirect, this returns the final URL after all
+  // redirects. Otherwise, it returns an empty URL.
+  virtual GURL GetRedirectURL() const = 0;
+
   virtual URLResponsePtr AsURLResponse(base::TaskRunner* task_runner,
                                        uint32_t skip) = 0;