Summary: | Enumeration of prototypes with more than 64 properties cache not invalidated | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Sebastian Markbåge <sebastian> | ||||
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Major | CC: | jankassens, john.david.dalton, oliver, sam | ||||
Priority: | P1 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
URL: | http://labs.calyptus.eu/WebKitPrototypeCache/ | ||||||
Attachments: |
|
Description
Sebastian Markbåge
2009-06-21 16:31:50 PDT
Created attachment 31625 [details]
Test case
Committing to http://svn.webkit.org/repository/webkit/trunk ... M JavaScriptCore/ChangeLog M JavaScriptCore/interpreter/Interpreter.cpp M JavaScriptCore/jit/JITStubs.cpp M JavaScriptCore/runtime/Structure.cpp M JavaScriptCore/runtime/StructureChain.cpp M JavaScriptCore/runtime/StructureChain.h M LayoutTests/ChangeLog A LayoutTests/fast/js/dictionary-no-cache.html A LayoutTests/fast/js/resources/dictionary-no-cache.js Committed r45039 Please verify in the next nightly :D In the attachment I noticed that performing something like test2._x = 1; delete test2._x; before the for-in loop seems to fix the issue. Sebastian's test case works now here (nighly). |