Bug 97314

Summary: [BlackBerry] Remove obsolete compositing surface code
Product: WebKit Reporter: Arvid Nilsson <anilsson>
Component: WebKit BlackBerryAssignee: Arvid Nilsson <anilsson>
Status: RESOLVED FIXED    
Severity: Normal CC: anilsson, gyuyoung.kim, jpetsovits, mifenton, rakuco, rwlbuis, tonikitoo, webkit.review.bot, yong.li.webkit
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Arvid Nilsson 2012-09-21 02:45:56 PDT
The removed code allowed rendering of sublayers to a separate offscreen surface.

Now that we composite root layer and all sublayers to the window surface, this code is not needed anymore. In addition, we save some memory by not allocating the unused offscreen surface.
Comment 1 Arvid Nilsson 2012-09-21 02:54:06 PDT
PR 208038
Comment 2 Arvid Nilsson 2012-09-21 02:56:16 PDT
Created attachment 165099 [details]
Patch
Comment 3 Antonio Gomes 2012-09-21 05:20:26 PDT
Comment on attachment 165099 [details]
Patch

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

> Source/WebKit/blackberry/Api/WebPage.cpp:-3669
> -    if (m_pendingOrientation != -1)
> -        SurfacePool::globalSurfacePool()->notifyScreenRotated();
> -

related?
Comment 4 Arvid Nilsson 2012-09-21 06:29:05 PDT
(In reply to comment #3)
> (From update of attachment 165099 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=165099&action=review
> 
> > Source/WebKit/blackberry/Api/WebPage.cpp:-3669
> > -    if (m_pendingOrientation != -1)
> > -        SurfacePool::globalSurfacePool()->notifyScreenRotated();
> > -
> 
> related?

Yes, the only reason the surface pool needed to know about screen rotation, was to reallocate the compositing surface at the new size. So it's not needed any more.
Comment 5 WebKit Review Bot 2012-09-21 06:38:07 PDT
Comment on attachment 165099 [details]
Patch

Clearing flags on attachment: 165099

Committed r129222: <http://trac.webkit.org/changeset/129222>
Comment 6 WebKit Review Bot 2012-09-21 06:38:11 PDT
All reviewed patches have been landed.  Closing bug.