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

Unified Diff: chrome/browser/devtools/devtools_window.cc

Issue 2444423002: Merge to 2883 "[DevTools] Move sanitize url to devtools_ui.cc." (Closed)
Patch Set: Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/ui/webui/devtools_ui.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/devtools/devtools_window.cc
diff --git a/chrome/browser/devtools/devtools_window.cc b/chrome/browser/devtools/devtools_window.cc
index 9aeeee44364470735b20980892337f9df82afdce..bf9e2d3465a811b6c6b6fdc5527dceeb2f0e9353 100644
--- a/chrome/browser/devtools/devtools_window.cc
+++ b/chrome/browser/devtools/devtools_window.cc
@@ -204,10 +204,8 @@ GURL DecorateFrontendURL(const GURL& base_url) {
url_string += "&experiments=true";
if (command_line->HasSwitch(switches::kDevToolsFlags)) {
- std::string flags = command_line->GetSwitchValueASCII(
- switches::kDevToolsFlags);
- flags = net::EscapeQueryParamValue(flags, false);
- url_string += "&flags=" + flags;
+ url_string += "&" + command_line->GetSwitchValueASCII(
+ switches::kDevToolsFlags);
}
#if defined(DEBUG_DEVTOOLS)
@@ -902,7 +900,7 @@ GURL DevToolsWindow::GetDevToolsURL(Profile* profile,
}
if (can_dock)
url_string += "&can_dock=true";
- return GURL(url_string);
+ return DevToolsUI::SanitizeFrontendURL(GURL(url_string));
}
// static
« no previous file with comments | « no previous file | chrome/browser/ui/webui/devtools_ui.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698