aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* remove SK_SUPPORT_LEGACY_BLURMASKFILTER_STYLE -- not neededGravatar reed2014-06-16
| | | | | | | | | BUG=skia: R=scroggo@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/332163005
* Refine bench_record and bench_playback:Gravatar mtklein2014-06-16
| | | | | | | | | | | | | | | | - use high-precision wall timer only - warm caches once before measuring - measure independent samples, calculating statistics - add --verbose to control how much data we output Also removed some unloved features from bench_record. BUG=skia: R=jcgregorio@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/338203002
* Fix Windows botsGravatar borenet2014-06-16
| | | | | | | | | BUG=skia: R=mtklein@google.com, rmistry@google.com Author: borenet@google.com Review URL: https://codereview.chromium.org/339733002
* Revert of Temporarily limit x86 SIMD to SSE2 only, to see effect on all ↵Gravatar mtklein2014-06-16
| | | | | | | | | | | | | | | | | | | | | | | | benches and bots. (https://codereview.chromium.org/331193004/) Reason for revert: Experiment is over: disabling SSSE3 is a 25-50% perf regression for bitmap scaling on every machine we've got. Original issue's description: > Temporarily limit x86 SIMD to SSE2 only, to see effect on all benches and bots. > > BUG=372232 > > Committed: https://skia.googlesource.com/skia/+/f1e5a04832e4d350f9ebf5d556c6d3897345f883 R=reed@google.com, mtklein@chromium.org TBR=mtklein@chromium.org, reed@google.com NOTREECHECKS=true NOTRY=true BUG=372232 Author: mtklein@google.com Review URL: https://codereview.chromium.org/332213005
* Prepare for internal SaveFlags removal.Gravatar fmalita2014-06-16
| | | | | | | | | | | | | Chromium has a couple of SkCanvas subclasses which override willSave(). This adds a transitional shim to facilitate converting these to the parameter-less API. BUG=skia:2297 R=reed@google.com, scroggo@google.com Author: fmalita@chromium.org Review URL: https://codereview.chromium.org/338913002
* Cleaned a repeated if(condition) with the same conditionGravatar piotaixr2014-06-16
| | | | | | | | R=junov@chromium.org Author: piotaixr@chromium.org Review URL: https://codereview.chromium.org/338933002
* Perlin noise cleanup.Gravatar senorblanco2014-06-16
| | | | | | | | | | | Remove Simplex noise implementation and now-redundant base classes. BUG=skia: R=sugoi@chromium.org, sugoi@google.com, reed@google.com Author: senorblanco@chromium.org Review URL: https://codereview.chromium.org/336043002
* Temporarily limit x86 SIMD to SSE2 only, to see effect on all benches and bots.Gravatar mtklein2014-06-16
| | | | | | | | | BUG=372232 R=reed@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/331193004
* Armv8/64bit doesn't (apparently) need rebaseliningGravatar kevin.petit2014-06-16
| | | | | | | | | | BUG=skia: R=reed@google.com NOTRY=True Author: kevin.petit@arm.com Review URL: https://codereview.chromium.org/334743009
* Add sugoi@ to CQ_COMMITTERS.Gravatar senorblanco2014-06-16
| | | | | | | | R=junov@chromium.org Author: senorblanco@chromium.org Review URL: https://codereview.chromium.org/337033005
* fix legacy code path for SK_SUPPORT_LEGACY_IMAGEDECODER_CONFIGGravatar reed2014-06-16
| | | | | | | | | BUG=skia: R=scroggo@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/332223002
* Enable compiling with automatically-downloaded VS toolchain on Windows botsGravatar Eric Boren2014-06-16
| | | | | | | | | Requires that the bots have CHROME_HEADLESS=1 and CHROME_PATH set. BUG=skia:2216 R=epoger@google.com Review URL: https://codereview.chromium.org/332833002
* Update SKP version to 24Gravatar borenet2014-06-15
| | | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Author: borenet@google.com Review URL: https://codereview.chromium.org/332983003
* Minor cleanups in picture_utils header file.Gravatar tfarina2014-06-15
| | | | | | | | | | | | | | 1) Remove unused include. 2) Remove unused forward declarations. BUG=None TEST=make tests R=epoger@google.com TBR=epoger@google.com Author: tfarina@chromium.org Review URL: https://codereview.chromium.org/339493004
* Update SKP version to 23Gravatar borenet2014-06-14
| | | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Author: borenet@google.com Review URL: https://codereview.chromium.org/332843006
* stop using SkBitmap::ConfigGravatar reed2014-06-14
| | | | | | | | R=scroggo@google.com Author: reed@chromium.org Review URL: https://codereview.chromium.org/338493005
* Update SKP version to 22Gravatar borenet2014-06-14
| | | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Author: borenet@google.com Review URL: https://codereview.chromium.org/332013002
* Delete unused path_utils.* component.Gravatar tfarina2014-06-14
| | | | | | | | | | | | | filter tool includes it but isn't using, and nobody else seems to be using it either. So just remove it now. BUG=None TEST=make filter R=mtklein@google.com Author: tfarina@chromium.org Review URL: https://codereview.chromium.org/324273002
* hide virtuals on device for width/height/isopaqueGravatar reed2014-06-14
| | | | | | | | | R=bsalomon@google.com NOTREECHECKS=True Author: reed@chromium.org Review URL: https://codereview.chromium.org/334993002
* Testing trybot triggeringGravatar rmistry2014-06-13
| | | | | | | | | | | | CQ_EXTRA_TRYBOTS=tryserver.skia:Build-Ubuntu13.10-GCC4.8-x86_64-Debug-CrOS_Link-Trybot BUG=skia:2659 NOTREECHECKS=true TBR= (SkipBuildbotRuns) Author: rmistry@google.com Review URL: https://codereview.chromium.org/332933005
* Extract "text align proc" functions as reusable classesGravatar kkinnunen2014-06-12
| | | | | | | | | | | | | | | | | | | | | | | Extract "text align proc" as reusable classes. These classes need to be used when writing GrTextContext subclasses. Moves "text align proc" code that is duplicated in SkDraw and SkBitmapTextContext to SkDrawProcs.h and SkTextMapState.h. This functionality is also used in the new GrStencilAndCoverTextContext. Creates new functor classes SkTextAlignProc and SkTextAlignProcScalar which represent the previous "text align procs". Moves TextMapState from SkDraw to SkTextMapStateProc and make it similar functor. The transform should be comparable in speed, as the compiler can and does avoid the call and eliminate some of the branches. R=jvanverth@google.com, reed@google.com Author: kkinnunen@nvidia.com Review URL: https://codereview.chromium.org/335573002
* Randomize seed for SkDiscretePathEffect::filterPath()Gravatar rs.prinja2014-06-12
| | | | | | | | | | | | | | | | | | Fix for https://code.google.com/p/skia/issues/detail?id=2581. Randomizes the seed in SkDiscretePathEffect::filterPath(). Prior to this we were using the path length as a seed. Now, if we have two different paths with identical contents and we apply an SkDiscretePathEffect to each, we obtain two different random paths. Previously, we would obtain two overlapping paths (identical path contents leading to the same seed). BUG=skia: R=reed@google.com, scroggo@google.com Author: rs.prinja@samsung.com Review URL: https://codereview.chromium.org/311803002
* hide Config in SkImageDecoder -- use SkColorType insteadGravatar reed2014-06-12
| | | | | | | | | | patch from issue 334613003 TBR=scroggo Author: reed@chromium.org Review URL: https://codereview.chromium.org/334793002
* New baselines for perlin noise tests affected by ce6a35.Gravatar senorblanco2014-06-12
| | | | | | | | | | | | | New baselines for perlinnoise, imagefiltersclipped and imagefiltersscaled GMs. R=scroggo@google.com TBR=scroggo NOTRY=true Author: senorblanco@chromium.org Review URL: https://codereview.chromium.org/336703002
* Remove SkPicture SK_SUPPORT_LEGACY_RECORDING_FLAG flagGravatar robertphillips2014-06-12
| | | | | | | | | | | | | | Chromium/Blink should no longer need this flag after: Chromium: Remove use of kUsePathBoundsForClip_RecordingFlag https://codereview.chromium.org/322123002/ Blink: Remove use of kUsePathBoundsForClip_RecordingFlag https://codereview.chromium.org/326953002/ R=mtklein@google.com, scroggo@google.com, reed@google.com, bsalomon@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/328343002
* rebaselines for removal of unitmapperGravatar reed2014-06-12
| | | | | | | | | TBR= NOTRY=True Author: reed@google.com Review URL: https://codereview.chromium.org/330513005
* Add more tests to ignore-tests.txt.Gravatar Stephen White2014-06-12
| | | | | | | | | | | Add imagefiltersclipped and imagefiltersscaled to ignored-tests.txt until they can be rebaselined, since they're failing on Android after https://codereview.chromium.org/332523006/. TBR=scroggo BUG=skia: Review URL: https://codereview.chromium.org/336663002
* Revert of third try at landing improved blur rect; this time with more ↵Gravatar scroggo2014-06-12
| | | | | | | | | | | | | | | | | | | | | | | | | | correctness (https://codereview.chromium.org/331443003/) Reason for revert: Failing layout test: https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux/32762/layout-test-results/virtual/gpu/fast/canvas/canvas-draw-canvas-on-canvas-shadow-pretty-diff.html Original issue's description: > third try at landing improved blur rect; this time with more correctness > > BUG=skia:2095 > R=bsalomon@google.com > TBR=bsalomon > > Committed: https://skia.googlesource.com/skia/+/72abfc2b4e7caead660f6b6a05e60d05eaf1a66f R=bsalomon@google.com, reed@google.com, humper@google.com TBR=bsalomon@google.com, humper@google.com, reed@google.com NOTREECHECKS=true NOTRY=true BUG=skia:2095 Author: scroggo@google.com Review URL: https://codereview.chromium.org/333763002
* Add another fail-ignore for flaky dashing4 gmGravatar egdaniel2014-06-12
| | | | | | | | | | | NOTREECHECKS=True NOTRY=True TBR=bsalomon@google.com BUG=skia: Author: egdaniel@google.com Review URL: https://codereview.chromium.org/329223007
* Greenify bench bot Perf-Ubuntu12-ShuttleA-GTX660-x86-Release at build 968Gravatar bensong2014-06-12
| | | | | | | | | | | | R=bsalomon@google.com TBR=scroggo@google.com Bypassing trybots: NOTRY=true Author: bensong@google.com Review URL: https://codereview.chromium.org/338453004
* Added in framework to get more bench dataGravatar kelvinly2014-06-12
| | | | | | | | | | BUG=skia: NOTRY=true R=jcgregorio@google.com, bensong@google.com Author: kelvinly@google.com Review URL: https://codereview.chromium.org/331683003
* Fix tiled perlin noise.Gravatar senorblanco2014-06-12
| | | | | | | | | | | | | | | | | | | | | | | | | It turns out that the perlin implementation we inherited from WebKit does not actually generate tileable noise (see Chromium bug http://crbug.com/383495). The main problem is that when generating coordinates for gradient interpolation, it was attempting to wrap both x and (x + 1) simultaneously at the tile boundary (that is, either both or neither are wrapped). This obviously won't work, since along the tile seams, (x + 1) should be wrapped, but x should not. The same is true in y. This patch fixes both the CPU and GPU paths, renames some variables to more closely match the spec, and modifies the perlin noise GM to actually test tiling. (Note that the clipping the GM was doing was removed, since it's superfluous: it used to be necessary for image filters, but isn't anymore, and this isn't an image filter GM anyway.) R=sugoi@google.com, sugoi TBR=senorblanco Author: senorblanco@chromium.org Review URL: https://codereview.chromium.org/332523006
* Use vertex attributes for dash effect in gpuGravatar egdaniel2014-06-12
| | | | | | | | | | | | | This will allow us to batch dashed lines together when drawing. Also, this removes the need for a coord transform matrix in the shader, thus we save the cost of uploading a new matrix uniform everytime we do a simple transform to the dashed line we are drawing. BUG=skia: R=bsalomon@google.com Author: egdaniel@google.com Review URL: https://codereview.chromium.org/326103002
* manual bench rebase after 679426c using old algorithmGravatar bensong2014-06-12
| | | | | | | | | | | | R=robertphillips@google.com TBR=borenet@google.com, robertphillips@google.com Bypassing trybots: NOTRY=true Author: bensong@google.com Review URL: https://codereview.chromium.org/330103002
* remove GetDeviceConfig/SetDeviceConfigGravatar reed2014-06-12
| | | | | | | | | BUG=skia: R=scroggo@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/332453002
* Use scratch with LATC data if possibleGravatar krajcevski2014-06-12
| | | | | | | | R=robertphillips@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/330593004
* Cleanup: Delete sk_tools::make_filepath() in favor of SkOSPath::SkPathJoin().Gravatar tfarina2014-06-12
| | | | | | | | | | BUG=None TEST=make tools tests && out/Debug/tests R=epoger@google.com Author: tfarina@chromium.org Review URL: https://codereview.chromium.org/327403002
* removes deprecated -q flag for gsutil.Gravatar bensong2014-06-12
| | | | | | | | | | | BUG=skia: NOTRY=true R=kelvinly@google.com TBR=kelvinly@google.com Author: bensong@google.com Review URL: https://codereview.chromium.org/333643005
* Really revert bench trigger changes.Gravatar bensong2014-06-12
| | | | | | | | | | | BUG=skia: NOTRY=true R=kelvinly@google.com TBR=kelvinly@google.com Author: bensong@google.com Review URL: https://codereview.chromium.org/324403009
* Rebase flakey windows dash4 gmGravatar egdaniel2014-06-12
| | | | | | | | | | | NOTREECHECKS=True NOTRY=True TBR=bsalomon@google.com BUG=skia:2667 Author: egdaniel@google.com Review URL: https://codereview.chromium.org/335443003
* Revert trigger changesGravatar kelvinly2014-06-12
| | | | | | | | | | BUG=skia: NOTRY=true R=robertphillips@google.com, bensong@google.com, jcgregorio@google.com Author: kelvinly@google.com Review URL: https://codereview.chromium.org/334673002
* Add expectations for dashing gmGravatar egdaniel2014-06-12
| | | | | | | | | TBR=bsalomon@google.com BUG=skia: Author: egdaniel@google.com Review URL: https://codereview.chromium.org/330033002
* Whitespace change to test trybot triggeringGravatar rmistry2014-06-12
| | | | | | | | | | | BUG=skia: CQ_EXTRA_TRYBOTS=tryserver.skia:Build-Ubuntu13.10-GCC4.8-Arm7-Release-Android-Trybot TBR= (SkipBuildbotRuns) Author: rmistry@google.com Review URL: https://codereview.chromium.org/333663002
* manual bench rebase after 679426cGravatar scroggo2014-06-12
| | | | | | | | | | | | | | | | In response to: - new SKPs - Kelvin's changes around regression testing R=robertphillips@google.com, kelvinly@google.com TBR=robertphillips@google.com Bypassing trybots: NOTRY=true Author: scroggo@google.com Review URL: https://codereview.chromium.org/332483004
* Remove SkPicture pointer from SkPicturePlaybackGravatar robertphillips2014-06-12
| | | | | | | | | | This CL simplifies the relationship between SkPicture and SkPicturePlayback by moving the path heap into SkPicturePlayback and removing SkPicturePlayback's SkPicture pointer. R=mtklein@google.com, reed@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/334493002
* Update SKP version to 21Gravatar borenet2014-06-12
| | | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Author: borenet@google.com Review URL: https://codereview.chromium.org/327213004
* rebaseline_server: delete no-longer-used weightedDiffMeasureGravatar epoger2014-06-11
| | | | | | | | | | | | (replaced by perceptual diff) NOTRY=True R=rmistry@google.com TBR=rmistry Author: epoger@google.com Review URL: https://codereview.chromium.org/334533003
* rebaseline_server: change 2 image expectations within unittestsGravatar epoger2014-06-11
| | | | | | | | | | NOTRY=True R=rmistry@google.com TBR=rmistry Author: epoger@google.com Review URL: https://codereview.chromium.org/329933002
* Suppress libpoppler and its dependent liblcms in TSAN.Gravatar mtklein2014-06-11
| | | | | | | | | | | This will let us eventually turn PDF back on in DM. BUG=skia: R=halcanary@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/330433003
* Add ifdef to not use function pointer that we don't have yet on Chromium??Gravatar krajcevski2014-06-11
| | | | | | | | R=bsalomon@google.com, robertphillips@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/326423003
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-07 11:00:15 +0000