Bug 71681 - [Qt][WK2] Remove TiledDrawingArea and dependencies.
Summary: [Qt][WK2] Remove TiledDrawingArea and dependencies.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: zalan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-07 06:47 PST by zalan
Modified: 2012-02-06 05:12 PST (History)
8 users (show)

See Also:


Attachments
Patch (27.08 KB, patch)
2011-11-07 06:52 PST, zalan
no flags Details | Formatted Diff | Diff
Patch (48.58 KB, patch)
2012-02-03 09:34 PST, Jocelyn Turcotte
kenneth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description zalan 2011-11-07 06:47:24 PST
[Qt][WK2] Remove TiledDrawingArea and dependencies.
Comment 1 zalan 2011-11-07 06:52:16 PST
Created attachment 113860 [details]
Patch
Comment 2 zalan 2011-11-07 07:17:42 PST
TiledDrawingArea and its dependencies are obsolete. In case that any port needs same/similar functionality, the current class structure needs to be reconsidered. Currently there are lots of duplicated code in TiledDrawingAreaProxy and DrawingAreaImpl. Either Tiled needs to be derived from Impl, or preferably Impl code should be moved up to the base class.
Comment 3 KwangHyuk 2011-11-07 17:16:36 PST
Hi, zbujtas,

The internal version of WK2-efl's tiled backing store implementation is now based on TiledDrawingArea.

So, I would like to know the next plan after landing of this patch.
Should we have to move from TiledDrawingArea to DrawingArea or LayerTreeHost ?
Comment 4 zalan 2011-11-08 07:14:17 PST
(In reply to comment #3)
> Hi, zbujtas,
> 
> The internal version of WK2-efl's tiled backing store implementation is now based on TiledDrawingArea.
> 
> So, I would like to know the next plan after landing of this patch.
> Should we have to move from TiledDrawingArea to DrawingArea or LayerTreeHost ?
It's really up to you whether you'd like to keep the original tile approach with a little bit of different class structure (see comment#2). However, that most likely will not work properly in all the cases when both AC and non-AC content are present. Or, as you mentioned, follow the LayerTreeHost(Qt) (TiledBackingStoreRemoteTile) implementation, which properly synchronizes the non-AC with the AC content using tiles. https://bugs.webkit.org/show_bug.cgi?id=69151
Comment 5 zalan 2011-11-09 04:39:47 PST
(In reply to comment #3)
> Hi, zbujtas,
> 
> The internal version of WK2-efl's tiled backing store implementation is now based on TiledDrawingArea.
> 
> So, I would like to know the next plan after landing of this patch.
> Should we have to move from TiledDrawingArea to DrawingArea or LayerTreeHost ?
Could you let me know, if you are fine with removing these classes? (would be great to know before submitting it for review)
Comment 6 KwangHyuk 2011-11-09 18:09:38 PST
(In reply to comment #5)
> (In reply to comment #3)
> > Hi, zbujtas,
> > 
> > The internal version of WK2-efl's tiled backing store implementation is now based on TiledDrawingArea.
> > 
> > So, I would like to know the next plan after landing of this patch.
> > Should we have to move from TiledDrawingArea to DrawingArea or LayerTreeHost ?
> Could you let me know, if you are fine with removing these classes? (would be great to know before submitting it for review)

Hi, zbujtas,

I am gathering idea from each related person to know whether anybody has any major issue caused by this patch. :)
I will give you any feedback for this soon.

I sincerely thank you for your notification of this patch in advance.
Comment 7 KwangHyuk 2011-11-10 22:24:14 PST
LGTM.

We are going to move into LayerTreeHost sooner or later.
However, I prefer it if you remove these files some days later. :)
Comment 8 zalan 2011-11-11 06:05:45 PST
(In reply to comment #7)
> LGTM.
> 
> We are going to move into LayerTreeHost sooner or later.
> However, I prefer it if you remove these files some days later. :)

sure, just let me know when. i prefer removing it sooner than latter to avoid confusion about its existence.
Comment 9 KwangHyuk 2011-11-14 01:57:44 PST
(In reply to comment #8)
> (In reply to comment #7)
> > LGTM.
> > 
> > We are going to move into LayerTreeHost sooner or later.
> > However, I prefer it if you remove these files some days later. :)
> 
> sure, just let me know when. i prefer removing it sooner than latter to avoid confusion about its existence.

Thanks for your consideration. :)
Until the end of this month. if it won't hurt your work.
Comment 10 KwangHyuk 2011-11-14 02:01:45 PST
(In reply to comment #9)
> (In reply to comment #8)
> > (In reply to comment #7)
> > > LGTM.
> > > 
> > > We are going to move into LayerTreeHost sooner or later.
> > > However, I prefer it if you remove these files some days later. :)
> > 
> > sure, just let me know when. i prefer removing it sooner than latter to avoid confusion about its existence.
> 
> Thanks for your consideration. :)
> Until the end of this month. if it won't hurt your work.

I want to see those files until the end of this month if it won't hurt your work.
Comment 11 zalan 2011-11-14 02:16:14 PST
(In reply to comment #10)
> (In reply to comment #9)
> > (In reply to comment #8)
> > > (In reply to comment #7)
> > > > LGTM.
> > > > 
> > > > We are going to move into LayerTreeHost sooner or later.
> > > > However, I prefer it if you remove these files some days later. :)
> > > 
> > > sure, just let me know when. i prefer removing it sooner than latter to avoid confusion about its existence.
> > 
> > Thanks for your consideration. :)
> > Until the end of this month. if it won't hurt your work.
> 
> I want to see those files until the end of this month if it won't hurt your work.

alright. sounds ok. thanks.
Comment 12 KwangHyuk 2011-11-14 02:26:02 PST
(In reply to comment #11)
> (In reply to comment #10)
> > (In reply to comment #9)
> > > (In reply to comment #8)
> > > > (In reply to comment #7)
> > > > > LGTM.
> > > > > 
> > > > > We are going to move into LayerTreeHost sooner or later.
> > > > > However, I prefer it if you remove these files some days later. :)
> > > > 
> > > > sure, just let me know when. i prefer removing it sooner than latter to avoid confusion about its existence.
> > > 
> > > Thanks for your consideration. :)
> > > Until the end of this month. if it won't hurt your work.
> > 
> > I want to see those files until the end of this month if it won't hurt your work.
> 
> alright. sounds ok. thanks.

Thanks too. :)
Comment 13 Jocelyn Turcotte 2012-02-03 09:34:00 PST
Created attachment 125352 [details]
Patch

Also remove QtSGTileNode and QtSGUpdateQueue.
Comment 14 Jocelyn Turcotte 2012-02-06 05:12:23 PST
Committed r106800: <http://trac.webkit.org/changeset/106800>