[DevTools] Handle valid urls with quotes inside them in SSP.

third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2009 Joseph Pecoraro
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 3089 matching lines @@
         valueElement.normalize();
         return valueElement;
     },
 
     /**
      * @param {string} url
      * @return {!Node}
      */
     _processURL: function(url)
     {
-        var hrefUrl = url;
-        var match = hrefUrl.match(/['"]?([^'"]+)/);
-        if (match)
-            hrefUrl = match[1];
+        if (url.length >= 2 && (url[0] == "\"" && url[url.length - 1] == "\"") || (url[0] == "'" && url[url.length - 1] == "'"))

[lushnikov, 2016/05/25 18:27:11]

1. ===
2. You can use template strings to avoid quote escaping: `"`

How do you like the regex version of this? I think it might be a little bit
easier to read:

var isQuoted = /^'.*'$/.test(url) || /^".*"$/.test(url);
if (isQuoted) {
    ...
}

[dgozman, 2016/05/25 20:19:19]

Done.

+            url = url.substring(1, url.length - 1);
         var container = createDocumentFragment();
         container.createTextChild("url(");
+        var hrefUrl = url;
         if (this._rule && this._rule.resourceURL())
             hrefUrl = WebInspector.ParsedURL.completeURL(this._rule.resourceURL(), hrefUrl);
         else if (this._node)
             hrefUrl = this._node.resolveURL(hrefUrl);
         var hasResource = hrefUrl && !!WebInspector.resourceForURL(hrefUrl);
         // FIXME: WebInspector.linkifyURLAsNode() should really use baseURI.
         container.appendChild(WebInspector.linkifyURLAsNode(hrefUrl || url, url, undefined, !hasResource));
         container.createTextChild(")");
         return container;
     }
@@ skipping 12 matching lines @@
     WebInspector.context.addFlavorChangeListener(WebInspector.DOMNode, onNodeChanged);
     onNodeChanged();
     return button;
 
     function onNodeChanged()
     {
         var node = WebInspector.context.flavor(WebInspector.DOMNode);
         button.setEnabled(!!node);
     }
 }