Devtools Color: Basic support for #RRGGBBAA and #RGBA

third_party/WebKit/Source/devtools/front_end/ui/ColorSwatch.js

 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  * @extends {HTMLSpanElement}
  */
 WebInspector.ColorSwatch = function()
 {
@@ skipping 117 matching lines @@
             return !color.hasAlpha() ? cf.RGB : cf.RGBA;
 
         case cf.RGB:
         case cf.RGBA:
             return !color.hasAlpha() ? cf.HSL : cf.HSLA;
 
         case cf.HSL:
         case cf.HSLA:
             if (color.nickname())
                 return cf.Nickname;
             if (!color.hasAlpha())

[dgozman, 2016/05/20 00:55:15]

I think this whole if should be just detectHEXFormat().

[samli, 2016/05/20 01:58:02]

Done.

-                return color.canBeShortHex() ? cf.ShortHEX : cf.HEX;
+                return color.detectHEXFormat() ? cf.ShortHEX : cf.HEX;
             else
                 return cf.Original;
 
         case cf.ShortHEX:
             return cf.HEX;
 
         case cf.HEX:
             return cf.Original;
 
         case cf.Nickname:
             if (!color.hasAlpha())

[dgozman, 2016/05/20 00:55:15]

Ditto.

[samli, 2016/05/20 01:58:02]

Done.

-                return color.canBeShortHex() ? cf.ShortHEX : cf.HEX;
+                return color.detectHEXFormat() ? cf.ShortHEX : cf.HEX;
             else
                 return cf.Original;
 
         default:
             return cf.RGBA;
     }
 }