RESOLVED FIXED 169348
[WK2] Move AcceleratedSurface files under WebProcess/WebPage/CoordinatedGraphics 
https://bugs.webkit.org/show_bug.cgi?id=169348
Summary [WK2] Move AcceleratedSurface files under WebProcess/WebPage/CoordinatedGraphics
Zan Dobersek
Reported 2017-03-08 01:38:48 PST
[WK2][CoordGraphics] Move AcceleratedSurface files under WebProcess/WebPage/CoordinatedGraphics
Attachments
Patch (12.64 KB, patch)
2017-03-08 01:40 PST, Zan Dobersek 		no flags
DetailsFormatted DiffDiff
Patch (12.74 KB, patch)
2017-03-10 09:33 PST, Zan Dobersek 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Zan Dobersek
Comment 1 2017-03-08 01:40:18 PST
Created attachment 303798 [details] Patch
Carlos Garcia Campos
Comment 2 2017-03-08 02:05:07 PST
Comment on attachment 303798 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=303798&action=review > Source/WebKit2/ChangeLog:10 > + Move the AcceleratedSurface header and implementation files under the > + WebProcess/WebPage/CoordinatedGraphics directory since that class is > + used in the platform-independent ThreadedCoordinatedLayerTreeHost. But it's not specific to coordinated graphics either, it was used by LayerTreeHostGtk before we removed it.
Zan Dobersek
Comment 3 2017-03-08 02:42:14 PST
Comment on attachment 303798 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=303798&action=review >> Source/WebKit2/ChangeLog:10 >> + used in the platform-independent ThreadedCoordinatedLayerTreeHost. > > But it's not specific to coordinated graphics either, it was used by LayerTreeHostGtk before we removed it. Who else could be using it? Nobody else is using it upstream.
Zan Dobersek
Comment 4 2017-03-10 09:33:44 PST
Created attachment 304048 [details] Patch
Zan Dobersek
Comment 5 2017-03-13 07:42:44 PDT
Comment on attachment 304048 [details] Patch Clearing flags on attachment: 304048 Committed r213844: <http://trac.webkit.org/changeset/213844>
Zan Dobersek
Comment 6 2017-03-13 07:42:54 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.