[SCM] WebKit Debian packaging branch, webkit-1.1, updated. upstream/1.1.15.1-1414-gc69ee75

eric at webkit.org eric at webkit.org
Thu Oct 29 20:33:04 UTC 2009


The following commit has been merged in the webkit-1.1 branch:
commit 5af10913dbe28322c6264ed836103ee7d4b12e29
Author: eric at webkit.org <eric at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Sep 24 10:42:45 2009 +0000

    2009-09-24  Benjamin Poulain  <benjamin.poulain at nokia.com>
    
            Reviewed by Eric Seidel.
    
            The indices of RuntimeArray should be enumerated like for a regular array.
            https://bugs.webkit.org/show_bug.cgi?id=29005
    
            * platform/mac/fast/dom/wrapper-classes-objc-expected.txt:
            * platform/mac/fast/dom/wrapper-classes-objc.html:
    2009-09-24  Benjamin Poulain  <benjamin.poulain at nokia.com>
    
            Reviewed by Eric Seidel.
    
            The indices of RuntimeArray should be enumerated like for a regular array.
            https://bugs.webkit.org/show_bug.cgi?id=29005
    
            * bridge/runtime_array.cpp:
            (JSC::RuntimeArray::getPropertyNames):
            * bridge/runtime_array.h:
    2009-09-24  Benjamin Poulain  <benjamin.poulain at nokia.com>
    
            Reviewed by Eric Seidel.
    
            https://bugs.webkit.org/show_bug.cgi?id=29005
            The indices of RuntimeArray should be enumerated like for a regular array.
    
            * DumpRenderTree/mac/ObjCController.m:
            (+[ObjCController isSelectorExcludedFromWebScript:]):
            (+[ObjCController webScriptNameForSelector:]):
            (-[ObjCController arrayOfString]):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@48712 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index d6246db..1aad15a 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,13 @@
+2009-09-24  Benjamin Poulain  <benjamin.poulain at nokia.com>
+
+        Reviewed by Eric Seidel.
+
+        The indices of RuntimeArray should be enumerated like for a regular array.
+        https://bugs.webkit.org/show_bug.cgi?id=29005
+
+        * platform/mac/fast/dom/wrapper-classes-objc-expected.txt:
+        * platform/mac/fast/dom/wrapper-classes-objc.html:
+
 2009-09-23  Geoffrey Garen  <ggaren at apple.com>
 
         Reviewed by Sam Weinig.
diff --git a/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc-expected.txt b/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc-expected.txt
index 103cc6d..7820552 100644
--- a/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc-expected.txt
+++ b/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc-expected.txt
@@ -192,4 +192,5 @@ PASS typeof objCObjectOfClass('NSCFNumber') is 'number'
 PASS typeof objCObjectOfClass('NSCFString') is 'string'
 PASS typeof objCObjectOfClass('WebScriptObject') is 'object'
 PASS objCObjectOfClass('NSArray') instanceof Array is true
+PASS concatenateArray(objCArrayOfString()) is 'onetwothree'
 
diff --git a/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc.html b/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc.html
index c29c26d..0cdb755 100644
--- a/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc.html
+++ b/LayoutTests/platform/mac/fast/dom/wrapper-classes-objc.html
@@ -34,6 +34,21 @@ function objCObjectOfClass(name)
     return objCController.objectOfClass(name);
 }
 
+function objCArrayOfString()
+{
+    if (!window.objCController)
+        return "only works under DumpRenderTree";
+    return objCController.arrayOfString();
+}
+
+function concatenateArray(array)
+{
+    var result = '';
+    for (i in array)
+        result += array[i];
+    return result;
+}
+
 function tagObjCWrapperClass(tagName)
 {
     return objCWrapperClass(document.createElement(tagName));
@@ -272,6 +287,8 @@ function runTest()
     shouldBe("typeof objCObjectOfClass('WebScriptObject')", "'object'");
     shouldBeTrue("objCObjectOfClass('NSArray') instanceof Array");
 
+    shouldBe("concatenateArray(objCArrayOfString())", "'onetwothree'");
+
     // Not yet tested:
 
     // CSSCharsetRule
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 3659115..cd028d1 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,14 @@
+2009-09-24  Benjamin Poulain  <benjamin.poulain at nokia.com>
+
+        Reviewed by Eric Seidel.
+
+        The indices of RuntimeArray should be enumerated like for a regular array.
+        https://bugs.webkit.org/show_bug.cgi?id=29005
+
+        * bridge/runtime_array.cpp:
+        (JSC::RuntimeArray::getPropertyNames):
+        * bridge/runtime_array.h:
+
 2009-09-23  Alexander Pavlov  <apavlov at chromium.org>
 
         Reviewed by Eric Seidel.
diff --git a/WebCore/bridge/runtime_array.cpp b/WebCore/bridge/runtime_array.cpp
index feadb07..61fbe00 100644
--- a/WebCore/bridge/runtime_array.cpp
+++ b/WebCore/bridge/runtime_array.cpp
@@ -28,6 +28,7 @@
 
 #include <runtime/ArrayPrototype.h>
 #include <runtime/Error.h>
+#include <runtime/PropertyNameArray.h>
 #include "JSDOMBinding.h"
 
 using namespace WebCore;
@@ -145,4 +146,13 @@ bool RuntimeArray::deleteProperty(ExecState*, unsigned)
     return false;
 }
 
+void RuntimeArray::getPropertyNames(ExecState* exec, PropertyNameArray& propertyNames)
+{
+    const unsigned int length = getLength();
+    for (unsigned i = 0; i < length; ++i)
+        propertyNames.add(Identifier::from(exec, i));
+
+    JSObject::getPropertyNames(exec, propertyNames);
+}
+
 }
diff --git a/WebCore/bridge/runtime_array.h b/WebCore/bridge/runtime_array.h
index f614f7f..4d68b42 100644
--- a/WebCore/bridge/runtime_array.h
+++ b/WebCore/bridge/runtime_array.h
@@ -43,6 +43,7 @@ public:
     
     virtual bool deleteProperty(ExecState *exec, const Identifier &propertyName);
     virtual bool deleteProperty(ExecState *exec, unsigned propertyName);
+    virtual void getPropertyNames(ExecState*, PropertyNameArray&);
     
     virtual const ClassInfo *classInfo() const { return &s_info; }
     
diff --git a/WebKitTools/ChangeLog b/WebKitTools/ChangeLog
index 2badadb..eb7f724 100644
--- a/WebKitTools/ChangeLog
+++ b/WebKitTools/ChangeLog
@@ -1,3 +1,15 @@
+2009-09-24  Benjamin Poulain  <benjamin.poulain at nokia.com>
+
+        Reviewed by Eric Seidel.
+
+        https://bugs.webkit.org/show_bug.cgi?id=29005
+        The indices of RuntimeArray should be enumerated like for a regular array.
+
+        * DumpRenderTree/mac/ObjCController.m:
+        (+[ObjCController isSelectorExcludedFromWebScript:]):
+        (+[ObjCController webScriptNameForSelector:]):
+        (-[ObjCController arrayOfString]):
+
 2009-09-23  David Kilzer  <ddkilzer at apple.com>
 
         <http://webkit.org/b/28910> Move bugzilla-tool mark-fixed to standalone mark-bug-fixed tool
diff --git a/WebKitTools/DumpRenderTree/mac/ObjCController.m b/WebKitTools/DumpRenderTree/mac/ObjCController.m
index e0d663e..d7cc6a0 100644
--- a/WebKitTools/DumpRenderTree/mac/ObjCController.m
+++ b/WebKitTools/DumpRenderTree/mac/ObjCController.m
@@ -58,6 +58,7 @@ static void* runJavaScriptThread(void* arg)
     if (0
             || aSelector == @selector(classNameOf:)
             || aSelector == @selector(objectOfClass:)
+            || aSelector == @selector(arrayOfString)
             || aSelector == @selector(identityIsEqual::)
             || aSelector == @selector(longLongRoundTrip:)
             || aSelector == @selector(unsignedLongLongRoundTrip:)
@@ -77,6 +78,8 @@ static void* runJavaScriptThread(void* arg)
         return @"className";
     if (aSelector == @selector(objectOfClass:))
         return @"objectOfClass";
+    if (aSelector == @selector(arrayOfString))
+        return @"arrayOfString";
     if (aSelector == @selector(identityIsEqual::))
         return @"identityIsEqual";
     if (aSelector == @selector(longLongRoundTrip:))
@@ -122,6 +125,16 @@ static void* runJavaScriptThread(void* arg)
     return nil;
 }
 
+- (NSArray *)arrayOfString
+{
+    NSString *strings[3];
+    strings[0] = @"one";
+    strings[1] = @"two";
+    strings[2] = @"three";
+    NSArray *array = [NSArray arrayWithObjects:strings count:3];
+    return array;
+}
+
 - (BOOL)identityIsEqual:(WebScriptObject *)a :(WebScriptObject *)b
 {
     if ([a isKindOfClass:[NSString class]] && [b isKindOfClass:[NSString class]])

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list