Bug 89838

Summary: [WK2][Qt][EFL] Move LayerBackingStore into common directory
Product: WebKit Reporter: YoungTaeck Song <youngtaeck.song>
Component: WebKit EFLAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: cmarcelo, gyuyoung.kim, lucas.de.marchi, menard, noam, seojin.kim, webkit.review.bot, zoltan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Linux   
Bug Depends on:    
Bug Blocks: 89840    
Attachments:
Description Flags
patch
none
Patch
noam: review-, noam: commit-queue-
patch none

Description YoungTaeck Song 2012-06-24 17:12:46 PDT
Move LayerBackingStore into common directory ‘Source/WebKit2/UIProcess/’ to be used by both Qt and Efl.
Comment 1 YoungTaeck Song 2012-06-24 17:13:30 PDT
Created attachment 149210 [details]
patch
Comment 2 Ryuan Choi 2012-06-24 18:57:33 PDT
Comment on attachment 149210 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=149210&action=review

> Source/WebKit2/ChangeLog:8
> +        Move LayerBackingStore into common directory âSource/WebKit2/UIProcess/â to be used by both Qt and Efl.

There are wrong characters.
Comment 3 YoungTaeck Song 2012-06-24 20:46:14 PDT
Created attachment 149233 [details]
Patch
Comment 4 Noam Rosenthal 2012-06-24 21:12:24 PDT
Comment on attachment 149233 [details]
Patch

This is TextureMapper specific, why don't we move it to UIProcess/texmap?
Comment 5 YoungTaeck Song 2012-06-24 23:00:09 PDT
Created attachment 149245 [details]
patch
Comment 6 YoungTaeck Song 2012-06-25 04:41:11 PDT
(In reply to comment #4)
> (From update of attachment 149233 [details])
> This is TextureMapper specific, why don't we move it to UIProcess/texmap?

Thanks for your kind review. :-)
I moved  LayerBackingStore under Source/WebKit2/UIProcess/texmap.
Comment 7 WebKit Review Bot 2012-06-25 06:16:41 PDT
Comment on attachment 149245 [details]
patch

Clearing flags on attachment: 149245

Committed r121151: <http://trac.webkit.org/changeset/121151>
Comment 8 WebKit Review Bot 2012-06-25 06:16:49 PDT
All reviewed patches have been landed.  Closing bug.