DevTools: Make addFileForURL more robust

Source/devtools/front_end/sdk/NetworkWorkspaceBinding.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 111 matching lines @@
     /**
      * @param {string} url
      * @param {!WebInspector.ContentProvider} contentProvider
      * @param {boolean=} isContentScript
      * @return {!WebInspector.UISourceCode}
      */
     addFileForURL: function(url, contentProvider, isContentScript)
     {
         var splitURL = WebInspector.ParsedURL.splitURL(url);
         var projectName = splitURL[0];
-        var parentPath = decodeURI(splitURL.slice(1, splitURL.length - 1).join("/"));
-        var name = decodeURI(splitURL[splitURL.length - 1]);
+        var parentPath = splitURL.slice(1, -1).join("/");
+        try {
+            parentPath = decodeURIComponent(parentPath);

[vsevik, 2014/07/02 12:43:40]

We should use decodeURI here because of the difference in %2f treatment in this
two functions. We should never allow decoding %2f to a slash symbol here since
it will affect the way we split the URL into the path components.

[alph, 2014/07/02 13:14:35]

Done.

+        } catch (e) { }
+        var name = splitURL.peekLast() || "";
+        try {
+            name = decodeURIComponent(name);
+        } catch (e) { }
         var projectDelegate = this._projectDelegate(projectName, isContentScript || false);
         var path = projectDelegate.addFile(parentPath, name, url, contentProvider);
         var uiSourceCode = /** @type {!WebInspector.UISourceCode} */ (this._workspace.uiSourceCode(projectDelegate.id(), path));
         console.assert(uiSourceCode);
         return uiSourceCode;
     },
 
     reset: function()
     {
         for (var projectId in this._projectDelegates)
             this._projectDelegates[projectId].reset();
         this._projectDelegates = {};
     },
 
     __proto__: WebInspector.Object.prototype
 }