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

Unified Diff: Source/devtools/front_end/bindings/NetworkWorkspaceBinding.js

Issue 656453002: DevTools: [SSP] show links for matched styles with spaces in source URL (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: rebaseline Created 6 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
Index: Source/devtools/front_end/bindings/NetworkWorkspaceBinding.js
diff --git a/Source/devtools/front_end/bindings/NetworkWorkspaceBinding.js b/Source/devtools/front_end/bindings/NetworkWorkspaceBinding.js
index 48cf791ea136bfd0036cccb5e966640f376b276d..ede705213cc13c848fc978e7d4e0f3f90e06302f 100644
--- a/Source/devtools/front_end/bindings/NetworkWorkspaceBinding.js
+++ b/Source/devtools/front_end/bindings/NetworkWorkspaceBinding.js
@@ -135,15 +135,15 @@ WebInspector.NetworkWorkspaceBinding.prototype = {
*/
addFileForURL: function(url, contentProvider, isContentScript)
{
- var splitURL = WebInspector.ParsedURL.splitURL(url);
+ var splitURL = WebInspector.ParsedURL.splitURLIntoPathComponents(url);
var projectName = splitURL[0];
var parentPath = splitURL.slice(1, -1).join("/");
try {
- parentPath = decodeURI(parentPath);
+ parentPath = parentPath;
} catch (e) { }
var name = splitURL.peekLast() || "";
try {
- name = decodeURI(name);
+ name = name;
} catch (e) { }
var projectDelegate = this._projectDelegate(projectName, isContentScript || false);
var path = projectDelegate.addFile(parentPath, name, url, contentProvider);
@@ -157,7 +157,7 @@ WebInspector.NetworkWorkspaceBinding.prototype = {
*/
removeFileForURL: function(url)
{
- var splitURL = WebInspector.ParsedURL.splitURL(url);
+ var splitURL = WebInspector.ParsedURL.splitURLIntoPathComponents(url);
var projectName = splitURL[0];
var projectDelegate = this._projectDelegates[projectName];
projectDelegate.removeFile(url);
« no previous file with comments | « Source/devtools/front_end/bindings/DefaultScriptMapping.js ('k') | Source/devtools/front_end/common/ParsedURL.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698