WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 27871
[commit+] Allow custom memory allocation control for JavaScriptCore's HashMap class
https://bugs.webkit.org/show_bug.cgi?id=27871
Summary
[commit+] Allow custom memory allocation control for JavaScriptCore's HashMap...
Zoltan Horvath
Reported
2009-07-31 02:21:35 PDT
Inherits HashMap class from FastAllocBase because it has been instantiated by 'new' in JavaScriptCore/API/JSClassRef.cpp:148.
Attachments
proposed patch
(1.32 KB, patch)
2009-07-31 02:24 PDT
,
Zoltan Horvath
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Zoltan Horvath
Comment 1
2009-07-31 02:24:35 PDT
Created
attachment 33864
[details]
proposed patch
Darin Adler
Comment 2
2009-07-31 09:29:41 PDT
Comment on
attachment 33864
[details]
proposed patch
> + * wtf/RefPtrHashMap.h: > + (WTF::):
Please don't leave broken lines like this in the change log just because the prepare-ChangeLog script put them there.
Eric Seidel (no email)
Comment 3
2009-07-31 22:02:14 PDT
I had this failure again: tables/mozilla/bugs/bug2479-4.html -> failed Starting to think it might not be related to these patches.
Adam Barth
Comment 4
2009-08-01 11:35:59 PDT
Comment on
attachment 33864
[details]
proposed patch Clearing review flag on attachment: 33864 Committing to
http://svn.webkit.org/repository/webkit/trunk
... M JavaScriptCore/ChangeLog M JavaScriptCore/wtf/RefPtrHashMap.h Committed
r46675
A LayoutTests/platform/mac-tiger/http/tests/misc/favicon-loads-with-images-disabled-expected.txt M LayoutTests/platform/win/Skipped M LayoutTests/ChangeLog
r46674
= d76ae4ee615ab1d3fa82d062c8ef4299bd89bd76 (trunk) M JavaScriptCore/wtf/RefPtrHashMap.h M JavaScriptCore/ChangeLog
r46675
= a3b8eb2865ca4b26d3b75bc0f0e72e669b2e99ce (trunk) First, rewinding head to replay your work on top of it... Nothing to do.
http://trac.webkit.org/changeset/46675
Adam Barth
Comment 5
2009-08-01 11:36: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