2009-07-08 Darin Adler <darin@apple.com>
authormitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Jul 2009 04:09:04 +0000 (04:09 +0000)
committermitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Jul 2009 04:09:04 +0000 (04:09 +0000)
        Reviewed by Dan Bernstein

        - fix <rdar://problem/6971484> REGRESSION (r30534): "Scrolling Credits"
          effect does not scroll on iMovie 7.1.4 on Tiger

        In <http://trac.webkit.org/changeset/30534>, the prototype of
        JSObject::put() which UserObjectImp had been overriding changed, but
        UserObjectImp wasn't updated, so its override was no longer getting
        called after that.

        * UserObjectImp.cpp:
        (UserObjectImp::put): Match the current prototype of JSObject::put().
        The PutPropertySlot argument is unused.
        * UserObjectImp.h: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@45657 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptGlue/ChangeLog
JavaScriptGlue/UserObjectImp.cpp
JavaScriptGlue/UserObjectImp.h

index ecca987..4285bbe 100644 (file)
@@ -1,3 +1,20 @@
+2009-07-08  Darin Adler  <darin@apple.com>
+
+        Reviewed by Dan Bernstein
+
+        - fix <rdar://problem/6971484> REGRESSION (r30534): "Scrolling Credits"
+          effect does not scroll on iMovie 7.1.4 on Tiger
+
+        In <http://trac.webkit.org/changeset/30534>, the prototype of
+        JSObject::put() which UserObjectImp had been overriding changed, but
+        UserObjectImp wasn't updated, so its override was no longer getting
+        called after that.
+
+        * UserObjectImp.cpp:
+        (UserObjectImp::put): Match the current prototype of JSObject::put().
+        The PutPropertySlot argument is unused.
+        * UserObjectImp.h: Ditto.
+
 2009-06-19  Zoltan Horvath  <hzoltan@inf.u-szeged.hu>
 
         Reviewed by Darin Adler.
index b5aaeaa..cf7c278 100644 (file)
@@ -150,7 +150,7 @@ bool UserObjectImp::getOwnPropertySlot(ExecState *exec, const Identifier& proper
     return JSObject::getOwnPropertySlot(exec, propertyName, slot);
 }
 
-void UserObjectImp::put(ExecState *exec, const Identifier &propertyName, JSValue value, int attr)
+void UserObjectImp::put(ExecState *exec, const Identifier &propertyName, JSValue value, PutPropertySlot&)
 {
     if (!fJSUserObject)
         return;
index d4d0a3a..ec48758 100644 (file)
@@ -48,7 +48,7 @@ public:
 
     virtual JSValue callAsFunction(ExecState *exec, JSObject *thisObj, const ArgList &args);
     virtual bool getOwnPropertySlot(ExecState *, const Identifier&, PropertySlot&);
-    virtual void put(ExecState *exec, const Identifier &propertyName, JSValue value, int attr = None);
+    virtual void put(ExecState *exec, const Identifier &propertyName, JSValue value, PutPropertySlot&);
 
     JSValue toPrimitive(ExecState*, JSType preferredType = UnspecifiedType) const;
     virtual bool toBoolean(ExecState *exec) const;