Bug 154445

Summary: [EFL] Merge WebView and WebViewEfl
Product: WebKit Reporter: Ryuan Choi <ryuan.choi>
Component: WebKit EFLAssignee: Ryuan Choi <ryuan.choi>
Status: RESOLVED FIXED    
Severity: Normal CC: cgarcia, commit-queue, gyuyoung.kim, lucas.de.marchi, yoon, zan
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 152037    
Attachments:
Description Flags
Patch none

Ryuan Choi
Reported 2016-02-19 02:09:32 PST
SSIA
Attachments
Patch (43.61 KB, patch)
2016-02-19 02:17 PST, Ryuan Choi 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryuan Choi
Comment 1 2016-02-19 02:17:48 PST
Created attachment 271742 [details] Patch
Gyuyoung Kim
Comment 2 2016-02-22 22:14:49 PST
Comment on attachment 271742 [details] Patch WebViewEfl is merged into WebView because no port uses Coordinated Graphics WebView except for EFL port. And we guess no port will use it in future as well. rs=me. But I'd like to listen GTK's sounds before landing. yoon, zan, any comment on this or do you have any plan to use it ?
Gwang Yoon Hwang
Comment 3 2016-02-22 22:24:25 PST
(In reply to comment #2) > Comment on attachment 271742 [details] > Patch > > WebViewEfl is merged into WebView because no port uses Coordinated Graphics > WebView except for EFL port. And we guess no port will use it in future as > well. rs=me. > > But I'd like to listen GTK's sounds before landing. yoon, zan, any comment > on this or do you have any plan to use it ? I like this change. We are going to add UI side compositing support using Coordinated Graphics to the GTK port, but not this time. And I'll try to simplify UI sides code when GTK supports UI side compositing, so this patch makes me easier to do it.
WebKit Commit Bot
Comment 4 2016-02-22 23:58:58 PST
Comment on attachment 271742 [details] Patch Clearing flags on attachment: 271742 Committed r196972: <http://trac.webkit.org/changeset/196972>
WebKit Commit Bot
Comment 5 2016-02-22 23:59:05 PST
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.