Commit Graph

15543 Commits

Author SHA1 Message Date
bsalomon
6ae83cf7f6 add inverted path draw cases to complexclip gm
Review URL: https://codereview.chromium.org/809163002
2014-12-17 14:38:49 -08:00
egdaniel
bb63cb7d1c Ignore buggy clip gm on Mac 10.9
TBR=
NOTRY=true

BUG=skia:

Review URL: https://codereview.chromium.org/793403005
2014-12-17 14:03:51 -08:00
bsalomon
3e79124a69 Stop creating GrODS for stencilPath commands.
Review URL: https://codereview.chromium.org/816513003
2014-12-17 13:43:13 -08:00
egdaniel
8750924a14 Add Coverage Drawing XP
BUG=skia:

Review URL: https://codereview.chromium.org/808813002
2014-12-17 13:37:13 -08:00
reed
3739913b6a rebaseline after perspective/conic fix
BUG=skia:
TBR=
NOTRY=True

Review URL: https://codereview.chromium.org/816543002
2014-12-17 13:32:20 -08:00
reed
a816d95b1e version 0 of imgconv tool
BUG=skia:

Review URL: https://codereview.chromium.org/809143002
2014-12-17 13:08:17 -08:00
jvanverth
41bc28843a Rebaseline HD7770 GMs for https://codereview.chromium.org/811133002.
TBR=bensong@google.com

Review URL: https://codereview.chromium.org/808173004
2014-12-17 12:07:21 -08:00
reed
baf9daa809 rebaseline after perspective/conic change
BUG=skia:
TBR=
NOTRY=True

Review URL: https://codereview.chromium.org/812863002
2014-12-17 11:28:35 -08:00
robertphillips
eb3379fe70 Turn off warnings as errors for the debugger
Compiling the debugger on Linux was running afoul of warnings-as-errors

Review URL: https://codereview.chromium.org/812673004
2014-12-17 11:27:47 -08:00
egdaniel
df795036fd Add new complexclip3 gm.
This GM is used to test the combined clipping of a complex clip (packman shape)
and a simple one (circle). We loop over all combinations of clip ops, aa/bw clip,
and inverse/non-inverse clips.

This GM triggers a current bug in the gpu clipping code which fires an assert. Thus
the skipGPU flag is set until that bug is fixed.

BUG=skia:

Review URL: https://codereview.chromium.org/798793003
2014-12-17 11:22:37 -08:00
Brian Salomon
4b0a04fd78 Mark flaky gm as ignore-failure on mac 10.9
BUG=skia:3256

Review URL: https://codereview.chromium.org/803443005
2014-12-17 14:05:17 -05:00
jvanverth
f72b752836 Re-enable default sized formats, handling BGRA exception.
Sized texture formats are accepted by OpenGL desktop and ES 3.0,
except when using BGRA with GL_EXT_texture_format_BGRA8888.
In that case we fall back to using the unsized format.

Review URL: https://codereview.chromium.org/811133002
2014-12-17 10:46:01 -08:00
reed
042a6fe10a rebaseline after perspective/conic fix
BUG=skia:
TBR=
NOTRY=True

Review URL: https://codereview.chromium.org/785333004
2014-12-17 09:47:28 -08:00
bsalomon
1f78c0a2dd Stop calling flushGLState in onStencilPath
Review URL: https://codereview.chromium.org/805293003
2014-12-17 09:43:13 -08:00
reed
438d23c627 remove flags that are now in chrome's userconfig
BUG=skia:

Review URL: https://codereview.chromium.org/809073002
2014-12-17 09:40:50 -08:00
reed
dbf9f1973e rebaseline after perspective fix (using conics)
BUG=skia:
TBR=
NOTRY=True

Review URL: https://codereview.chromium.org/811123003
2014-12-17 09:27:35 -08:00
reed
220f926d9d use conics
BUG=skia:

Review URL: https://codereview.chromium.org/455043002
2014-12-17 08:21:05 -08:00
reed
ed9f7a9220 rebaseline after fix to cornerpatheffect
BUG=skia:
TBR=
NOTRY=True

Review URL: https://codereview.chromium.org/814523004
2014-12-17 07:49:59 -08:00
scroggo
c6b8ffa0ac Fix a case of kPartialSuccess.
Missed in https://codereview.chromium.org/647023006/

When JPEG decoder does not have the remainder of the image, it should
return kPartialSuccess.

BUG:b/17419670
BUG:b/18769124

Review URL: https://codereview.chromium.org/807983002
2014-12-17 06:55:02 -08:00
jvanverth
b270fc105d Rebaseline HD7770 images after sized texture format revert.
TBR=bensong@google.com

Review URL: https://codereview.chromium.org/815463002
2014-12-17 06:34:45 -08:00
reed
2b9445b84e fix end-point and conic bugs in cornerpatheffect
fixes bug in GM/patheffect -- will need to be rebaselined

BUG=skia:

Review URL: https://codereview.chromium.org/803213003
2014-12-17 05:50:55 -08:00
skia.buildbots
ccb28e16fd Update SKP version
Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/813743002
2014-12-17 04:52:31 -08:00
kkinnunen
7445aa7d27 webpages_playback.py: Page sets to use changed telemetry API (AddUserStory)
Make page sets to use telemetry.page.page_set.AddUserStory instead of
AddPage.

BUG=chromium:439512
BUG=skia:3196

NOTRY=true

Review URL: https://codereview.chromium.org/811003002
2014-12-17 04:06:54 -08:00
reed
8b5752443f add arcto patheffect
BUG=skia:
TBR=

Review URL: https://codereview.chromium.org/813513003
2014-12-17 01:47:32 -08:00
fmalita
287ccfece6 Whitespace change
To re-trigger the build bots.

TBR=bensong@google.com
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/811623004
2014-12-17 01:25:33 -08:00
robertphillips
a86a233de5 Give the hoisting surfaces surface props
This mimics how layers are set up in SkGpuDevice::onCreateCompatibleDevice

Review URL: https://codereview.chromium.org/809823003
2014-12-16 14:25:08 -08:00
jvanverth
e1869ca25f Revert using sized formats for everything except ES 2.0.
Temporary fix for WritePixels test on Nexus 5/Nexus 7-2013.

Review URL: https://codereview.chromium.org/794273005
2014-12-16 13:32:28 -08:00
bsalomon
d95263c60e Remove GrGpu::flushGraphicsState
Review URL: https://codereview.chromium.org/809843002
2014-12-16 13:05:12 -08:00
scroggo
5a602ca73c Catch the stragglers from skbug.com/3241
Update expectations for tests that were using SkLCGRandom (and now
use SkRandom).

TBR=benchen
BUG=skia:3241

Review URL: https://codereview.chromium.org/813513002
2014-12-16 12:59:56 -08:00
jvanverth
bc102746a4 Update expectations for Win8-HD7770.
TBR=bensong@google.com

Review URL: https://codereview.chromium.org/806283002
2014-12-16 12:00:16 -08:00
scroggo
41b4d77d02 Massive rebaseline for SkRandom.
All of the following GMs have been switched from SkLCGRandom to
SkRandom. Update their expectations for the new images.

-beziers
-circles
-convexpaths
-complexclip2
-complexclip2_rect_aa
-complexclip2_rect_bw
-complexclip2_path_aa
-complexclip2_path_bw
-complexclip2_rrect_aa
-complexclip2_rrect_bw
-cubicpath
-cubicclosepath
-degeneratesegments
-emptypath
-getpostextpath
-hittestpath
-linepath
-lineclosepath
-points
-polygons
-quadpath
-quadclosepath
-strokerects
-strokes_round
-strokes_poly

BUG=skia:3241
TBR=mtklein

Review URL: https://codereview.chromium.org/806263002
2014-12-16 10:59:56 -08:00
jvanverth
3f801cbb67 Change desktop and ES 3.0 to always use sized internal texture formats.
Committed: https://skia.googlesource.com/skia/+/bc02bf0ee4221604796cd6d0394ca3af60c0a579

Review URL: https://codereview.chromium.org/806943002
2014-12-16 09:49:39 -08:00
robertphillips
ed31343805 Alter GM's --mpd option to write out images
Review URL: https://codereview.chromium.org/811863002
2014-12-16 09:31:42 -08:00
fs
88640cf363 SkMatrix44(const SkMatrix&) needs to initialize the type mask
Review URL: https://codereview.chromium.org/791723006
2014-12-16 08:36:11 -08:00
robertphillips
478dd72336 Fix layer hoisting image filter corner cases
This CL fixes 5 bugs related to hoisting image filters:

For image filters the src layer (the one prior to filtering) often needs to be
smaller then the final layer. This requires the saveLayer's optional bounds
to be stored (in SkLayerInfo.h and SkRecordDraw.cpp) and then used in
compute_source_rect and carried around in GrCachedLayer.

The image filters can add an extra offset to the final draw operation.
This is now computed in GrLayerHoister::FilterLayer and  carried around in
GrCachedLayer.

Filtered layers must use exact matches. This is now done in GrLayerCache::lock.

The filter cache requires a valid matrix so it can compute the correct offset.
This is now done in GrLayerHoister::FilterLayer.

Filtered layers need to be drawn with drawSprite while unfiltered (and therefore
hopefully atlased) layers can be drawn with drawBitmap. This is now done in
draw_replacement_bitmap.

Committed: https://skia.googlesource.com/skia/+/702eb9622102599d94ab6798e6227cf29f48c2d3

Review URL: https://codereview.chromium.org/803183003
2014-12-16 08:25:55 -08:00
bsalomon
29097f920a Rename GrGpuGL_program.cpp to GrGLGpuProgramCache.cpp
Review URL: https://codereview.chromium.org/807133002
2014-12-16 08:13:53 -08:00
reed
a7a8b10134 add sample for arcTo
BUG=skia:
TBR=

Review URL: https://codereview.chromium.org/808793002
2014-12-16 08:07:43 -08:00
robertphillips
86f032832f Revert of Fix layer hoisting image filter corner cases (patchset #3 id:40001 of https://codereview.chromium.org/803183003/)
Reason for revert:
Breaking DM

Original issue's description:
> Fix layer hoisting image filter corner cases
>
> This CL fixes 5 bugs related to hoisting image filters:
>
> For image filters the src layer (the one prior to filtering) often needs to be
> smaller then the final layer. This requires the saveLayer's optional bounds
> to be stored (in SkLayerInfo.h and SkRecordDraw.cpp) and then used in
> compute_source_rect and carried around in GrCachedLayer.
>
> The image filters can add an extra offset to the final draw operation.
> This is now computed in GrLayerHoister::FilterLayer and  carried around in
> GrCachedLayer.
>
> Filtered layers must use exact matches. This is now done in GrLayerCache::lock.
>
> The filter cache requires a valid matrix so it can compute the correct offset.
> This is now done in GrLayerHoister::FilterLayer.
>
> Filtered layers need to be drawn with drawSprite while unfiltered (and therefore
> hopefully atlased) layers can be drawn with drawBitmap. This is now done in
> draw_replacement_bitmap.
>
> Committed: https://skia.googlesource.com/skia/+/702eb9622102599d94ab6798e6227cf29f48c2d3

TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/790643009
2014-12-16 07:41:37 -08:00
bsalomon
861e103701 Rename GrGpuGL to GrGLGpu for consistency
Review URL: https://codereview.chromium.org/808593003
2014-12-16 07:33:50 -08:00
robertphillips
702eb96221 Fix layer hoisting image filter corner cases
This CL fixes 5 bugs related to hoisting image filters:

For image filters the src layer (the one prior to filtering) often needs to be
smaller then the final layer. This requires the saveLayer's optional bounds
to be stored (in SkLayerInfo.h and SkRecordDraw.cpp) and then used in
compute_source_rect and carried around in GrCachedLayer.

The image filters can add an extra offset to the final draw operation.
This is now computed in GrLayerHoister::FilterLayer and  carried around in
GrCachedLayer.

Filtered layers must use exact matches. This is now done in GrLayerCache::lock.

The filter cache requires a valid matrix so it can compute the correct offset.
This is now done in GrLayerHoister::FilterLayer.

Filtered layers need to be drawn with drawSprite while unfiltered (and therefore
hopefully atlased) layers can be drawn with drawBitmap. This is now done in
draw_replacement_bitmap.

Review URL: https://codereview.chromium.org/803183003
2014-12-16 07:27:30 -08:00
mtklein
9a5380dd35 Revert of Use SkPaint::getFontBounds() for text bounding boxes in pictures. (patchset #1 id:1 of https://codereview.chromium.org/805983003/)
Reason for revert:
Test-Win7-ShuttleA-HD2000-x86-Debug-GDI

Original issue's description:
> Use SkPaint::getFontBounds() for text bounding boxes in pictures.
>
> Now that SkTextBlobs have landed, this is a perf no-op, but it at least lets us eliminate a bunch of questionable hacks.
>
> CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot,Test-Mac10.8-MacMini4.1-GeForce320M-x86_64-Debug-Trybot
>
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/02d2b9831579173e783569530ab7bae08de907e9

TBR=reed@google.com,mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/789793004
2014-12-16 06:31:01 -08:00
jvanverth
bdd22a35bf Revert of Change desktop and ES 3.0 to always use sized internal texture formats. (patchset #2 id:20001 of https://codereview.chromium.org/806943002/)
Reason for revert:
Failing GMs on Mac, Windows and Ubuntu12.

Original issue's description:
> Change desktop and ES 3.0 to always use sized internal texture formats.
>
> Committed: https://skia.googlesource.com/skia/+/bc02bf0ee4221604796cd6d0394ca3af60c0a579

TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/805193003
2014-12-16 06:17:59 -08:00
mtklein
02d2b98315 Use SkPaint::getFontBounds() for text bounding boxes in pictures.
Now that SkTextBlobs have landed, this is a perf no-op, but it at least lets us eliminate a bunch of questionable hacks.

CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot,Test-Mac10.8-MacMini4.1-GeForce320M-x86_64-Debug-Trybot

BUG=skia:

Review URL: https://codereview.chromium.org/805983003
2014-12-16 06:17:54 -08:00
jvanverth
bc02bf0ee4 Change desktop and ES 3.0 to always use sized internal texture formats.
Review URL: https://codereview.chromium.org/806943002
2014-12-16 05:45:49 -08:00
joshualitt
af7aac81b4 rebase gms and update ignore
NOTRY=True
TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/811433004
2014-12-15 17:42:43 -08:00
joshualitt
a047f12bd7 whitespace change to context to force rebuild of skia
NOTREECHECKS=True
NOTRY=True
TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/807663002
2014-12-15 14:55:30 -08:00
joshualitt
9b98932ada This change will ultimately pull uniform color, and to a much lesser degree uniform coverage, into GPs. There are still some loose ends because drawstate has the ability to override the GP, but fixing these cleanly will have to wait until we have deferred geometry in place and can make attribute / uniform decisions on the fly.
BUG=skia:

Review URL: https://codereview.chromium.org/746423007
2014-12-15 14:16:27 -08:00
bsalomon
bc3d0de755 Move all non-program cache functions out of GrGpuGL_program.cpp
Review URL: https://codereview.chromium.org/794343006
2014-12-15 13:45:04 -08:00
reed
0393912de7 pass dst-rect through from drawBitmapRect for better precision
BUG=skia:

Review URL: https://codereview.chromium.org/797963002
2014-12-15 13:42:51 -08:00
mtklein
0faaac6418 Look at DYNAMIC_ANNOTATIONS_ENABLED now.
Defining SK_DYNAMIC_ANNOTATIONS_ENABLED as 1 whenever DYNAMIC_ANNOTATIONS_ENABLED was 1
seems to be working fine for Chrome.  Should be we can just use DYNAMIC_ANNOTATIONS_ENABLED.

BUG=skia:

Review URL: https://codereview.chromium.org/810513002
2014-12-15 13:38:07 -08:00