Title: [117909] trunk/Source/WebCore
Revision
117909
Author
[email protected]
Date
2012-05-21 23:05:07 -0700 (Mon, 21 May 2012)

Log 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:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (117908 => 117909)


--- trunk/Source/WebCore/ChangeLog	2012-05-22 05:58:52 UTC (rev 117908)
+++ trunk/Source/WebCore/ChangeLog	2012-05-22 06:05:07 UTC (rev 117909)
@@ -1,3 +1,17 @@
+2012-05-21  Alexis Menard  <[email protected]>
+
+        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:
+
 2012-05-21  Adam Barth  <[email protected]>
 
         !HAVE(ACCESSIBILITY) should be able to build without linking in any code from WebCore/accessibility

Modified: trunk/Source/WebCore/inspector/front-end/CSSStyleModel.js (117908 => 117909)


--- trunk/Source/WebCore/inspector/front-end/CSSStyleModel.js	2012-05-22 05:58:52 UTC (rev 117908)
+++ trunk/Source/WebCore/inspector/front-end/CSSStyleModel.js	2012-05-22 06:05:07 UTC (rev 117909)
@@ -443,16 +443,6 @@
 
     /**
      * @param {string} name
-     * @return {string}
-     */
-    getPropertyShorthand: function(name)
-    {
-        var property = this._livePropertyMap[name];
-        return property ? property.shorthand : "";
-    },
-
-    /**
-     * @param {string} name
      * @return {boolean}
      */
     isPropertyImplicit: function(name)
@@ -462,40 +452,6 @@
     },
 
     /**
-     * @return {string}
-     */
-    styleTextWithShorthands: function()
-    {
-        var cssText = "";
-        var foundProperties = {};
-        for (var i = 0; i < this.length; ++i) {
-            var individualProperty = this[i];
-            var shorthandProperty = this.getPropertyShorthand(individualProperty);
-            var propertyName = (shorthandProperty || individualProperty);
-
-            if (propertyName in foundProperties)
-                continue;
-
-            if (shorthandProperty) {
-                var value = this.getShorthandValue(shorthandProperty);
-                var priority = this.getShorthandPriority(shorthandProperty);
-            } else {
-                var value = this.getPropertyValue(individualProperty);
-                var priority = this.getPropertyPriority(individualProperty);
-            }
-
-            foundProperties[propertyName] = true;
-
-            cssText += propertyName + ": " + value;
-            if (priority)
-                cssText += " !" + priority;
-            cssText += "; ";
-        }
-
-        return cssText;
-    },
-
-    /**
      * @param {string} name
      * @return {Array.<WebInspector.CSSProperty>}
      */
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to