Bug 172496 - [WPE] Use AcceleratedDrawingArea instead of its fork
Summary: [WPE] Use AcceleratedDrawingArea instead of its fork
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 172642
Blocks:
  Show dependency treegraph
 
Reported: 2017-05-23 02:23 PDT by Carlos Garcia Campos
Modified: 2017-06-05 05:02 PDT (History)
3 users (show)

See Also:


Attachments
Patch (24.40 KB, patch)
2017-05-23 02:26 PDT, Carlos Garcia Campos
zan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Garcia Campos 2017-05-23 02:23:00 PDT
WPE uses its own drawing area implementation, which is actually a fork of AcceleratedDrawingArea, but simplified for the case of compositing being always forced. AcceleratedDrawingArea already handles the case of compositing being forced, so now that WPE is upstream we could simply use AcceleratedDrawingArea instead.
Comment 1 Carlos Garcia Campos 2017-05-23 02:26:51 PDT
Created attachment 310988 [details]
Patch
Comment 2 Gwang Yoon Hwang 2017-05-23 03:53:15 PDT
Comment on attachment 310988 [details]
Patch

I'm really okay for this change!
Comment 3 Carlos Garcia Campos 2017-05-26 01:07:40 PDT
Committed r217479: <http://trac.webkit.org/changeset/217479>
Comment 4 WebKit Commit Bot 2017-05-26 06:25:56 PDT
Re-opened since this is blocked by bug 172642
Comment 5 Zan Dobersek 2017-06-05 00:16:05 PDT
This is now fine to land with the forceRepaint() being made exclusive to PLATFORM(GTK).
Comment 6 Carlos Garcia Campos 2017-06-05 05:02:07 PDT
Committed r217779: <http://trac.webkit.org/changeset/217779>