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

See Also:

Patch (3.29 KB, patch)
2012-06-28 14:51 PDT, Jakob Petsovits
no flags Details | Formatted Diff | Diff

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]
Comment 2 Jakob Petsovits 2012-06-29 10:33:16 PDT
Comment on attachment 150018 [details]

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]

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.