aboutsummaryrefslogtreecommitdiffhomepage
path: root/tools
Commit message (Collapse)AuthorAge
...
* Create jsondiff tool to capture diffs between two JSON expectations files.Gravatar epoger@google.com2013-07-02
| | | | | | | | | | | | | This will be used to view diffs before committing rebaselines, or to review exactly what changed between two baseline revisions in the past. Part of Step 3 in https://goto.google.com/ChecksumTransitionDetail R=scroggo@google.com Review URL: https://codereview.chromium.org/18103005 git-svn-id: http://skia.googlecode.com/svn/trunk@9878 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix rebaseline.py self-test to account for r9859Gravatar epoger@google.com2013-07-02
| | | | | | | | R=robertphillips@google.com Review URL: https://codereview.chromium.org/18559002 git-svn-id: http://skia.googlecode.com/svn/trunk@9872 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add N4 to rebaseline.pyGravatar robertphillips@google.com2013-07-02
| | | | | | | | | https://codereview.chromium.org/18023014/ (SkipBuildbotRuns) git-svn-id: http://skia.googlecode.com/svn/trunk@9859 2bbb7eff-a529-9590-31e7-b0007b416f81
* Replace SkPicture(SkStream) constructors with a factory.Gravatar scroggo@google.com2013-06-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SkPicture: Remove the constructors which take an SkStream as an argument. Rather than having to check a variable for success, the factory will return NULL on failure. Add a protected function for determining if an SkStream is an SKP to share code with SkTimedPicture. In the factory, check for a NULL SkStream. Use a default decoder (from BUG: https://code.google.com/p/skia/issues/detail?id=1325) SkDebuggerGUI: Call SkPicture::CreateFromStream when necessary. Write a factory for creating SkTimedPictures and use it. Use the factory throughout tools. Add include/lazy to utils and effects gyp include_dirs so SkPicture.h can reference SkImageDecoder.h which references SkBitmapFactory.h (in include/lazy). Changes code Chromium uses, so this will require a temporary Skia and then a change to Chromium to use the new Skia code. TODO: Create a decoder that does nothing to be used by pinspect, lua pictures, etc, and allow it to not assert in SkOrderedReadBuffer. R=reed@google.com Review URL: https://codereview.chromium.org/17113004 git-svn-id: http://skia.googlecode.com/svn/trunk@9822 2bbb7eff-a529-9590-31e7-b0007b416f81
* Skip the TIMESTAMP file.Gravatar scroggo@google.com2013-06-28
| | | | | | | | | | | Allows run_decoding_tests.py to pass the name of the folder, so Android will not fail on account of the long adb command. R=borenet@google.com Review URL: https://codereview.chromium.org/18175008 git-svn-id: http://skia.googlecode.com/svn/trunk@9817 2bbb7eff-a529-9590-31e7-b0007b416f81
* Print the file being deserialized before deserializing.Gravatar scroggo@google.com2013-06-28
| | | | | | | | | | That way it is easier to tell which file failed. R=edisonn@google.com Review URL: https://codereview.chromium.org/18219002 git-svn-id: http://skia.googlecode.com/svn/trunk@9811 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix run_decoding_tests on xoom.Gravatar scroggo@google.com2013-06-27
| | | | | | | | | | | | | | | | | Builder Test-Android-Xoom-Tegra2-Arm7-Debug fails run_decoding_tests due to a couple of images. Add a way in skimage to ignore failures if an image is expected to fail. Add an expectations file for xoom that includes ignore-failure for the two files which are failing. I have created https://code.google.com/p/skia/issues/detail?id=1382 to track the fact that these images fail to decode, despite the bot showing green. R=borenet@google.com Review URL: https://codereview.chromium.org/18023008 git-svn-id: http://skia.googlecode.com/svn/trunk@9797 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add modulo flag to lua_pictures.Gravatar commit-bot@chromium.org2013-06-21
| | | | | | | | | | | | The intended use is spawning processes to save time. R=caryclark@google.com, reed@google.com Author: sglez@google.com Review URL: https://chromiumcodereview.appspot.com/16950025 git-svn-id: http://skia.googlecode.com/svn/trunk@9736 2bbb7eff-a529-9590-31e7-b0007b416f81
* Treat empty directory properly in skimage.Gravatar scroggo@google.com2013-06-21
| | | | | | | | | | | | | | | | Instead of treating an empty directory as a file that failed to decode, treat it as an empty directory. Add self tests to check for this. Builds upon from https://codereview.chromium.org/16866010, which was reverted. R=epoger@google.com Review URL: https://codereview.chromium.org/17101029 git-svn-id: http://skia.googlecode.com/svn/trunk@9730 2bbb7eff-a529-9590-31e7-b0007b416f81
* Allow skimage expectations file to not exist.Gravatar scroggo@google.com2013-06-19
| | | | | | | | | | | If the file is not there, assume that all images are mismatches. This allows running the test before the initial rebaseline. R=epoger@google.com Review URL: https://codereview.chromium.org/17448011 git-svn-id: http://skia.googlecode.com/svn/trunk@9690 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py: split image-based rebaselining, which will go away soon, into ↵Gravatar epoger@google.com2013-06-19
| | | | | | | | | | | | | | its own script (part of step 3 in https://goto.google.com/ChecksumTransitionDetail ) also adds new --expectations-root option R=scroggo@google.com, senorblanco@chromium.org Review URL: https://codereview.chromium.org/17379004 git-svn-id: http://skia.googlecode.com/svn/trunk@9689 2bbb7eff-a529-9590-31e7-b0007b416f81
* force linking inside of skdiffGravatar zachr@google.com2013-06-18
| | | | | | Review URL: https://codereview.chromium.org/16879012 git-svn-id: http://skia.googlecode.com/svn/trunk@9669 2bbb7eff-a529-9590-31e7-b0007b416f81
* Rename SkDrawCommand subclassesGravatar commit-bot@chromium.org2013-06-18
| | | | | | | | | | | | | | | | | | | | | | SkDrawCommand subclass names are too generic and likely to clash in the default namespace. This makes the header unusable in Chrome - for example, on Mac builders: ../../third_party/skia/src/utils/debugger/SkDrawCommand.h:251:7: error: redefinition of 'Comment' class Comment : public SkDrawCommand { ^ /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.6.sdk/System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/AIFF.h:192:8: note: previous definition is here struct Comment { This CL renames SkDrawCommand subclasses using the following pattern: "${cmd}" -> "Sk${cmd}Command". R=robertphillips@google.com, bungeman@google.com, djsollen@google.com Author: fmalita@chromium.org Review URL: https://chromiumcodereview.appspot.com/17101005 git-svn-id: http://skia.googlecode.com/svn/trunk@9667 2bbb7eff-a529-9590-31e7-b0007b416f81
* Allow files, making ".skp files or directories are required." less of a lie.Gravatar commit-bot@chromium.org2013-06-18
| | | | | | | | | | | BUG= R=reed@google.com Author: mtklein@google.com Review URL: https://chromiumcodereview.appspot.com/17151008 git-svn-id: http://skia.googlecode.com/svn/trunk@9665 2bbb7eff-a529-9590-31e7-b0007b416f81
* skimage: revert r9642Gravatar epoger@google.com2013-06-18
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@9647 2bbb7eff-a529-9590-31e7-b0007b416f81
* skimage: if --readPath points at an empty dir, create an empty expectations fileGravatar epoger@google.com2013-06-17
| | | | | | | | TBR=scroggo Review URL: https://codereview.chromium.org/17294006 git-svn-id: http://skia.googlecode.com/svn/trunk@9642 2bbb7eff-a529-9590-31e7-b0007b416f81
* skimage self-tests: confirm correctness of bitmap hash valuesGravatar epoger@google.com2013-06-13
| | | | | | | | R=scroggo@google.com Review URL: https://codereview.chromium.org/16866010 git-svn-id: http://skia.googlecode.com/svn/trunk@9591 2bbb7eff-a529-9590-31e7-b0007b416f81
* use ForceLinking to get codecs into lua_pictureGravatar reed@google.com2013-06-12
| | | | | | | | | BUG= R=scroggo@google.com Review URL: https://codereview.chromium.org/16847002 git-svn-id: http://skia.googlecode.com/svn/trunk@9549 2bbb7eff-a529-9590-31e7-b0007b416f81
* add script to count glyph -vs- char draw callsGravatar reed@google.com2013-06-12
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@9548 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py: keep going if we encounter an "INVALID" entry in the JSON fileGravatar epoger@google.com2013-06-12
| | | | | | Review URL: https://codereview.chromium.org/16826003 git-svn-id: http://skia.googlecode.com/svn/trunk@9543 2bbb7eff-a529-9590-31e7-b0007b416f81
* Small change that eliminates Parse Errors with encoded skps.Gravatar commit-bot@chromium.org2013-06-12
| | | | | | | | | | R=reed@google.com, caryclark@google.com, scroggo@google.com Author: sglez@google.com Review URL: https://chromiumcodereview.appspot.com/16034015 git-svn-id: http://skia.googlecode.com/svn/trunk@9538 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py: try to download images from Google Storage before skia-autogenGravatar epoger@google.com2013-06-12
| | | | | | | | | | This will allow us to complete Step 1 of https://goto.google.com/ChecksumTransitionDetail R=scroggo@google.com Review URL: https://codereview.chromium.org/16311011 git-svn-id: http://skia.googlecode.com/svn/trunk@9534 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py: add --add-new option, and only add new expectations if it is setGravatar epoger@google.com2013-06-12
| | | | | | | | R=senorblanco@chromium.org Review URL: https://codereview.chromium.org/16363006 git-svn-id: http://skia.googlecode.com/svn/trunk@9527 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py: improve readability of dry-run outputGravatar epoger@google.com2013-06-11
| | | | | | | | Split off of already-reviewed https://codereview.chromium.org/15789010/ ('rebaseline.py: --tests and --configs are now FILTERS within json results') Review URL: https://codereview.chromium.org/16509006 git-svn-id: http://skia.googlecode.com/svn/trunk@9507 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py: if rebaselining all platforms (not listing them explicitly), ↵Gravatar epoger@google.com2013-06-11
| | | | | | | | | | allow missing platforms TBR=any-reviewer Review URL: https://codereview.chromium.org/16782003 git-svn-id: http://skia.googlecode.com/svn/trunk@9506 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py: in option names, replace underscores with hyphensGravatar epoger@google.com2013-06-11
| | | | | | | | Split off of already-reviewed https://codereview.chromium.org/15789010/ ('rebaseline.py: --tests and --configs are now FILTERS within json results') Review URL: https://codereview.chromium.org/15904018 git-svn-id: http://skia.googlecode.com/svn/trunk@9504 2bbb7eff-a529-9590-31e7-b0007b416f81
* more paint methods to luaGravatar mike@reedtribe.org2013-06-09
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@9483 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Housekeeper-NightlyGravatar skia.committer@gmail.com2013-06-08
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@9478 2bbb7eff-a529-9590-31e7-b0007b416f81
* SkDocument base for pdf, xps, etc.Gravatar reed@google.com2013-06-07
| | | | | | | | R=scroggo@google.com Review URL: https://codereview.chromium.org/16660002 git-svn-id: http://skia.googlecode.com/svn/trunk@9476 2bbb7eff-a529-9590-31e7-b0007b416f81
* Preparations for running skimage tool on bots.Gravatar scroggo@google.com2013-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | skimage_main: Specify the full name of the expectations file to write on the command line, rather than the folder to write 'results.json'. This is consistent with writeJsonSummaryPath in GM, as well as the format for reading the file. Further, it allows more flexibility to name the file. Do not force opaque when writing bitmaps. skimage does its comparison on the raw bitmaps. The images are written for visual comparison, and mostly transparent bitmaps (like resources/plane.png) aren't meaningful when forced opaque. Provide an option to to write only mismatches to a directory. Handled similar to the writePath. Report a failure if there is no expectations file. This allows writing to the mismatchPath so we can create an initial baseline. Pass SkBitmap by const&. skimage_self_test: Include the name of the json file in the parameter passed to createExpectationsPath. The name has been changed to be more specific. .gitignore: Ignore the new name of the file created by skimage_self_test. run.sh: Run skimage_self_tests.py BUG=http://code.google.com/p/skia/issues/detail?id=1241 R=epoger@google.com Review URL: https://codereview.chromium.org/15813016 git-svn-id: http://skia.googlecode.com/svn/trunk@9474 2bbb7eff-a529-9590-31e7-b0007b416f81
* Run skimage as part of run.shGravatar scroggo@google.com2013-06-07
| | | | | | | | | | | | | Test skimage to ensure that it does not break. Split off from https://codereview.chromium.org/15813016/ BUG=http://code.google.com/p/skia/issues/detail?id=1241 R=epoger@google.com Review URL: https://codereview.chromium.org/15774017 git-svn-id: http://skia.googlecode.com/svn/trunk@9473 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py self-tests: split test inputs and outputs into completely ↵Gravatar epoger@google.com2013-06-06
| | | | | | | | separate directories Review URL: https://codereview.chromium.org/16557002 git-svn-id: http://skia.googlecode.com/svn/trunk@9459 2bbb7eff-a529-9590-31e7-b0007b416f81
* add more rebaseline test resources (no behavioral changes)Gravatar epoger@google.com2013-06-05
| | | | | | | | (SkipBuildbotRuns) Review URL: https://codereview.chromium.org/15774019 git-svn-id: http://skia.googlecode.com/svn/trunk@9446 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py: if --tests is not specified, get test list from ↵Gravatar epoger@google.com2013-06-05
| | | | | | | | | | actual-results.json R=scroggo@google.com Review URL: https://codereview.chromium.org/16093025 git-svn-id: http://skia.googlecode.com/svn/trunk@9443 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py : add self-testsGravatar epoger@google.com2013-06-04
| | | | | | | | R=senorblanco@chromium.org Review URL: https://codereview.chromium.org/15660014 git-svn-id: http://skia.googlecode.com/svn/trunk@9426 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Housekeeper-NightlyGravatar skia.committer@gmail.com2013-06-01
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@9387 2bbb7eff-a529-9590-31e7-b0007b416f81
* sort the glyphID arraysGravatar reed@google.com2013-05-31
| | | | | | add head/tail code git-svn-id: http://skia.googlecode.com/svn/trunk@9376 2bbb7eff-a529-9590-31e7-b0007b416f81
* Separate core and images project.Gravatar scroggo@google.com2013-05-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SkImage calls functions on SkImageDecoder and SkImageEncoder. This is desired behavior, and it is also desired to include SkImage as a part of core. In order to keep core from depending on images, update SkImageDecoder_empty.cpp to implement all of SkImageDecoder and SkImageEncoder. This file will be built by chrome (in https://codereview.chromium.org/15960015). Move force_linking from SkImageDecoder.cpp to its own file. It must be called to force linking with the image decoders if desired. Call the function in tools that need it: sk_image render_pictures render_pdfs sk_hello filter bench_pictures debugger SkImageDecoder: Derive from SkNoncopyable, instead of duplicating its hiding of constructors. skhello: Return rather than trying to write a null SkData to the stream. Revert "Hamfistedly removed core dependence on images" (commit 0f05f682a90bc125323677abf3476e1027d174f5) and "Move SkImage::encode to SkImage_Codec.cpp." (commit 83e47a954d0bf65439f3d9c0c93213063dd70da3.) These two commits were temporary fixes that this change cleans up. SkSnapshot.cpp: Check for a NULL encoder returned by SkImageEncoder::Create. BUG=https://code.google.com/p/skia/issues/detail?id=1275 R=djsollen@google.com, robertphillips@google.com Review URL: https://codereview.chromium.org/15806010 git-svn-id: http://skia.googlecode.com/svn/trunk@9364 2bbb7eff-a529-9590-31e7-b0007b416f81
* add script to scrape glyph usage in drawText callsGravatar reed@google.com2013-05-30
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@9353 2bbb7eff-a529-9590-31e7-b0007b416f81
* rebaseline.py: use argparse command-line flags for more flexibilityGravatar epoger@google.com2013-05-30
| | | | | | | | | | | | | | | | | | | | --tests is the only mandatory argument. If you used to run this: rebaseline.py aaclip bigmatrix Run this instead: rebaseline.py --tests aaclip bigmatrix That's the only change you NEED to make. And then, if you WANT to specify --configs, --subdirs, etc. you CAN. R=senorblanco@chromium.org Review URL: https://codereview.chromium.org/15675010 git-svn-id: http://skia.googlecode.com/svn/trunk@9348 2bbb7eff-a529-9590-31e7-b0007b416f81
* Refactor rebaseline.py into functions ; no behavioral changes.Gravatar epoger@google.com2013-05-29
| | | | | | | | R=senorblanco@chromium.org Review URL: https://codereview.chromium.org/16160008 git-svn-id: http://skia.googlecode.com/svn/trunk@9318 2bbb7eff-a529-9590-31e7-b0007b416f81
* Reland 'Add path utils, plus a test for it.'Gravatar scroggo@google.com2013-05-28
| | | | | | | | | | | | | | | Build SkPathJoin and SkBasename on windows also. Previous CL did not build on Windows because the two functions were accidentally placed inside an ifdef that did not include windows. Move the functions to the top of the file, and add a comment by the endif for clarity. Previously reviewed at https://codereview.chromium.org/15747004/ Review URL: https://codereview.chromium.org/15740024 git-svn-id: http://skia.googlecode.com/svn/trunk@9295 2bbb7eff-a529-9590-31e7-b0007b416f81
* Revert "Add path utils, plus a test for it."Gravatar scroggo@google.com2013-05-24
| | | | | | | | | | This reverts commit d172374b46cae0bd5d7c024b9848f5bdafcc6a16. This CL broke on Windows. Review URL: https://codereview.chromium.org/15986004 git-svn-id: http://skia.googlecode.com/svn/trunk@9278 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add path utils, plus a test for it.Gravatar scroggo@google.com2013-05-24
| | | | | | | | | | | | | | | | | | | | | | | | SkOSFile: Added class SkOSPath with functions for modifying strings representing path names. OSPathTest.cpp: Test of the new utilities. factory.cpp: Use SkPathJoin. gmmain and gm_expectations: Use SkOSPath::SkPathJoin instead of a local version. skimage_main.cpp: Use the new location of SkPathJoin and SkBasename. R=epoger@google.com Review URL: https://codereview.chromium.org/15747004 git-svn-id: http://skia.googlecode.com/svn/trunk@9277 2bbb7eff-a529-9590-31e7-b0007b416f81
* GM: create GmResultDigest that encapsulates digest type ("bitmap-64bitMD5") ↵Gravatar epoger@google.com2013-05-24
| | | | | | | | | | and value (12345) R=scroggo@google.com Review URL: https://codereview.chromium.org/15883004 git-svn-id: http://skia.googlecode.com/svn/trunk@9271 2bbb7eff-a529-9590-31e7-b0007b416f81
* expand SkLua to handle creation of its own StateGravatar reed@google.com2013-05-22
| | | | | | | | | | | add lua sample BUG= R=robertphillips@google.com Review URL: https://codereview.chromium.org/15742009 git-svn-id: http://skia.googlecode.com/svn/trunk@9247 2bbb7eff-a529-9590-31e7-b0007b416f81
* add dumpops.lua as a sample scraper that just dumps the argumentsGravatar reed@google.com2013-05-22
| | | | | | | | | | | add SkLua.h for common utilities BUG= R=rmistry@google.com Review URL: https://codereview.chromium.org/15737010 git-svn-id: http://skia.googlecode.com/svn/trunk@9242 2bbb7eff-a529-9590-31e7-b0007b416f81
* move all flag bits up by 1, to accomodate new flag in SkPaintGravatar reed@google.com2013-05-22
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/15739006 git-svn-id: http://skia.googlecode.com/svn/trunk@9237 2bbb7eff-a529-9590-31e7-b0007b416f81
* New API for encoding bitmaps during serialization.Gravatar scroggo@google.com2013-05-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change gives more flexibility to the implementation of EncodeBitmap to prefer calling refEncodedData, doing its own encode, or even doing both and making a decision about which to use. The new function signature also allows the implementation to tell the ordered write buffer whether to store the pixel offset, in the case where the encoded bitmap represents the larger bitmap, or to ignore the pixel offset, in the case where the implementation only encoded the subset that is used. Requires changes to chromium to use the new function signature. (https://codereview.chromium.org/15496006/) SkPicture: New API for EncodeBitmap. SkOrderedReadBuffer: Ifdef'd out addition of reading the offset. SkOrderedWriteBuffer: Never call refEncodedData. Allow the user to call that from their EncodeBitmap function, if desired. This addresses https://code.google.com/p/skia/issues/detail?id=1239 Add in ifdef'd out code to record the offset. PictureTest and PictureRenderer: Implement the new definition of EncodeBitmap. Also update the name of the function to meet coding style guidelines. BUG=https://code.google.com/p/skia/issues/detail?id=1239 R=reed@google.com Review URL: https://codereview.chromium.org/15489004 git-svn-id: http://skia.googlecode.com/svn/trunk@9226 2bbb7eff-a529-9590-31e7-b0007b416f81
* add SkPath as real lua objectGravatar reed@google.com2013-05-21
| | | | | | | | R=robertphillips@google.com Review URL: https://codereview.chromium.org/15563013 git-svn-id: http://skia.googlecode.com/svn/trunk@9221 2bbb7eff-a529-9590-31e7-b0007b416f81
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-04 02:40:10 +0000