[SCM] WebKit Debian packaging branch, webkit-1.1, updated. upstream/1.1.21-584-g1e41756

jberlin at webkit.org jberlin at webkit.org
Fri Feb 26 22:21:52 UTC 2010


The following commit has been merged in the webkit-1.1 branch:
commit 4b41ab18290f886b75a37c73fac581fff1e065fa
Author: jberlin at webkit.org <jberlin at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Wed Feb 17 02:25:47 2010 +0000

    Inspector should display Array lengths in the property lists.
    https://bugs.webkit.org/show_bug.cgi?id=20695
    
    Reviewed by Timothy Hatcher.
    
    * inspector/front-end/InjectedScript.js:
    (injectedScriptConstructor):
    For objects of type 'array' (which includes Arrays, NodeLists, and HTMLCollections), make the length of the object available via propertyLength on the proxy object.
    * inspector/front-end/ObjectPropertiesSection.js:
    (WebInspector.ObjectPropertyTreeElement.prototype.update):
    For properties that have propertyLength defined, append it to the displayed description.
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@54856 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 3a6411d..54823a9 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,17 @@
+2010-02-16  Jessie Berlin  <jberlin at webkit.org>
+
+        Reviewed by Timothy Hatcher.
+
+        Inspector should display Array lengths in the property lists.
+        https://bugs.webkit.org/show_bug.cgi?id=20695
+
+        * inspector/front-end/InjectedScript.js:
+        (injectedScriptConstructor):
+        For objects of type 'array' (which includes Arrays, NodeLists, and HTMLCollections), make the length of the object available via propertyLength on the proxy object.
+        * inspector/front-end/ObjectPropertiesSection.js:
+        (WebInspector.ObjectPropertyTreeElement.prototype.update):
+        For properties that have propertyLength defined, append it to the displayed description.
+
 2010-02-16  Yusuke Sato  <yusukes at chromium.org>
 
         Reviewed by Dimitri Glazkov.
diff --git a/WebCore/inspector/front-end/InjectedScript.js b/WebCore/inspector/front-end/InjectedScript.js
index 20e440a..f248b91 100644
--- a/WebCore/inspector/front-end/InjectedScript.js
+++ b/WebCore/inspector/front-end/InjectedScript.js
@@ -1150,6 +1150,8 @@ InjectedScript.createProxyObject = function(object, objectId, abbreviate)
     result.injectedScriptId = injectedScriptId;
     result.objectId = objectId;
     result.type = InjectedScript._type(object);
+    if (result.type === "array")
+        result.propertyLength = object.length;
 
     var type = typeof object;
     if ((type === "object" && object !== null) || type === "function") {
diff --git a/WebCore/inspector/front-end/ObjectPropertiesSection.js b/WebCore/inspector/front-end/ObjectPropertiesSection.js
index f06a14d..a32e799 100644
--- a/WebCore/inspector/front-end/ObjectPropertiesSection.js
+++ b/WebCore/inspector/front-end/ObjectPropertiesSection.js
@@ -178,6 +178,8 @@ WebInspector.ObjectPropertyTreeElement.prototype = {
         this.valueElement = document.createElement("span");
         this.valueElement.className = "value";
         this.valueElement.textContent = this.property.value.description;
+        if (typeof this.property.value.propertyLength !== "undefined")
+            this.valueElement.textContent += " (" + this.property.value.propertyLength + ")";
         if (this.property.isGetter)
             this.valueElement.addStyleClass("dimmed");
         if (this.property.isError)

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list