DevTools: [SSP] show links for matched styles with spaces in source URL

Source/devtools/front_end/workspace/Workspace.js

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 717 matching lines @@
 
     /**
      * @param {string} url
      * @return {?WebInspector.UISourceCode}
      */
     _networkUISourceCodeForURL: function(url)
     {
         var splitURL = WebInspector.ParsedURL.splitURL(url);
         var projectId = splitURL[0];
         var project = this.project(projectId);
-        return project ? project.uiSourceCode(splitURL.slice(1).join("/")) : null;
+        var path = decodeURI(splitURL.slice(1).join("/"));
+        return project ? project.uiSourceCode(path) : null;
     },
 
     /**
      * @param {string} url
      * @return {?WebInspector.UISourceCode}
      */
     _contentScriptUISourceCodeForURL: function(url)
     {
         var splitURL = WebInspector.ParsedURL.splitURL(url);
         var projectId = "contentscripts:" + splitURL[0];
         var project = this.project(projectId);
-        return project ? project.uiSourceCode(splitURL.slice(1).join("/")) : null;
+        var path = decodeURI(splitURL.slice(1).join("/"));

[vsevik, 2014/10/14 07:41:04]

you are doing decodeURI for the full path here whereas in the other place we are
doing it to the name only.
My understanding is that decodeURI return value could depend on this.
Also I think we should fix all the usages of WebInspector.ParsedURL.splitURL, so
consider patching the splitURL() method itself instead and adding a test to
parsed-url.html as well.

+        return project ? project.uiSourceCode(path) : null;
     },
 
     /**
      * @param {string} url
      * @return {?WebInspector.UISourceCode}
      */
     uiSourceCodeForURL: function(url)
     {
         var file = this._fileSystemMapping.fileForURL(url);
         if (!file)
@@ skipping 82 matching lines @@
         this.addEventListener(WebInspector.Workspace.Events.UISourceCodeAdded, listener, this);
     },
 
     __proto__: WebInspector.Object.prototype
 }
 
 /**
  * @type {!WebInspector.Workspace}
  */
 WebInspector.workspace;