Bug 90211 - [BlackBerry] Add blitVisibleContents() as public API
Summary: [BlackBerry] Add blitVisibleContents() as public API
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jakob Petsovits
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-28 14:45 PDT by Jakob Petsovits
Modified: 2012-06-29 11:27 PDT (History)
4 users (show)

See Also:

Attachments
Patch (3.29 KB, patch)
2012-06-28 14:51 PDT, Jakob Petsovits 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jakob Petsovits 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!
Comment 1 Jakob Petsovits 2012-06-28 14:51:15 PDT 
Created attachment 150018 [details]
Patch
Comment 2 Jakob Petsovits 2012-06-29 10:33:16 PDT 
Comment on attachment 150018 [details]
Patch

Seems I forgot the commit-queue flag. There we go!
Comment 3 WebKit Review Bot 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>
Comment 4 WebKit Review Bot 2012-06-29 11:27:50 PDT 
All reviewed patches have been landed.  Closing bug.