RESOLVED FIXED Bug 161136
More cleanup for the mpadded implementation 
https://bugs.webkit.org/show_bug.cgi?id=161136
Summary More cleanup for the mpadded implementation
Frédéric Wang (:fredw)
Reported 2016-08-24 06:31:47 PDT
It seems that the m_ascent member is not really needed as the ascent can be easily calculated in firstLineBaseLine() after some adjustment with how the attribute values are resolved. I was not able to find a testcase that shows how it improves dynamic update. Actually, it seems that we no longer need to override styleDidChange and updateFromElement so I'll remove them in that bug too.
Attachments
Patch (11.53 KB, patch)
2016-08-24 06:47 PDT, Frédéric Wang (:fredw) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Frédéric Wang (:fredw)
Comment 1 2016-08-24 06:47:20 PDT
Created attachment 286850 [details] Patch
WebKit Commit Bot
Comment 2 2016-08-28 01:27:55 PDT
Comment on attachment 286850 [details] Patch Clearing flags on attachment: 286850 Committed r205098: <http://trac.webkit.org/changeset/205098>
WebKit Commit Bot
Comment 3 2016-08-28 01:28:02 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.