RESOLVED INVALID181140
computeTextPaintStyle() does not need to know about frame 
https://bugs.webkit.org/show_bug.cgi?id=181140
Summary computeTextPaintStyle() does not need to know about frame
Daniel Bates
Reported 2017-12-22 14:59:43 PST
computeTextPaintStyle() should take the viewport size instead of a frame as it is unnecessary for it to know about a frame.
Attachments
Patch (5.37 KB, patch)
2017-12-22 15:05 PST, Daniel Bates 		dbates: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Daniel Bates
Comment 1 2017-12-22 15:05:47 PST
Created attachment 330148 [details] Patch
Daniel Bates
Comment 2 2017-12-22 15:14:38 PST
Comment on attachment 330148 [details] Patch r-, this won't work :( we need the frame to also query for the focus controller and to query the document if it is printing
Daniel Bates
Comment 3 2017-12-22 15:28:17 PST
One way to fix this is to have computeTextPaintStyle() take individual arguments for whether the document is printing, whether we should print backgrounds and whether the page is currently focused. This does not sound like a big improvement.
Note You need to log in before you can comment on or make changes to this bug.