Bug 44507 - Partial deployment of adoptPtr to WebCore/html
Summary: Partial deployment of adoptPtr to WebCore/html
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Adam Barth
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-24 02:25 PDT by Adam Barth
Modified: 2010-08-24 11:33 PDT (History)
1 user (show)

See Also:


Attachments
Patch (6.42 KB, patch)
2010-08-24 02:29 PDT, Adam Barth
no flags Details | Formatted Diff | Diff
Patch (9.08 KB, patch)
2010-08-24 02:41 PDT, Adam Barth
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2010-08-24 02:25:20 PDT
Partial deployment of adoptPtr to WebCore/html
Comment 1 Adam Barth 2010-08-24 02:29:31 PDT
Created attachment 65245 [details]
Patch
Comment 2 Adam Barth 2010-08-24 02:41:49 PDT
Created attachment 65248 [details]
Patch
Comment 3 Adam Barth 2010-08-24 02:42:11 PDT
Comment on attachment 65245 [details]
Patch

This patch isn't obsolete.  We want both.
Comment 4 Eric Seidel (no email) 2010-08-24 07:29:19 PDT
Comment on attachment 65245 [details]
Patch

These all belong as create() functions. :(
Comment 5 Eric Seidel (no email) 2010-08-24 07:30:31 PDT
Comment on attachment 65248 [details]
Patch

OK.
Comment 6 Adam Barth 2010-08-24 11:31:43 PDT
Comment on attachment 65245 [details]
Patch

Clearing flags on attachment: 65245

Committed r65916: <http://trac.webkit.org/changeset/65916>
Comment 7 Adam Barth 2010-08-24 11:33:39 PDT
Comment on attachment 65248 [details]
Patch

Clearing flags on attachment: 65248

Committed r65917: <http://trac.webkit.org/changeset/65917>
Comment 8 Adam Barth 2010-08-24 11:33:44 PDT
All reviewed patches have been landed.  Closing bug.