RESOLVED FIXED 47380
[V8] Add FileWriter to active DOM map 
https://bugs.webkit.org/show_bug.cgi?id=47380
Summary [V8] Add FileWriter to active DOM map
Eric U.
Reported 2010-10-07 14:28:54 PDT
As in https://bugs.webkit.org/show_bug.cgi?id=47205, it needs to be added in CodeGeneratorV8.pm. However, it's not directly constructable, so it doesn't need the custom constructor. The ref() is done in wrapSlow(), which gets called via FileWriterCallback.
Attachments
One-liner. (1019 bytes, patch)
2010-10-07 17:58 PDT, Eric U. 		jianli: review-
DetailsFormatted DiffDiff
Fixed ChangeLog and bug title. (1021 bytes, patch)
2010-10-07 18:27 PDT, Eric U. 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Eric U.
Comment 1 2010-10-07 17:58:42 PDT
Created attachment 70181 [details] One-liner. Hope this works; I haven't created a patch manually on Windows before.
Jian Li
Comment 2 2010-10-07 18:02:06 PDT
Comment on attachment 70181 [details] One-liner. View in context: https://bugs.webkit.org/attachment.cgi?id=70181&action=review > WebCore/ChangeLog:5 > + Add FileWriter to V8 active DOM map Please add "[V8]" prefix to the bug title and also update the ChangeLog.
Eric U.
Comment 3 2010-10-07 18:27:10 PDT
Created attachment 70183 [details] Fixed ChangeLog and bug title.
WebKit Commit Bot
Comment 4 2010-10-07 19:03:28 PDT
Comment on attachment 70183 [details] Fixed ChangeLog and bug title. Clearing flags on attachment: 70183 Committed r69370: <http://trac.webkit.org/changeset/69370>
WebKit Commit Bot
Comment 5 2010-10-07 19:03:33 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.