Chromium Code Reviews (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out

Issue 21050: POSIX: Backing store scrolling. (Closed)

11 years, 10 months ago by agl
9 years, 7 months ago


POSIX: Backing store scrolling. In the long term, we plan to get rid of all this code and "do things properly". However, for now we are implementing several shortcuts to get something which can render. Part one of this was BitmapWireData (r9065) which transported updates from the renderer over the IPC channel. This patch implements the fast-scrolling path, slowly. It might seem that this is something which Skia should be doing. However, copying from one bitmap to the same bitmap needs to be handled very differently from bitblitting from one to another to save writing to locations that you'll need to read from later on. I can't see any indication in the Skia code that it handles this case so, in order to get something working quickly, we write our own, small, bitblitter. Since this code hasn't been tested, it's almost certainly buggy. However, we don't get to test it until we get a little more of the browser up and running.

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+129 lines, -2 lines) Patch
M chrome/browser/renderer_host/ View 2 chunks +129 lines, -2 lines 2 comments Download


Total messages: 2 (0 generated)
11 years, 10 months ago (2009-02-04 19:10:29 UTC) #1
11 years, 10 months ago (2009-02-04 20:36:06 UTC) #2
I don't see any of the glaring bugs that inevitably will exist. LGTM
File chrome/browser/renderer_host/ (right):
Line 39: void BackingStore::ScrollRect(base::ProcessHandle process,
Can you make some comment that this is a temporary implementation until a "real"
one is implemented for each platform?
Line 129: // scrolling down; dest rect is above src rect.
This comment and the next 3 need capital letters at the beginning.

Powered by Google App Engine
This is Rietveld 408576698