RESOLVED FIXED 133056
Reduce constructor copypasta in RenderText and RenderElement. 
https://bugs.webkit.org/show_bug.cgi?id=133056
Summary Reduce constructor copypasta in RenderText and RenderElement.
Andreas Kling
Reported 2014-05-18 14:59:18 PDT
We can use delegating constructors to avoid duplicating some initializer lists.
Attachments
Patch (5.31 KB, patch)
2014-05-18 15:12 PDT, Andreas Kling 		andersca: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2014-05-18 15:12:32 PDT
Created attachment 231667 [details] Patch
Darin Adler
Comment 2 2014-05-18 20:50:52 PDT
Comment on attachment 231667 [details] Patch All our compilers support delegating constructors!? Hooray! What about inlining? Is it OK to add another level of function call here?
Andreas Kling
Comment 3 2014-05-18 23:25:25 PDT
(In reply to comment #2) > (From update of attachment 231667 [details]) > All our compilers support delegating constructors!? Hooray! > > What about inlining? Is it OK to add another level of function call here? Looks like they do! I wasn't sure so wanted to wait for EWS. I'll slap an "inline" on the delegate ctor's definition for good measure.
Andreas Kling
Comment 4 2014-05-18 23:45:07 PDT
Committed r169038: <http://trac.webkit.org/changeset/169038>
Note You need to log in before you can comment on or make changes to this bug.