AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Make BBH parameter to SkRecordComputeLayers and SkRecordFillBounds optional

Browse Source 
Chromium creates a picture to contain their picture pile in order to use MultiPictureDraw. They currently do not create a bounding box for that picture but I still need layer information for it. This change allows Chromium to continue without a BBH but still have layer information.

In the future, the brute force BBH might be suitable for their use case.

Please see gpu_raster_worker_pool.cc in (Add flag to beginRecording to request saveLayer information - https://codereview.chromium.org/721883002/) for where this is happening in Chromium.

Review URL: https://codereview.chromium.org/733963004
This commit is contained in:
robertphillips 2014-11-17 08:16:15 -08:00 committed by Commit bot
parent 912b3d24a0
commit 8910879098
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/core/SkRecordDraw.cpp
View File

@ -168,8 +168,9 @@ public:
}
// Finally feed all stored bounds into the BBH. They'll be returned in this order.
SkASSERT(bbh);
bbh->insert(&fBounds, fNumRecords);
if (bbh) {
bbh->insert(&fBounds, fNumRecords);
}
}
template <typename T> void operator()(const T& op) {