| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
| |
BUG=None
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1566813002
Cr-Commit-Position: refs/heads/master@{#368140}
|
|
|
|
|
|
|
|
|
| |
BUG=529938
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1498593005
Cr-Commit-Position: refs/heads/master@{#363151}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
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}
|