Review Request: KJS: Implement Object.prototypeOf, Object.keys, Object.getOwnPropertyNames

Bernd Buschinski b.buschinski at googlemail.com
Thu Apr 26 17:34:41 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104511/
-----------------------------------------------------------

(Updated April 26, 2012, 4:34 p.m.)


Review request for kdelibs.


Changes
-------

- droped default argument
- fixed typo that make compile in debug fail

checkEnumerable is problematic to make it a inline function.
As this would requiere to inlcude "object.h" in property_map.h, but object.h already includes property_map.h.


Description
-------

KJS: Implement Object.prototypeOf, Object.keys, Object.getOwnPropertyNames

NOTE: Array was left out on purpose, as currentl imeplementation does not save attributes (next patch will fix this)

keys&GetOwnPropertyNames requieres to change the JSObject::getOwnPropertyNames implementation,
for future use a enum is better than than a bool, maybe there will be more ways to include/exclude properties.

All changes for khtml/ecma/ are to silense the -Woverloaded-virtual warnings


Diffs (updated)
-----

  kjs/array_instance.cpp fe9b8b4 
  kjs/function.h 3757fe8 
  kjs/array_instance.h 3f2b630 
  kjs/JSVariableObject.cpp b00ef76 
  khtml/ecma/kjs_scriptable.h af5343c 
  khtml/ecma/kjs_scriptable.cpp 5d4ea68 
  kjs/JSVariableObject.h a8f01eb 
  khtml/ecma/debugger/localvariabledock.cpp 289c910 
  khtml/ecma/kjs_css.h aba44b8 
  khtml/ecma/kjs_css.cpp e3e7417 
  khtml/ecma/kjs_data.cpp df6fa85 
  khtml/ecma/kjs_dom.h d0433c3 
  khtml/ecma/kjs_dom.cpp 5fff7e3 
  khtml/ecma/kjs_html.h 0f3f544c 
  khtml/ecma/kjs_html.cpp e3da95c 
  kjs/function.cpp 5f39ae6 
  kjs/object.h 047c242 
  kjs/object.cpp c19122f 
  kjs/object_object.cpp 986f03f 
  kjs/property_map.h 6b127ff 
  kjs/property_map.cpp b2ff08e 
  kjs/string_object.h e890d52 
  kjs/string_object.cpp 170e2f7 

Diff: http://git.reviewboard.kde.org/r/104511/diff/


Testing
-------

ecma script & daily surfing


Thanks,

Bernd Buschinski

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120426/8b23f173/attachment.htm>


More information about the kde-core-devel mailing list