DevTools: represent ComputedStyle with simple Map

Source/devtools/front_end/sdk/CSSStyleModel.js

 /*
  * Copyright (C) 2010 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 149 matching lines @@
                 }
             }
             return new WebInspector.CSSStyleModel.MatchedStyleResult(matchedRules, inherited, pseudoElements);
         }
 
         return this._agent.getMatchedStylesForNode(nodeId, excludePseudo, excludeInherited, callback.bind(this));
     },
 
     /**
      * @param {!DOMAgent.NodeId} nodeId
-     * @return {!Promise.<?WebInspector.CSSStyleDeclaration>}
+     * @return {!Promise.<?Map.<string, string>>}
      */
     computedStylePromise: function(nodeId)
     {
         return this._styleLoader.computedStylePromise(nodeId);
     },
 
     /**
      * @param {number} nodeId
      * @return {!Promise.<?Array.<!CSSAgent.PlatformFontUsage>>}
      */
@@ skipping 528 matching lines @@
 /**
  * @param {!WebInspector.CSSStyleModel} cssModel
  * @param {!CSSAgent.CSSStyle} payload
  * @return {!WebInspector.CSSStyleDeclaration}
  */
 WebInspector.CSSStyleDeclaration.parsePayload = function(cssModel, payload)
 {
     return new WebInspector.CSSStyleDeclaration(cssModel, payload);
 }
 
-/**
- * @param {!WebInspector.CSSStyleModel} cssModel
- * @param {!Array.<!CSSAgent.CSSComputedStyleProperty>} payload
- * @return {!WebInspector.CSSStyleDeclaration}
- */
-WebInspector.CSSStyleDeclaration.parseComputedStylePayload = function(cssModel, payload)
-{
-    var newPayload = /** @type {!CSSAgent.CSSStyle} */ ({ cssProperties: [], shorthandEntries: [], width: "", height: "" });
-    if (payload)
-        newPayload.cssProperties = /** @type {!Array.<!CSSAgent.CSSProperty>} */ (payload);
-
-    return new WebInspector.CSSStyleDeclaration(cssModel, newPayload);
-}
-
 WebInspector.CSSStyleDeclaration.prototype = {
     /**
      * @return {!WebInspector.Target}
      */
     target: function()
     {
         return this._cssModel.target();
     },
 
     /**
@@ skipping 1141 matching lines @@
 WebInspector.CSSStyleModel.ComputedStyleLoader = function(cssModel)
 {
     this._cssModel = cssModel;
     /** @type {!Map.<!DOMAgent.NodeId, !Promise.<?WebInspector.CSSStyleDeclaration>>} */
     this._nodeIdToPromise = new Map();
 }
 
 WebInspector.CSSStyleModel.ComputedStyleLoader.prototype = {
     /**
      * @param {!DOMAgent.NodeId} nodeId
-     * @return {!Promise.<?WebInspector.CSSStyleDeclaration>}
+     * @return {!Promise.<?Map.<string, string>>}
      */
     computedStylePromise: function(nodeId)
     {
         if (!this._nodeIdToPromise[nodeId])
-            this._nodeIdToPromise[nodeId] = this._cssModel._agent.getComputedStyleForNode(nodeId, parsePayload.bind(this)).then(cleanUp.bind(this));
+            this._nodeIdToPromise[nodeId] = this._cssModel._agent.getComputedStyleForNode(nodeId, parsePayload).then(cleanUp.bind(this));
 
         return this._nodeIdToPromise[nodeId];
 
         /**
          * @param {?Protocol.Error} error
          * @param {!Array.<!CSSAgent.CSSComputedStyleProperty>} computedPayload
-         * @return {?WebInspector.CSSStyleDeclaration}
-         * @this {WebInspector.CSSStyleModel.ComputedStyleLoader}
+         * @return {?Map.<string, string>}
          */
         function parsePayload(error, computedPayload)
         {
-            return !error && computedPayload ? WebInspector.CSSStyleDeclaration.parseComputedStylePayload(this._cssModel, computedPayload) : null;
+            if (error || !computedPayload)
+                return null;
+            var result = new Map();
+            for (var property of computedPayload)
+                result.set(property.name, property.value);
+            return result;
         }
 
         /**
-         * @param {?WebInspector.CSSStyleDeclaration} computedStyle
-         * @return {?WebInspector.CSSStyleDeclaration}
+         * @param {?Map.<string, string>} computedStyle
+         * @return {?Map.<string, string>}
          * @this {WebInspector.CSSStyleModel.ComputedStyleLoader}
          */
         function cleanUp(computedStyle)
         {
             delete this._nodeIdToPromise[nodeId];
             return computedStyle;
         }
     }
 }
 
@@ skipping 56 matching lines @@
  * @constructor
  * @param {?WebInspector.CSSStyleDeclaration} inlineStyle
  * @param {?WebInspector.CSSStyleDeclaration} attributesStyle
  */
 WebInspector.CSSStyleModel.InlineStyleResult = function(inlineStyle, attributesStyle)
 {
     this.inlineStyle = inlineStyle;
     this.attributesStyle = attributesStyle;
 }