Summary: | operationPutById's can potentially generate code for an out of date structure | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Matthew Mirman <mmirman> | ||||||
Component: | JavaScriptCore | Assignee: | Matthew Mirman <mmirman> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, dino, fpizlo, ggaren, kling, mmirman | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | 139424 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Matthew Mirman
2014-12-02 14:07:50 PST
Created attachment 242451 [details]
operationPutByIds fix
Comment on attachment 242451 [details]
operationPutByIds fix
Can you add a test that would fail without your patch?
Created attachment 242584 [details]
operationPutByIds fix
Adds a test that fails without the patch.
Comment on attachment 242584 [details] operationPutByIds fix Clearing flags on attachment: 242584 Committed r176979: <http://trac.webkit.org/changeset/176979> All reviewed patches have been landed. Closing bug. I think this broke some JS tests: https://build.webkit.org/builders/Apple%20Yosemite%20Release%20WK2%20%28Tests%29/builds/1158 Re-opened since this is blocked by bug 139424 Committed a patch similar to the reviewed patch https://bug-139196-attachments.webkit.org/attachment.cgi?id=242584 but without the failing test. Committed r176979: http://trac.webkit.org/changeset/177083 Test added from the patch https://bugs.webkit.org/attachment.cgi?id=242584&action=edit is now included in <https://bugs.webkit.org/show_bug.cgi?id=139500> |