Bug 90211

Summary: [BlackBerry] Add blitVisibleContents() as public API
Product: WebKit Reporter: Jakob Petsovits <jpetsovits>
Component: WebKit BlackBerryAssignee: Jakob Petsovits <jpetsovits>
Status: RESOLVED FIXED    
Severity: Normal CC: manyoso, mifenton, tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Jakob Petsovits
Reported 2012-06-28 14:45:07 PDT
We had some discussions and decided that it's a good idea to get rid of partial-screen blitting, as fast blitting makes up for the complexity and potential corner-case bugs we currently have in managing non-preserved double-buffering. I wanted to remove BackingStore::blitContents() from the public API wholesale, but it appears we're still using it in some code path that will hopefully go away soon. So for now, just add blitVisibleContents() (blits the whole visible area) to the public API. Patch below!
Attachments
Patch (3.29 KB, patch)
2012-06-28 14:51 PDT, Jakob Petsovits 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jakob Petsovits
Comment 1 2012-06-28 14:51:15 PDT
Created attachment 150018 [details] Patch
Jakob Petsovits
Comment 2 2012-06-29 10:33:16 PDT
Comment on attachment 150018 [details] Patch Seems I forgot the commit-queue flag. There we go!
WebKit Review Bot
Comment 3 2012-06-29 11:27:41 PDT
Comment on attachment 150018 [details] Patch Clearing flags on attachment: 150018 Committed r121576: <http://trac.webkit.org/changeset/121576>
WebKit Review Bot
Comment 4 2012-06-29 11:27:50 PDT
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.