summaryrefslogtreecommitdiffstats
path: root/cc/playback/discardable_image_map.h
Commit message (Collapse)AuthorAgeFilesLines
* DrawImage updatesericrk2016-03-091-4/+0
| | | | | | | | | | | | | | | | GPU Image Decode Controller needs to get an image's full matrix, so a matrix was added to DrawImage. DrawImage can now calculate its scale and decomposability itself, rather than relying on the call site. Additionaly, it turns out that perspective implies non-decomposability, so the perspective flag was redundant. This has been removed. BUG= CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1781493003 Cr-Commit-Position: refs/heads/master@{#380170}
* Revert checkerboarding image/no-image metrics.vollick2016-03-011-1/+0
| | | | | | | | | | | | | | This is currently providing no more value. Let's ditch it. This is a revert of https://codereview.chromium.org/1566813002 BUG=None CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1741183003 Cr-Commit-Position: refs/heads/master@{#378388}
* cc: Fix a use-after-free in discardable image map.vmpstr2016-02-011-8/+0
| | | | | | | | | | | | | | | | In display item list world, we may use a temporary paint during a save which is gone by the time we process the paint, causing use after free. This patch ensures that we take a copy of the paint, which means that when we process the paint, we won't use after free. R=enne, wkorman BUG=582606 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1651533003 Cr-Commit-Position: refs/heads/master@{#372769}
* Remove scoped_ptr<SkCanvas> from cc/tomhudson2016-02-011-3/+4
| | | | | | | | | | | | | | After crrev.com/1646593002 there was only one scoped_ptr<> to a reference-counted Skia object remaining in the compositor. After this CL, there will be none. R=danakj@chromium.org BUG=346599 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1657753002 Cr-Commit-Position: refs/heads/master@{#372751}
* cc: Add image decode control in the compositor.vmpstr2016-01-191-0/+15
| | | | | | | | | | | | | | | | | | | This patch reworks ImageDecodeController and the surrounding system to give control over image decodes to the compositor. Perf note: This is disabled on android, and should only affect desktop. That being said, this patch changes the behavior with respect to image decodes, so some perf changes are expected. BUG=516751 R=enne, reed1 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1418573002 Cr-Commit-Position: refs/heads/master@{#370188}
* Measure how often we checkerboard tiles without/with-some image contentvollick2016-01-071-0/+1
| | | | | | | | | BUG=None CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1566813002 Cr-Commit-Position: refs/heads/master@{#368140}
* Use Rect rather than RectF in cc's RTree.wkorman2015-12-041-2/+1
| | | | | | | | | BUG=529938 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1498593005 Cr-Commit-Position: refs/heads/master@{#363151}
* cc: Save only the scale for discardable images.vmpstr2015-10-211-0/+1
| | | | | | | | | | | | | | | | This patch replaces storing a matrix with storing a scale in DrawImage. This means when copying this, we're copying a lot fewer floats. This patch also adds raster scale to functions that get the images at raster tile. The raster scale ensures that the returned image has correctly adjusted scale. R=enne CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1419743002 Cr-Commit-Position: refs/heads/master@{#355198}
* cc: Rename PositionImage to DrawImage and remove rect.vmpstr2015-10-141-12/+7
| | | | | | | | | | | | | | | | This patch removes the rect from position image since it might be misleading once we scale the matrix to account for contents scale. It also removes position image to draw image to better reflect that this struct stores an images with a matrix that is going to be used to draw it. R=danakj CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1403643004 Cr-Commit-Position: refs/heads/master@{#354156}
* cc: Simplify the discardable image map using an rtree.vmpstr2015-09-291-49/+23
| | | | | | | | | | | | | | | | | | | | | | | | This patch changes the way the discardable image map accesses its data. Instead of a custom structure that can have false positives, this patch makes it use an rtree. It also yields itself to be able to be used with a display item list's non-cached version. Note that since the picture pile stuff is close to being removed, I didn't reimplement the picture pile functions. This depends on https://codereview.chromium.org/1344633002/ https://codereview.chromium.org/1353193002/ R=enne, weiliangc BUG=532662, 510294 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1354923005 Cr-Commit-Position: refs/heads/master@{#351380}
* cc/skia: Move gather images code to cc::DiscardableImageMap.vmpstr2015-09-251-9/+10
| | | | | | | | | | | | | | | | | | | I'd like to change the way we gather images. Right now, we use a static function that takes a picture as input and produces a list of images as output. Instead, I'd like to integrate this closer to DiscardableImageMap so that we can easily extend the gather to work with individual display items. This patch is the first step, which is to move the code to discardable image map from skia/ext. R=enne, reed@google.com BUG=532662 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1353193002 Cr-Commit-Position: refs/heads/master@{#350918}
* Rename GatherDiscardableImages to better reflect functionalityvmpstr2015-09-251-1/+2
| | | | | | | | | | | | | | | | | | | | | | | This patch changes some uses of GatherDiscardableImages in the following way: Settings leading up to the recording part are called "Image Decode Tasks Enabled" At recording time, this is called "Generate Discardable Images Metadata" At raster time, this is called "Get Discardable Images in Rect" Hopefully this makes it a bit easier to navigate this part of the code. R=weiliangc, danakj, enne CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1371493002 Cr-Commit-Position: refs/heads/master@{#350860}
* cc: Remove Picture.vmpstr2015-09-241-5/+2
| | | | | | | | | | | | | | | This patch removes the cc::Picture class, since the display list approach is the only one supported now. Depends on https://codereview.chromium.org/1362663002/ BUG=533151 R=danakj, enne CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1365673004 Cr-Commit-Position: refs/heads/master@{#350661}
* Convert pixel_ref_map -> discardable_image_mapfmalita2015-09-091-0/+94
As blink now only creates discardable images (not discardable bitmaps/pixelrefs, we need to track the images. (based on reed's http://crrev.com/1318323003) BUG=527246 R=reed@google.com,vmpstr@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1314353005 Cr-Commit-Position: refs/heads/master@{#347985}