Register application connections under their post-redirect URL.

shell/application_manager/network_fetcher.cc

 // Copyright 2015 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 "shell/application_manager/network_fetcher.h"
 
 #include "base/bind.h"
 #include "base/files/file.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
@@ skipping 21 matching lines @@
       url_(url),
       weak_ptr_factory_(this) {
   StartNetworkRequest(url, network_service);
 }
 
 NetworkFetcher::~NetworkFetcher() {
   if (!path_.empty())
     base::DeleteFile(path_, false);
 }
 
+const GURL& NetworkFetcher::GetURL() const {
+  return url_;
+}
+
+GURL NetworkFetcher::GetRedirectURL() const {
+  if (!response_)
+    return GURL::EmptyGURL();
+
+  if (response_->redirect_url.is_null())
+    return GURL::EmptyGURL();
+
+  return GURL(response_->redirect_url);

[qsr, 2015/02/25 14:50:27]

Did you test this?

The doc for those fields are:

  // These fields are set to non-NULL if this response corresponds to a
  // redirect.  Call the |FollowRedirect| method on the URLLoader instance to
  // follow this redirect.
  string? redirect_method;
  string? redirect_url;

 So my understanding is that if you ask the url loader to follow redirect, it
will handle those by himself, and those fields would always be null. If you want
to use those, you need to handle redirect by yourself (and as you said, I think
this is exactly what we should do anyway)

[Aaron Boodman, 2015/02/25 18:43:08]

Ah, you are right. Fixed.

+}
+
 URLResponsePtr NetworkFetcher::AsURLResponse(base::TaskRunner* task_runner,
                                              uint32_t skip) {
   if (skip != 0) {
     MojoResult result = ReadDataRaw(
         response_->body.get(), nullptr, &skip,
         MOJO_READ_DATA_FLAG_ALL_OR_NONE | MOJO_READ_DATA_FLAG_DISCARD);
     DCHECK_EQ(result, MOJO_RESULT_OK);
   }
   return response_.Pass();
 }
@@ skipping 141 matching lines @@
                << "while fetching " << response->url;
     loader_callback_.Run(make_scoped_ptr<Fetcher>(NULL));
     return;
   }
 
   response_ = response.Pass();
   loader_callback_.Run(make_scoped_ptr(this));
 }
 
 }  // namespace mojo