WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
41176
Fix testapi.c to account for movable objects
https://bugs.webkit.org/show_bug.cgi?id=41176
Summary
Fix testapi.c to account for movable objects
Nathan Lawrence
Reported
2010-06-24 14:22:02 PDT
Created
attachment 59701
[details]
Patch We assume in testapi.c that the value aHeapRef refers to will not be moved. When we have movable objects, this will not be the case.
Attachments
Patch
(1.96 KB, patch)
2010-06-24 14:22 PDT
,
Nathan Lawrence
ggaren
: review+
ggaren
: commit-queue+
Details
Formatted Diff
Diff
Patch
(2.14 KB, patch)
2010-06-25 11:45 PDT
,
Nathan Lawrence
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Geoffrey Garen
Comment 1
2010-06-24 15:53:29 PDT
Comment on
attachment 59701
[details]
Patch r=me
Oliver Hunt
Comment 2
2010-06-24 23:45:49 PDT
Comment on
attachment 59701
[details]
Patch aStackRef should be cleared immediately after JSObjectSetPrivateProperty as the private storage should then keep the heap reference live.
Oliver Hunt
Comment 3
2010-06-24 23:48:06 PDT
Basically because aHeapRef needs to be pinned somehow -- realistically we'll need some kind of layer for API objects to GC allocate handles i guess :-/
Nathan Lawrence
Comment 4
2010-06-25 11:45:12 PDT
Created
attachment 59784
[details]
Patch
Nathan Lawrence
Comment 5
2010-06-25 11:46:09 PDT
Updated per Oliver's comment.
Geoffrey Garen
Comment 6
2010-06-25 11:54:13 PDT
(In reply to
comment #3
)
> Basically because aHeapRef needs to be pinned somehow -- realistically we'll need some kind of layer for API objects to GC allocate handles i guess :-/
The current plan is just not to move objects pointed to by the stack.
Geoffrey Garen
Comment 7
2010-06-25 16:54:26 PDT
Comment on
attachment 59784
[details]
Patch r=me
WebKit Commit Bot
Comment 8
2010-06-25 23:25:58 PDT
Comment on
attachment 59784
[details]
Patch Clearing flags on attachment: 59784 Committed
r61946
: <
http://trac.webkit.org/changeset/61946
>
WebKit Commit Bot
Comment 9
2010-06-25 23:26:03 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug