DevTools: Add shadow-editor swatch/icon before box-shadows and text-shadows

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 1935 matching lines @@
      * @param {string} text
      * @return {!Node}
      */
     _processColor: function(text)
     {
         // We can be called with valid non-color values of |text| (like 'none' from border style)
         var color = WebInspector.Color.parse(text);
         if (!color)
             return createTextNode(text);
 
-        if (!this._editable()) {
-            var swatch = WebInspector.ColorSwatch.create();
-            swatch.setColorText(text);
-            return swatch;
-        }
-
         var swatchPopoverHelper = this._parentPane._swatchPopoverHelper;
         var swatchIcon = new WebInspector.ColorSwatchPopoverIcon(this, swatchPopoverHelper, text);
 
         /**
          * @param {?Array<string>} backgroundColors
          */
         function computedCallback(backgroundColors)
         {
             // TODO(aboxhall): distinguish between !backgroundColors (no text) and
             // !backgroundColors.length (no computed bg color)
@@ skipping 19 matching lines @@
 
         if (this.property.name === "color" && this._parentPane.cssModel() && this.node()) {
             var cssModel = this._parentPane.cssModel();
             cssModel.backgroundColorsPromise(this.node().id).then(computedCallback);
         }
 
         return swatchIcon.element();
     },
 
     /**
+     * @param {string} text
+     * @return {!Node}
+     */
+    _processColorUneditable: function(text)
+    {
+        // We can be called with valid non-color values of |text| (like 'none' from border style)
+        if (!WebInspector.Color.parse(text))
+            return createTextNode(text);
+        var swatch = WebInspector.ColorSwatch.create();
+        swatch.setColorText(text);
+        return swatch;
+    },
+
+    /**
      * @return {string}
      */
     renderedPropertyText: function()
     {
         return this.nameElement.textContent + ": " + this.valueElement.textContent;
     },
 
     /**
      * @param {string} text
      * @return {!Node}
      */
     _processBezier: function(text)
     {
-        var geometry = WebInspector.Geometry.CubicBezier.parse(text);
-        if (!geometry || !this._editable())
+        if (!WebInspector.Geometry.CubicBezier.parse(text))
             return createTextNode(text);
         var swatchPopoverHelper = this._parentPane._swatchPopoverHelper;
         return new WebInspector.BezierPopoverIcon(this, swatchPopoverHelper, text).element();
     },
 
+    /**
+     * @param {string} propertyValue
+     * @param {string} propertyName
+     * @return {!Node}
+     */
+    _processShadow: function(propertyValue, propertyName)
+    {
+        var shadowValues = [];
+        // Split by commas that aren't inside of color values to get the individual shadow values.
+        var results = WebInspector.TextUtils.splitStringByRegexes(propertyValue, [WebInspector.Color.Regex, /,/g]);

[lushnikov, 2016/08/10 19:29:02]

we should put this logic somewhere else - we might want to reuse it in sources
panel

[flandy, 2016/08/11 00:23:19]

Done.

+        var currentIndex = 0;
+        for (var i = 0; i < results.length; i++) {
+            if (results[i].regexIndex === 1) {
+                var comma = results[i];
+                shadowValues.push(propertyValue.substring(currentIndex, comma.position));
+                currentIndex = comma.position + 1;
+            }
+        }
+        shadowValues.push(propertyValue.substring(currentIndex, propertyValue.length));
+
+        var container = createDocumentFragment();
+        for (var i = 0; i < shadowValues.length; i++) {
+            if (i !== 0)
+                container.appendChild(createTextNode(",")); // Add back the removed commas.
+            var shadowPopoverIcon = new WebInspector.ShadowPopoverIcon(this, this._parentPane._swatchPopoverHelper, shadowValues[i], propertyName);
+            container.appendChild(shadowPopoverIcon.element());
+        }
+        return container;
+    },
+
     _updateState: function()
     {
         if (!this.listItemElement)
             return;
 
         if (this._style.isPropertyImplicit(this.name))
             this.listItemElement.classList.add("implicit");
         else
             this.listItemElement.classList.remove("implicit");
 
@@ skipping 141 matching lines @@
     },
 
     updateTitle: function()
     {
         this._updateState();
         this._expandElement = createElement("span");
         this._expandElement.className = "expand-element";
 
         var propertyRenderer = new WebInspector.StylesSidebarPropertyRenderer(this._style.parentRule, this.node(), this.name, this.value);
         if (this.property.parsedOk) {
-            propertyRenderer.setColorHandler(this._processColor.bind(this));
-            propertyRenderer.setBezierHandler(this._processBezier.bind(this));
+            if (this._editable()) {
+                propertyRenderer.setColorHandler(this._processColor.bind(this));
+                propertyRenderer.setBezierHandler(this._processBezier.bind(this));
+                propertyRenderer.setShadowHandler(this._processShadow.bind(this));
+            } else {
+                propertyRenderer.setColorHandler(this._processColorUneditable.bind(this));

[lushnikov, 2016/08/10 19:29:02]

ideally, we should show bezier and shadow swatches for read-only rules as well;
they
just should not be editable.

This is good thing to address in a follow-up

[flandy, 2016/08/11 00:23:19]

Okay let's hold off on this for now.

+            }
         }
 
         this.listItemElement.removeChildren();
         this.nameElement = propertyRenderer.renderName();
         this.valueElement = propertyRenderer.renderValue();
         if (!this.treeOutline)
             return;
 
         var indent = WebInspector.moduleSetting("textEditorIndent").get();
         this.listItemElement.createChild("span", "styles-clipboard-only").createTextChild(indent + (this.property.disabled ? "/* " : ""));
@@ skipping 791 matching lines @@
 
     /**
      * @param {function(string):!Node} handler
      */
     setBezierHandler: function(handler)
     {
         this._bezierHandler = handler;
     },
 
     /**
+     * @param {function(string, string):!Node} handler
+     */
+    setShadowHandler: function(handler)
+    {
+        this._shadowHandler = handler;
+    },
+
+    /**
      * @return {!Element}
      */
     renderName: function()
     {
         var nameElement = createElement("span");
         nameElement.className = "webkit-css-property";
         nameElement.textContent = this._propertyName;
         nameElement.normalize();
         return nameElement;
     },
 
     /**
      * @return {!Element}
      */
     renderValue: function()
     {
         var valueElement = createElement("span");
         valueElement.className = "value";
         if (!this._propertyValue)
             return valueElement;
 
+        if (this._shadowHandler && (this._propertyName === "box-shadow" || this._propertyName === "text-shadow")
+                && !WebInspector.CSSMetadata.VariableRegex.test(this._propertyValue) && Runtime.experiments.isEnabled("shadowEditor")) {
+            valueElement.appendChild(this._shadowHandler(this._propertyValue, this._propertyName));

[lushnikov, 2016/08/10 19:29:02]

does this mean there could be only one shadow swatch in value?

[flandy, 2016/08/11 00:23:19]

No, value could have multiple shadow swatches if _propertyValue has multiple
shadows separated by commas. 

Should these be plural, _shadowsHandler? _processShadows?

+            valueElement.normalize();
+            return valueElement;
+        }
+
         var regexes = [WebInspector.CSSMetadata.VariableRegex, WebInspector.CSSMetadata.URLRegex];
         var processors = [createTextNode, this._processURL.bind(this)];
         if (this._bezierHandler && WebInspector.cssMetadata().isBezierAwareProperty(this._propertyName)) {
             regexes.push(WebInspector.Geometry.CubicBezier.Regex);
             processors.push(this._bezierHandler);
         }
         if (this._colorHandler && WebInspector.cssMetadata().isColorAwareProperty(this._propertyName)) {
             regexes.push(WebInspector.Color.Regex);
             processors.push(this._colorHandler);
         }
@@ skipping 73 matching lines @@
 
     /**
      * @override
      * @return {!WebInspector.ToolbarItem}
      */
     item: function()
     {
         return this._button;
     }
 }