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

Side by Side Diff: shell/application_manager/query_util.cc

Issue 972473003: Strip query string before applying url mapping. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Follow review Created 5 years, 9 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "shell/application_manager/query_util.h"
6
7 #include "base/strings/string_util.h"
8
9 namespace mojo {
10
11 GURL GetBaseURLAndQuery(const GURL& url, std::string* query) {
12 if (!url.has_query()) {
13 if (query)
14 *query = "";
15 return url;
16 }
17
18 if (query)
19 *query = "?" + url.query();
20 GURL::Replacements repl;
21 repl.SetQueryStr("");
22 std::string result = url.ReplaceComponents(repl).spec();
23
24 // Remove the dangling '?' because it's ugly.
25 base::ReplaceChars(result, "?", "", &result);
26 return GURL(result);
27 }
28
29 } // namespace mojo
OLDNEW
« no previous file with comments | « shell/application_manager/query_util.h ('k') | shell/application_manager/query_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698