Age | Commit message (Collapse) | Author |
|
This commit was generated by merge_to_master.py.
Change-Id: I07848840df4da46e0aa5b11fb8533dafb972af91
|
|
https://chromium.googlesource.com/external/skia/include.git at c7557fbeb626b286d27a1b46b93da5a030df12b3
This commit was generated by merge_from_chromium.py.
Change-Id: I3081cc5a45d49aefc4804c5c0a56e943ed5f9909
|
|
BUG=skia:
R=scroggo@google.com, dominikg@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/263293005
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14592 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
https://chromium.googlesource.com/external/skia/include.git at abc68a8eeb478a8123d4d283bab5088d0bb949f9
This commit was generated by merge_from_chromium.py.
Change-Id: Ia575c4a48fa4dfb8fca42fba218c13c3fa56b1c3
|
|
https://chromium.googlesource.com/external/skia/include.git at 08a3eed60c27d0ab0e3059769a4b3470435bd348
This commit was generated by merge_from_chromium.py.
Change-Id: Ic20e392150c3f7a9604815f06ad08372652c7520
|
|
results can be seen in new gm: vertices_80
BUG=skia:
R=scroggo@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/270023002
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14581 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
https://chromium.googlesource.com/external/skia/include.git at d2b8224663697e9ce19c97862eaf783b1e9e1151
This commit was generated by merge_from_chromium.py.
Change-Id: I55e9f482313eacde51846a44218ed59fa7835706
|
|
Adds methods to GrContext for client code to store custom resources in
the cache.
BUG=skia:
R=bsalomon@google.com
Author: cdalton@nvidia.com
Review URL: https://codereview.chromium.org/261593009
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14577 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
when their size has changed. GrResourceCacheEntry now holds a
reference to the cache, and a cached value of the resource's most
recent size.
Also utilizes this new functionality for mipmaps, and adds a test for
changing resource sizes.
R=bsalomon@google.com, robertphillips@google.com
Author: cdalton@nvidia.com
Review URL: https://codereview.chromium.org/257093002
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14576 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
patch from issue 267923005
BUG=skia:
R=scroggo@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/261773005
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14573 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
Inspired by the excellent repro case for https://crbug.com/364224
patch from issue 265933010
BUG=skia:
R=bungeman@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/267003002
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14566 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
https://chromium.googlesource.com/external/skia/include.git at 2daacdf0cc5790a8ed017bfdfcf802105682a846
This commit was generated by merge_from_chromium.py.
Change-Id: I09348bfa2464ec11d24696af0f4e8800d52dbb5a
|
|
BUG=skia:2402
Committed: http://code.google.com/p/skia/source/detail?r=14533
R=robertphillips@google.com, djsollen@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/243413002
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14564 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
https://chromium.googlesource.com/external/skia/include.git at 408cb61cd625026f204980897f638ed6ba3cf3e8
This commit was generated by merge_from_chromium.py.
Change-Id: I257aa35320638c9cd19f88b14623b08774d5a2b5
|
|
on os x 10.9.2 unistd.h defines sysconf as
long sysconf(int);
R=mtklein@google.com, reed@google.com
Author: caryclark@google.com
Review URL: https://codereview.chromium.org/261673004
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14559 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
https://chromium.googlesource.com/external/skia/include.git at a92579d00463a0002a20a49af68da8260d73600f
This commit was generated by merge_from_chromium.py.
Change-Id: I6cff178d506857fa1ceb71f51f63e0771ebb9384
|
|
https://chromium.googlesource.com/external/skia/include.git at fa2a9d0672d7d30bb37b8fc2e62b5c93e2fa314c
This commit was generated by merge_from_chromium.py.
Change-Id: I9001c1d165f951b9409a94a9eaa6f43f38ebef6f
|
|
Before this change, an object needed to inherit from GrResource (and
thus be a GPU object) in order to live in the GrResourceCache. That
was a problem for caching items that weren't GPU objects themselves,
but owned GPU objects.
This change splits GrResource into two classes:
1. GrCacheable: The base class for objects that can live in the
GrResourceCache.
2. GrGpuObject, which inherits from GrCacheable: The base class for
objects that get tracked by GrGpu.
This change is purely a refactor; there is no change in functionality.
Change-Id: I3e8daeb1f123041f414aa306c1366e959ae9e39e
BUG=skia:
R=bsalomon@google.com
Author: cdalton@nvidia.com
Review URL: https://codereview.chromium.org/251013002
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14553 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
BUG=skia:
R=bungeman@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/263883006
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14552 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
This makes all --skr tests pass for me. Enabling it by default in DM.
BUG=skia:2378
Committed: http://code.google.com/p/skia/source/detail?r=14549
R=reed@google.com, mtklein@google.com, fmalita@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/260863007
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14551 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
(https://codereview.chromium.org/260863007/)
Reason for revert:
changed GM unexpectedly. will sort out and try again
Original issue's description:
> Remove setLocalMatrix calls from picture shader GM.
>
> This makes all --skr tests pass for me. Enabling it by default in DM.
>
> BUG=skia:2378
>
> Committed: http://code.google.com/p/skia/source/detail?r=14549
R=reed@google.com, fmalita@google.com, mtklein@chromium.org
TBR=fmalita@google.com, mtklein@chromium.org, reed@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:2378
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/265013003
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14550 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
This makes all --skr tests pass for me. Enabling it by default in DM.
BUG=skia:2378
R=reed@google.com, mtklein@google.com, fmalita@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/260863007
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14549 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
TSAN shows us racing on the function pointers. Might as well fix it.
WARNING: ThreadSanitizer: data race (pid=19995)
Read of size 8 at 0x7f703affb048 by thread T12 (mutexes: write M2957):
#0 SkBitmap::internalErase(SkIRect const&, unsigned int, unsigned int, unsigned int, unsigned int) const /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/src/core/SkBitmap.cpp:886 (tests+0x0000003511ca)
#1 SkBitmap::eraseARGB(unsigned int, unsigned int, unsigned int, unsigned int) const /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/src/core/SkBitmap.cpp:919 (tests+0x0000003534bf)
#2 (anonymous namespace)::DecodingImageGenerator::getPixels(SkImageInfo const&, void*, unsigned long) /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/src/images/SkDecodingImageGenerator.cpp:195 (tests+0x00000051bee1)
#3 SkDiscardablePixelRef::onNewLockPixels(SkPixelRef::LockRec*) /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/src/lazy/SkDiscardablePixelRef.cpp:63 (tests+0x00000039ad9c)
#4 SkPixelRef::lockPixels(SkPixelRef::LockRec*) /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/src/core/SkPixelRef.cpp:179 (tests+0x0000003fec23)
#5 SkBitmap::lockPixels() const /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/src/core/SkBitmap.cpp:414 (tests+0x00000034e41e)
#6 SkAutoLockPixels /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/include/core/SkBitmap.h:819 (tests+0x0000002752f3)
#7 ImageDecoderOptions(skiatest::Reporter*) /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/tests/ImageDecodingTest.cpp:565 (tests+0x000000275d03)
#8 skiatest::Test::run() /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/tests/Test.cpp:107 (tests+0x0000002263e7)
#9 SkTestRunnable::run() /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/tests/skia_test.cpp:108 (tests+0x0000001d8607)
#10 SkThreadPoolPrivate::ThreadLocal<void>::run(SkTRunnable<void>*) /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/include/utils/SkThreadPool.h:108 (tests+0x0000001d817e)
#11 thread_start(void*) /var/scratch/Release/../../../usr/local/google/home/mtklein/skia/src/utils/SkThreadUtils_pthread.cpp:66 (tests+0x000000604347)
Previous write of size 8 at 0x7f703affb048 by thread T26:
[failed to restore the stack]
BUG=skia:1792
R=bungeman@google.com, mtklein@google.com, reed@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/250503003
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14548 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
glMapBufferSubData. (https://codereview.chromium.org/243413002/)
Reason for revert:
This is firing an assert on many of the Android debug bots
Original issue's description:
> Add support for glMapBufferRange. Use glMapBufferRange and glMapBufferSubData.
>
> BUG=skia:2402
>
> Committed: http://code.google.com/p/skia/source/detail?r=14533
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14541 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
https://chromium.googlesource.com/external/skia/include.git at 131e138f57d72cde1c1549ba33301b2d5ea5c3e9
This commit was generated by merge_from_chromium.py.
Change-Id: If6ea7da1269c010f39fb719fc1fb2a3f5511269e
|
|
BUG=skia:2402
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/243413002
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14533 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
BUG=skia:2173
R=bsalomon@google.com, reed@google.com, bungeman@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/261783004
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14525 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
BUG=skia:
R=scroggo@google.com, dominikg@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/264843006
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14514 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
https://chromium.googlesource.com/external/skia/include.git at 3fb4cc8c3b7c3f77245eefef9691f207615938e1
This commit was generated by merge_from_chromium.py.
Change-Id: I4fed3a641666e9c79244c61c9f597111bf41cb88
|
|
Like yesterday's change to run CPU-parent child tasks serially in thread, this
reduces peak memory usage by improving the temporaly locality of the bitmaps we
create.
E.g. Let's say we start with tasks A B C and D
Queue: [ A B C D ]
Running A creates A' and A", which depend on a bitmap created by A.
Queue: [ B C D A' A" * ]
That bitmap now needs sit around in RAM while B C and D run pointlessly and can
only be destroyed at *. If instead we do this and push dependent child tasks
to the front of the queue, the queue and bitmap lifetime looks like this:
Queue: [ A' A" * B C D ]
This is much, much worse in practice because the queue is often several thousand
tasks long. 100s of megs of bitmaps can pile up for 10s of seconds pointlessly.
To make this work we add addNext() to SkThreadPool and its cousin DMTaskRunner.
I also took the opportunity to swap head and tail in the threadpool
implementation so it matches the comments and intuition better: we always pop
the head, add() puts it at the tail, addNext() at the head.
Before
Debug: 49s, 1403352k peak
Release: 16s, 2064008k peak
After
Debug: 49s, 1234788k peak
Release: 15s, 1903424k peak
BUG=skia:2478
R=bsalomon@google.com, borenet@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/263803003
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14506 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
This commit was generated by merge_to_master.py.
Change-Id: I352045c456812e4311361c41049f1f6bcbeec5a3
|
|
BUG=skia:
R=scroggo@google.com
Review URL: https://codereview.chromium.org/264923004
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14504 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
NOTRY=True
R=robertphillips@google.com, fmalita@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/264923002
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14501 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
These are reasonable return values, since both of these methods can return a known value (0)
which means that no context can be created. This also makes it easier for chrome's subclasses
which already do not want to create a context, but having them actually overridden makes
changing the virtual signatures much harder.
BUG=skia:
R=scroggo@google.com, dominikg@google.com, reed@chromium.org
Author: reed@google.com
Review URL: https://codereview.chromium.org/262703002
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14491 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
Committed: http://code.google.com/p/skia/source/detail?r=14473
R=caryclark@google.com, reed@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/263553012
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14481 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
(https://codereview.chromium.org/263553012/)
Reason for revert:
Broke windows bot
Original issue's description:
> Rename SK_DEBUGBREAK to SK_ALWAYSBREAK
>
> Committed: http://code.google.com/p/skia/source/detail?r=14473
R=caryclark@google.com, reed@google.com
TBR=caryclark@google.com, reed@google.com
NOTREECHECKS=true
NOTRY=true
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/260763003
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14476 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
R=caryclark@google.com, reed@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/263553012
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14473 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
R=robertphillips@google.com, reed@google.com, mtklein@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/257393004
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14460 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
https://chromium.googlesource.com/external/skia/include.git at 394ec1d498fc2f93a45c150aa8ad24bec26c3760
This commit was generated by merge_from_chromium.py.
Change-Id: I856ef4fed1234dcbe0f3fd6bc2a6a4c48afb7411
|
|
https://chromium.googlesource.com/external/skia/include.git at 0e69b9ce679b69d9e870c993ecddc2c0cd06e496
This commit was generated by merge_from_chromium.py.
Change-Id: I8ebaa44197b07ad10b1b3d2feecd52050f0b3e30
|
|
An integer overflow was causing an issue when reading a string with a very large (or negative) size.
BUG=367764
R=senorblanco@google.com, senorblanco@chromium.org, reed@google.com, borenet@google.com
Author: sugoi@chromium.org
Review URL: https://codereview.chromium.org/255693003
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14434 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
Add a new GM that exercises tiled drawing all pixel-moving filters
(and some non-pixel-moving ones) and compares it against non-tiled
drawing of the same filters. Fixing this test revealed that tile and
matrix convolution filters had no onFilterBounds() traversals
(test-driven development FTW). Tile requires (conservatively) the
bounds to include the whole source rect, since it may end up in the
result. Matrix convolution requires the bounds to be offset by the
kernel size and target.
R=reed@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/258243005
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14432 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
BUG=skia:
R=djsollen@google.com, scroggo@google.com
Review URL: https://codereview.chromium.org/253633003
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14431 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
Failed builders https://codereview.chromium.org/225023031/ at PS 24 look like they're failing because we're not exposing these symbols when Skia's a dynamic library.
BUG=skia:2378
R=reed@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/253883003
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14425 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
https://chromium.googlesource.com/external/skia/include.git at 01b0da4230d5ffe8a656a413058037194606d45f
This commit was generated by merge_from_chromium.py.
Change-Id: I8976a9f0a3ba986fa98e6791d24344a4b21b4811
|
|
BUG=skia:
R=scroggo@google.com, djsollen@google.com
Author: reed@google.com
Review URL: https://codereview.chromium.org/253833002
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14411 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
As a first step towards removing SkShader::setLocalMatrix, which will make
SkShader thread-safe, remove calls to setLocalMatrix that happen immediately
after the shader is being created. Instead, pass the matrix into the constructor
or factory method.
BUG=skia:1976
R=scroggo@google.com, reed@google.com, skyostil@google.com, mtklein@google.com
Author: dominikg@chromium.org
Review URL: https://codereview.chromium.org/245963010
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14401 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
BUG=skia:
Committed: http://code.google.com/p/skia/source/detail?r=14353
NOTRY=true
NOTREECHECKS=true
Committed: http://code.google.com/p/skia/source/detail?r=14354
R=bsalomon@google.com, bungeman@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/247813005
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14390 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
(https://codereview.chromium.org/247813005/)
Reason for revert:
breaks blink in roll
Original issue's description:
> teach TSAN about SkSpinlock, SkRefCnt, and SkOnce
>
> BUG=skia:
>
> Committed: http://code.google.com/p/skia/source/detail?r=14353
>
> NOTRY=true
> NOTREECHECKS=true
>
> Committed: http://code.google.com/p/skia/source/detail?r=14354
R=bsalomon@google.com, bungeman@google.com, mtklein@chromium.org
TBR=bsalomon@google.com, bungeman@google.com, mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/258783005
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14389 2bbb7eff-a529-9590-31e7-b0007b416f81
|
|
Use EXT_direct_state_access for path matrix manipulation when using
NV_path_rendering extension. This makes Chromium command buffer
integration easier, since the current matrix mode does not need to be
exposed as state and fewer function calls and enums are needed.
BUG=chromium:344330
R=bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://codereview.chromium.org/245963009
git-svn-id: http://skia.googlecode.com/svn/trunk/include@14374 2bbb7eff-a529-9590-31e7-b0007b416f81
|