Changeset 117909 in webkit


Ignore:
Timestamp:
May 21, 2012 11:05:07 PM (12 years ago)
Author:
alexis.menard@openbossa.org
Message:

Web Inspector: Remove some dead code in CSSStyleModel.js.
https://bugs.webkit.org/show_bug.cgi?id=87048

Reviewed by Pavel Feldman.

Remove dead and unused code in CSSStyleModel.js.

No new tests : the code is supposedly dead so we should not have
any regressions.

  • inspector/front-end/CSSStyleModel.js:
Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r117907 r117909  
     12012-05-21  Alexis Menard  <alexis.menard@openbossa.org>
     2
     3        Web Inspector: Remove some dead code in CSSStyleModel.js.
     4        https://bugs.webkit.org/show_bug.cgi?id=87048
     5
     6        Reviewed by Pavel Feldman.
     7
     8        Remove dead and unused code in CSSStyleModel.js.
     9
     10        No new tests : the code is supposedly dead so we should not have
     11        any regressions.
     12
     13        * inspector/front-end/CSSStyleModel.js:
     14
    1152012-05-21  Adam Barth  <abarth@webkit.org>
    216
  • trunk/Source/WebCore/inspector/front-end/CSSStyleModel.js

    r117755 r117909  
    444444    /**
    445445     * @param {string} name
    446      * @return {string}
    447      */
    448     getPropertyShorthand: function(name)
    449     {
    450         var property = this._livePropertyMap[name];
    451         return property ? property.shorthand : "";
    452     },
    453 
    454     /**
    455      * @param {string} name
    456446     * @return {boolean}
    457447     */
     
    460450        var property = this._livePropertyMap[name];
    461451        return property ? property.implicit : "";
    462     },
    463 
    464     /**
    465      * @return {string}
    466      */
    467     styleTextWithShorthands: function()
    468     {
    469         var cssText = "";
    470         var foundProperties = {};
    471         for (var i = 0; i < this.length; ++i) {
    472             var individualProperty = this[i];
    473             var shorthandProperty = this.getPropertyShorthand(individualProperty);
    474             var propertyName = (shorthandProperty || individualProperty);
    475 
    476             if (propertyName in foundProperties)
    477                 continue;
    478 
    479             if (shorthandProperty) {
    480                 var value = this.getShorthandValue(shorthandProperty);
    481                 var priority = this.getShorthandPriority(shorthandProperty);
    482             } else {
    483                 var value = this.getPropertyValue(individualProperty);
    484                 var priority = this.getPropertyPriority(individualProperty);
    485             }
    486 
    487             foundProperties[propertyName] = true;
    488 
    489             cssText += propertyName + ": " + value;
    490             if (priority)
    491                 cssText += " !" + priority;
    492             cssText += "; ";
    493         }
    494 
    495         return cssText;
    496452    },
    497453
Note: See TracChangeset for help on using the changeset viewer.