Bug 89838 - [WK2][Qt][EFL] Move LayerBackingStore into common directory
Summary: [WK2][Qt][EFL] Move LayerBackingStore into common directory
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 89840
  Show dependency treegraph
 
Reported: 2012-06-24 17:12 PDT by YoungTaeck Song
Modified: 2012-06-25 06:16 PDT (History)
8 users (show)

See Also:


Attachments
patch (18.92 KB, patch)
2012-06-24 17:13 PDT, YoungTaeck Song
no flags Details | Formatted Diff | Diff
Patch (19.36 KB, patch)
2012-06-24 20:46 PDT, YoungTaeck Song
noam: review-
noam: commit-queue-
Details | Formatted Diff | Diff
patch (19.43 KB, patch)
2012-06-24 23:00 PDT, YoungTaeck Song
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.