Strip query string before applying url mapping.

shell/url_resolver.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 "shell/url_resolver.h"
 
 #include "base/base_paths.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
+#include "shell/application_manager/query_util.h"
 #include "shell/filename_util.h"
 #include "shell/switches.h"
 #include "url/url_util.h"
 
 namespace mojo {
 namespace shell {
 
 URLResolver::URLResolver() {
   // Needed to treat first component of mojo URLs as host, not path.
   url::AddStandardScheme("mojo");
@@ skipping 37 matching lines @@
       origin != origin.GetOrigin()) {
     // Disallow invalid mappings.
     LOG(ERROR) << "Invalid origin for mapping: " << origin;
     return;
   }
   // Force both origin and base_url to have trailing slashes.
   origin_map_[origin] = AddTrailingSlashIfNeeded(base_url);
 }
 
 GURL URLResolver::ApplyMappings(const GURL& url) const {
-  GURL mapped_url(url);
+  auto url_and_suffix = GetBaseURLAndQuery(url);
+  GURL mapped_url = url_and_suffix.first;
   for (;;) {
     const auto& url_it = url_map_.find(mapped_url);
     if (url_it != url_map_.end()) {
       mapped_url = url_it->second;
       continue;
     }
 
     GURL origin = mapped_url.GetOrigin();
     const auto& origin_it = origin_map_.find(origin);
     if (origin_it == origin_map_.end())
       break;
     mapped_url = GURL(origin_it->second.spec() +
                       mapped_url.spec().substr(origin.spec().length()));
   }
 
+  if (url_and_suffix.second.length())
+    mapped_url = GURL(mapped_url.spec() + url_and_suffix.second);
   return mapped_url;
 }
 
 void URLResolver::SetMojoBaseURL(const GURL& mojo_base_url) {
   DCHECK(mojo_base_url.is_valid());
   // Force a trailing slash on the base_url to simplify resolving
   // relative files and URLs below.
   mojo_base_url_ = AddTrailingSlashIfNeeded(mojo_base_url);
 }
 
 GURL URLResolver::ResolveMojoURL(const GURL& mojo_url) const {
   if (mojo_url.scheme() != "mojo") {
     // The mapping has produced some sort of non-mojo: URL - file:, http:, etc.
     return mojo_url;
   } else {
     // It's still a mojo: URL, use the default mapping scheme.
-    std::string suffix = "";
-    if (mojo_url.has_query()) {
-      suffix = "?" + mojo_url.query();
-    }
-    std::string lib = mojo_url.host() + ".mojo" + suffix;
+    auto url_and_suffix = GetBaseURLAndQuery(mojo_url);
+    std::string lib =
+        url_and_suffix.first.host() + ".mojo" + url_and_suffix.second;
     return mojo_base_url_.Resolve(lib);
   }
 }
 
 }  // namespace shell
 }  // namespace mojo