Summary: | Create SVG and WML elements with create functions instead of direct invocations of new | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Darin Adler <darin> | ||||||
Component: | SVG | Assignee: | Darin Adler <darin> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | darin, dglazkov, eric, gustavo, mjs, webkit-ews, webkit.review.bot, xan.lopez, zimmermann | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 28068 | ||||||||
Attachments: |
|
Description
Darin Adler
2010-08-29 20:09:58 PDT
Created attachment 65872 [details]
Patch
Created attachment 65873 [details]
Patch
Attachment 65872 [details] did not build on qt: Build output: http://queues.webkit.org/results/3882133 Attachment 65872 [details] did not build on mac: Build output: http://queues.webkit.org/results/3815144 Attachment 65872 [details] did not build on chromium: Build output: http://queues.webkit.org/results/3890031 Comment on attachment 65873 [details]
Patch
r=me
Your ChangeLog comments imply that you left some constructors public, if so, why?
Attachment 65872 [details] did not build on gtk: Build output: http://queues.webkit.org/results/3887113 (In reply to comment #6) > Your ChangeLog comments imply that you left some constructors public, if so, why? Just wanted to limit the size of the change this round. I’ll make sure to make all of the constructors private or protected before changing them to call adoptRef. Committed r66362: <http://trac.webkit.org/changeset/66362> |