mtklein
efb7e42e37
Add harfbuzz to DEPS
...
https://skia.googlesource.com/third_party/harfbuzz/
BUG=skia:3067
Review URL: https://codereview.chromium.org/679733002
2014-10-24 10:20:22 -07:00
jvanverth
00c83ddfcb
Ignore simpleaaclip_path on Win7-gpu for now.
...
BUG=skia:3066
TBR=mtklein@google.com
Review URL: https://codereview.chromium.org/646523005
2014-10-24 09:40:43 -07:00
bsalomon
24234fe777
Build gpu stats tracking in dev builds.
...
Fix cache stats, add more info.
BUG=skia:2889
Review URL: https://codereview.chromium.org/655263005
2014-10-24 09:34:41 -07:00
Hal Canary
42d1db2ccc
add color wheel to validate decoding colors
...
BUG=skia:3061
R=mtklein@google.com
Review URL: https://codereview.chromium.org/681433002
2014-10-24 12:18:46 -04:00
joshualitt
c6f3e2c17b
Revert of Added varying struct (patchset #9 id:160001 of https://codereview.chromium.org/671023002/ )
...
Reason for revert:
may have caused gm change on arm
Original issue's description:
> Added varying struct
>
> TBR=
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/852ae80b9c3c6fd53f993ac35133d80863993cbe
TBR=bsalomon@google.com ,joshualitt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/675193002
2014-10-24 08:24:08 -07:00
tfarina
20b7960798
Add isScale() helper function to SkMatrix44.
...
This will be used later in Chromium to cleanup gfx::Transform::IsScale2d().
BUG=408710,skia:997
TEST=None
R=bsalomon@google.com ,danakj@chromium.org
Review URL: https://codereview.chromium.org/676583002
2014-10-24 08:07:05 -07:00
mtklein
da0de7ed90
rebaseline for new shaderproc
...
BUG=skia:
Review URL: https://codereview.chromium.org/674123003
2014-10-24 08:05:59 -07:00
joshualitt
852ae80b9c
Added varying struct
...
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/671023002
2014-10-24 07:56:04 -07:00
bsalomon
8dae0f364b
Rebaseline lots of images, associate some with bug 3063.
...
TBR=
NOTRY=true
Review URL: https://codereview.chromium.org/661323003
2014-10-24 07:26:01 -07:00
jvanverth
b3eb687f8a
Set temporary paths volatile so we don't cache them.
...
Any path that is generated frame-to-frame should not be rendered by using the
DistanceFieldPathRenderer, because generating the initial distance field,
uploading it and rendering it takes longer than the SoftwarePathRenderer.
BUG=skia:2935
Review URL: https://codereview.chromium.org/677463002
2014-10-24 07:12:51 -07:00
scroggo
3ceef9a66a
Add test for new FrontBufferedStream behavior.
...
Test for https://skia.googlesource.com/skia/+/dd5a1e094c19fa10202c37c50a1f799e5af5dac0
Verify that FrontBufferedStream does not attempt to read beyond the
end of its underlying stream.
Make SkStreamToCGImageSource handle an empty stream better.
Review URL: https://codereview.chromium.org/641813009
2014-10-24 06:55:07 -07:00
scroggo
b752f9f838
Don't read random data in ICO check.
...
Check to ensure SkStream::read succeeds before checking the result.
Review URL: https://codereview.chromium.org/656673005
2014-10-24 06:49:57 -07:00
jcgregorio
cee4ddf1c4
Re-add in v8 to DEPS.
...
BUG=skia:
Review URL: https://codereview.chromium.org/673713004
2014-10-24 05:57:37 -07:00
skia-deps-roller
9717e4a773
Update SKP version
...
Automatic commit by the RecreateSKPs bot.
TBR=
Review URL: https://codereview.chromium.org/679693002
2014-10-24 05:47:24 -07:00
joshualitt
976386b75d
Possible optimization to SkString
...
BUG=skia:
Review URL: https://codereview.chromium.org/679433003
2014-10-23 18:23:32 -07:00
tfarina
bf54e49e30
Cleanup: Turn GrReducedClip into a class with a static function.
...
Clean up namespace usage.
Similar to what was done in
https://skia.googlesource.com/skia/+/a5414c4a8efc3119ee20fcee96c0bf68a04909c7
BUG=None
TEST=None
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/653393003
2014-10-23 17:47:18 -07:00
scroggo
11ed6b8140
Add flag to skimage for decoding into unpremul.
...
Review URL: https://codereview.chromium.org/666073006
2014-10-23 15:42:11 -07:00
mtklein
afc9b6ec95
squelch filterindiabox too
...
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/677463003
2014-10-23 15:24:11 -07:00
senorblanco
cbf6b6ecd5
Fix imagemagnifier GM quality and speed.
...
The imagemagnifier GM was applying the filter both to a saveLayer()
restore() around the whole scene, as well as to each individual
text draw.
Applying the filter only in the saveLayer() makes it run ~20x faster
on my Linux box, and the quality is improved as well (since the
primitives are not double-filtered).
BUG=skia:
Review URL: https://codereview.chromium.org/637283009
2014-10-23 15:00:12 -07:00
mtklein
14e4d392d4
Revert "Revert of create shaderproc for nofilter-opaque-dx (patchset #7 id:120001 of https://codereview.chromium.org/664783004/ )"
...
This reverts commit 430b795cc8
and adds suppressions.
BUG=skia:
Review URL: https://codereview.chromium.org/673023002
2014-10-23 14:35:01 -07:00
bsalomon
efd7d45fc0
Don't use MIP filter mode on compressed textures
...
Add compressed texture image downsample GMs
BUG=chromium:426331
Review URL: https://codereview.chromium.org/672863003
2014-10-23 14:17:46 -07:00
senorblanco
c4d619e145
Remove test expectations from ignored-tests.txt.
...
TBR=mtklein@google.com
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/659393006
2014-10-23 14:01:38 -07:00
sugoi
234f036b3e
Adding an option to render only the shadow in SkDropShadowImageFilter
...
This is basically how blink uses the filter. Currently, I can't use it for "ShadowOnly" mode with the filter at all, but instead of copying the code and risking to have the codepaths diverge, I'm simply going to add the option here.
BUG=skia:
Review URL: https://codereview.chromium.org/646213004
2014-10-23 13:59:52 -07:00
senorblanco
d0777fd2cf
Rebaselines for tests affected by image filter PDF change.
...
(aka https://codereview.chromium.org/644323006/ )
TBR=mtklein@google.com
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/671303003
2014-10-23 13:50:54 -07:00
pavel
47eedcc848
Add utils to better quantize grayscale values to three bit indices while
...
compressing coverage masks.
Signed-off-by: Pavel Krajcevski <pavel@cs.unc.edu>
BUG=skia:
Review URL: https://codereview.chromium.org/669243003
2014-10-23 13:18:50 -07:00
caryclark
08fa28cd31
add float equivalent in dumpHex output as comment
...
R=reed@google.com ,bsalomon@google.com
Review URL: https://codereview.chromium.org/676803002
2014-10-23 13:08:57 -07:00
mtklein
430b795cc8
Revert of create shaderproc for nofilter-opaque-dx (patchset #7 id:120001 of https://codereview.chromium.org/664783004/ )
...
Reason for revert:
Many GMs fixed. Needs rebaseline, perhaps layout test rebaselines.
Original issue's description:
> create shaderproc for nofilter-opaque-dx
>
>
> speedup nofilter
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/a40a276bcee2246439dcf816273c1307f5c3c69f
TBR=djsollen@google.com ,reed@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/656913005
2014-10-23 12:44:40 -07:00
reed
4dca7a8296
use SkMaskCache inside SkBlurMaskFilter
...
depends on https://codereview.chromium.org/670063004/
BUG=skia:
Review URL: https://codereview.chromium.org/669993003
2014-10-23 12:42:46 -07:00
bungeman
4aa75b7980
Correct typo in SkRemotableFontMgr_win_dw.cpp.
...
New code was introduced behind a flag, but in that code there is a comma
before a parameter name instead of after.
Review URL: https://codereview.chromium.org/646323003
2014-10-23 12:25:29 -07:00
reed
a40a276bce
create shaderproc for nofilter-opaque-dx
...
speedup nofilter
BUG=skia:
Review URL: https://codereview.chromium.org/664783004
2014-10-23 12:22:40 -07:00
mtklein
404eb87993
Revert of Add test for new FrontBufferedStream behavior. (patchset #1 id:1 of https://codereview.chromium.org/641813009/ )
...
Reason for revert:
Test is SkASSERTing on Macs, e.g:
http://chromegw.corp.google.com/i/client.skia/builders/Test-Mac10.6-MacMini4.1-GeForce320M-x86_64-Debug/builds/257/steps/dm/logs/stdio
210 tasks left 1737M peak 1ms test FrontBufferedStream../../src/ports/SkImageDecoder_CG.cpp:43: failed assertion "data"
Signal 11:
_sigtramp (+0x1a)
SkStreamToCGImageSource(SkStream*) (+0x62)
SkImageDecoder_CG::onDecode(SkStream*, SkBitmap*, SkImageDecoder::Mode) (+0x2e)
SkImageDecoder::decode(SkStream*, SkBitmap*, SkColorType, SkImageDecoder::Mode) (+0x81)
SkImageDecoder::DecodeStream(SkStreamRewindable*, SkBitmap*, SkColorType, SkImageDecoder::Mode, SkImageDecoder::Format*) (+0xff)
SkImageDecoder::DecodeStream(SkStreamRewindable*, SkBitmap*) (+0x31)
test_ShortFrontBufferedStream(skiatest::Reporter*) (+0x97)
skiatest::ShortFrontBufferedStreamClass::onRun(skiatest::Reporter*) (+0x19)
skiatest::Test::run() (+0x7c)
DM::CpuTestTask::draw() (+0x5a)
DM::CpuTask::run() (+0x9e)
non-virtual thunk to DM::CpuTask::run() (+0x1c)
(anonymous namespace)::ThreadPool::Loop(void*) (+0xf4)
thread_start(void*) (+0x54)
_pthread_start (+0x14b)
Original issue's description:
> Add test for new FrontBufferedStream behavior.
>
> Test for https://skia.googlesource.com/skia/+/dd5a1e094c19fa10202c37c50a1f799e5af5dac0
>
> Verify that FrontBufferedStream does not attempt to read beyond the
> end of its underlying stream.
>
> Committed: https://skia.googlesource.com/skia/+/da59f05c6738dbb9a92cad21c608cdfae53a76b2
TBR=reed@google.com ,scroggo@google.com
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/649553003
2014-10-23 12:08:03 -07:00
jvanverth
73f1053450
Allocate only the vertices we need for text.
...
This restructures the vertex allocation for text rendering to compute
the max number of vertices we would need for a line of text, and then
only allocate that much. If this number exceeds the quad index limit,
then it will allocate for the max number of quads, and reallocate for
the rest later.
Review URL: https://codereview.chromium.org/663423003
2014-10-23 11:57:12 -07:00
joshualitt
70649c174b
updating k1 expectations
...
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/642543003
2014-10-23 11:34:08 -07:00
piotaixr
ec6e670669
Removed simple tests steps from CanvasTest.
...
CanvasTest only checks if the state of the different canvas
implementations are consistent with the one of the default
implementation.
Because most of the draw calls are stateless and already tested in GMs
the test steps that exercices them are not useful.
BUG=skia:3054
Review URL: https://codereview.chromium.org/672273002
2014-10-23 10:59:07 -07:00
djsollen
e3aee8fb7b
Remove android specific genID from SkPaint.
...
Review URL: https://codereview.chromium.org/677453002
2014-10-23 10:23:07 -07:00
scroggo
da59f05c67
Add test for new FrontBufferedStream behavior.
...
Test for https://skia.googlesource.com/skia/+/dd5a1e094c19fa10202c37c50a1f799e5af5dac0
Verify that FrontBufferedStream does not attempt to read beyond the
end of its underlying stream.
Review URL: https://codereview.chromium.org/641813009
2014-10-23 10:21:28 -07:00
mtklein
df78d0a81c
This return_false is now only used on Android. Squelch a warning.
...
BUG=skia:
Review URL: https://codereview.chromium.org/659393004
2014-10-23 10:13:21 -07:00
borenet
4f93f3c441
Rename Nexus9 expectations
...
BUG=skia:3047
TBR=djsollen
Review URL: https://codereview.chromium.org/652003003
2014-10-23 08:08:23 -07:00
qiankun.miao
d9aac34ece
Add SkMaskCache
...
BUG=skia:
Review URL: https://codereview.chromium.org/670063004
2014-10-23 07:58:17 -07:00
bsalomon
26979c8260
Revert of Delete unused method aliases and texture flag. (patchset #1 id:1 of https://codereview.chromium.org/672753002/ )
...
Reason for revert:
Need to remove one more reference to this in blink before relanding.
Original issue's description:
> Delete unused method aliases and texture flag.
>
> Committed: https://skia.googlesource.com/skia/+/7fdffe41494cbd78a8ee2a6ef0509e46ec77039f
TBR=robertphillips@google.com
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/671203002
2014-10-23 07:41:29 -07:00
bungeman
c20386e393
Update fontMgr to take list of bcp47 language tags.
...
This will enable clients to pass more than one bcp47 tag to ensure
that the most appropriate font is selected.
BUG=chromium:422180
Review URL: https://codereview.chromium.org/670243002
2014-10-23 07:08:06 -07:00
bsalomon
7fdffe4149
Delete unused method aliases and texture flag.
...
Review URL: https://codereview.chromium.org/672753002
2014-10-23 06:56:55 -07:00
joshualitt
58a654452f
fix memory leak
...
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/668333005
2014-10-22 20:53:26 -07:00
joshualitt
99db3b1308
some more expectations, update ignores
...
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/668393002
2014-10-22 20:23:08 -07:00
joshualitt
4a38fbd7f2
New baselines
...
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/668823006
2014-10-22 19:47:37 -07:00
joshualitt
9292fafd4a
one more test to ignore
...
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/674713002
2014-10-22 17:52:31 -07:00
joshualitt
5ead6da4ab
Oval and stroke AA rect now batch
...
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d
Committed: https://skia.googlesource.com/skia/+/18055afb838a278b5a8436cd51dbfbb688e1e0a0
Review URL: https://codereview.chromium.org/664193002
2014-10-22 16:00:30 -07:00
joshualitt
67f7f74fb2
Revert of Oval and stroke AA rect now batch (patchset #7 id:110001 of https://codereview.chromium.org/664193002/ )
...
Reason for revert:
whoops, accidental commit
Original issue's description:
> Oval and stroke AA rect now batch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d
>
> Committed: https://skia.googlesource.com/skia/+/18055afb838a278b5a8436cd51dbfbb688e1e0a0
TBR=robertphillips@google.com ,bsalomon@google.com,joshualitt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/675623002
2014-10-22 15:51:52 -07:00
joshualitt
18055afb83
Oval and stroke AA rect now batch
...
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d
Review URL: https://codereview.chromium.org/664193002
2014-10-22 15:48:48 -07:00
mtklein
9deb696f58
Revert of Oval and stroke AA rect now batch (patchset #7 id:110001 of https://codereview.chromium.org/664193002/ )
...
Reason for revert:
Many GMs failing, e.g. http://build.chromium.org/p/client.skia.android/builders/Test-Android-Nexus5-Adreno330-Arm7-Release/builds/95/steps/Compare%20GMs/logs/stdio
Original issue's description:
> Oval and stroke AA rect now batch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d
TBR=robertphillips@google.com ,joshualitt@google.com,bsalomon@google.com,joshualitt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/671993003
2014-10-22 15:23:29 -07:00