aboutsummaryrefslogtreecommitdiffhomepage
path: root/gm/gmmain.cpp
Commit message (Collapse)AuthorAge
* Don't use writeData to write the document in GM.Gravatar bungeman@google.com2013-05-06
| | | | | | | https://codereview.chromium.org/14824003/ git-svn-id: http://skia.googlecode.com/svn/trunk@9013 2bbb7eff-a529-9590-31e7-b0007b416f81
* GM: fix --mismatchPath optionGravatar epoger@google.com2013-05-02
| | | | | | | | | | (currently, it doesn't actually write any files!) TBR=rmistry Review URL: https://codereview.chromium.org/14855005 git-svn-id: http://skia.googlecode.com/svn/trunk@8963 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fixes for piping bitmaps with encoded data.Gravatar scroggo@google.com2013-04-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Similar goals as https://codereview.chromium.org/14437012. Builds on patch set 1 from that issue (https://codereview.chromium.org/14437012/#ps1). Instead of the changes in patch set 2 from that issue, this changes SkOrderedWriteBuffer::writeBitmap to store whether an SkBitmapHeap was used when to store the index of the SkBitmap. SkOrderedReadBuffer::readBitmap now uses that information to distinguish between using the heap and unflattening. In addition, writeBitmap now records the width/height first in all cases. If now SkBitmapHeapReader is attached, but an SkBitmapHeap was used to record the bitmap, reading will fail and provide the same red SkBitmap as in the case where the SkBitmap was encoded but could not be decoded. Updates the PICTURE_VERSION as well. The key differences in this CL to look at are in: SkOrderedWriteBuffer, SkOrderedReadBuffer, and SkPicture. BUG= R=djsollen@google.com Review URL: https://codereview.chromium.org/14230022 git-svn-id: http://skia.googlecode.com/svn/trunk@8917 2bbb7eff-a529-9590-31e7-b0007b416f81
* GM: specify that currently used checksums are CityHashes of SkBitmapsGravatar epoger@google.com2013-04-26
| | | | | | | | | (distinct from other hash algorithms we may switch to, or hashes of files such as PDFs that cannot be read into an SkBitmap) Review URL: https://codereview.chromium.org/14284018 git-svn-id: http://skia.googlecode.com/svn/trunk@8877 2bbb7eff-a529-9590-31e7-b0007b416f81
* Treat default command line argument properly.Gravatar scroggo@google.com2013-04-24
| | | | | | | | | | | | | | | | | | | In SkCommandLineFlags, if the client sets a default value of multiple arguments (e.g. "arg0 arg1 ..."), set the actual defaults to all of those arguments separately (i.e. an array with [0] == "arg0", [1] == "arg1", ...), rather than as one string (i.e. [0] == "arg0 arg1 ..."). Remove the hack that worked around this bug. Also move the increasingly complicated implementation of SkFlagInfo::CreateStringFlag into the cpp file. BUG=https://code.google.com/p/skia/issues/detail?id=1237 Review URL: https://codereview.chromium.org/14366034 git-svn-id: http://skia.googlecode.com/svn/trunk@8845 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add option to specify defaults configs to --config flag in gm. This makes it ↵Gravatar bsalomon@google.com2013-04-24
| | | | | | | | easier to run gm defaults configs with a delta. Also make ~ exclude a config. Review URL: https://codereview.chromium.org/13842014 git-svn-id: http://skia.googlecode.com/svn/trunk@8842 2bbb7eff-a529-9590-31e7-b0007b416f81
* Encode images with DCTDecode (JPEG) in PDFs if it makes sense. Fallback to ↵Gravatar edisonn@google.com2013-04-24
| | | | | | | | | FlateDecode (zip) if it makes sense. Otherewise include uncompressed stream. This change will reduce the size of PDFs to 50% (in the case of the existing SKPs, we reduce the total size of PDFs from 105MB to 50MB) Review URL: https://codereview.appspot.com/7068055 git-svn-id: http://skia.googlecode.com/svn/trunk@8835 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix an SkCommandLineFlags bug.Gravatar scroggo@google.com2013-04-23
| | | | | | | | | | | | | | | | | Previously I was storing an SkTDArray of const char*, which fails if the strings go out of scope. Instead, store an SkTArray of SkString, and copy the strings, so we do not depend on the strings sticking around. Using an SkTArray because it is smart enough to call the destructors, so the copies can be destroyed on program exit. BUG=https://code.google.com/p/skia/issues/detail?id=1237 Review URL: https://codereview.chromium.org/14414008 git-svn-id: http://skia.googlecode.com/svn/trunk@8829 2bbb7eff-a529-9590-31e7-b0007b416f81
* Delete gm after use.Gravatar scroggo@google.com2013-04-16
| | | | | | | | | | Fixes a memory leak when using the --modulo flag. Instead of deleting manually in each case we are done with it (which led to missing one case), use an SkAutoTDelete. Review URL: https://codereview.chromium.org/13929019 git-svn-id: http://skia.googlecode.com/svn/trunk@8707 2bbb7eff-a529-9590-31e7-b0007b416f81
* GM: allow caller to specify which result types trigger an errorGravatar epoger@google.com2013-04-12
| | | | | | Review URL: https://codereview.chromium.org/14187007 git-svn-id: http://skia.googlecode.com/svn/trunk@8652 2bbb7eff-a529-9590-31e7-b0007b416f81
* rename SkBitmapChecksummer as SkBitmapHasher, and prepare for it to possibly useGravatar epoger@google.com2013-04-12
| | | | | | | some algorithm other than CityHash Review URL: https://codereview.chromium.org/14170010 git-svn-id: http://skia.googlecode.com/svn/trunk@8639 2bbb7eff-a529-9590-31e7-b0007b416f81
* GM: print list of GMs run, even in non-verbose mode (rolls back part of r8600)Gravatar epoger@google.com2013-04-12
| | | | | | Review URL: https://codereview.chromium.org/14052003 git-svn-id: http://skia.googlecode.com/svn/trunk@8638 2bbb7eff-a529-9590-31e7-b0007b416f81
* GM: signal error if the total number of tests run was less than expectedGravatar epoger@google.com2013-04-11
| | | | | | Review URL: https://codereview.chromium.org/13863009 git-svn-id: http://skia.googlecode.com/svn/trunk@8614 2bbb7eff-a529-9590-31e7-b0007b416f81
* GM: remove enableMissingWarning in favor of verbose flag, and trim output in ↵Gravatar epoger@google.com2013-04-10
| | | | | | | | nonverbose mode Review URL: https://codereview.chromium.org/14080003 git-svn-id: http://skia.googlecode.com/svn/trunk@8600 2bbb7eff-a529-9590-31e7-b0007b416f81
* Run mixedxfermodes GM through the pipe.Gravatar scroggo@google.com2013-04-10
| | | | | | | | | | | | Pass the size of the GM to SkGPipeWriter.startRecording(), so that calls to getDeviceSize() on the returned SkCanvas return a reasonable result. This allows removing the flag to skip mixedxfermodes. BUG=https://code.google.com/p/skia/issues/detail?id=1199 Review URL: https://codereview.chromium.org/13814023 git-svn-id: http://skia.googlecode.com/svn/trunk@8598 2bbb7eff-a529-9590-31e7-b0007b416f81
* gm: hide list of tests with ignored errors, by defaultGravatar epoger@google.com2013-04-10
| | | | | | Review URL: https://codereview.chromium.org/13811031 git-svn-id: http://skia.googlecode.com/svn/trunk@8591 2bbb7eff-a529-9590-31e7-b0007b416f81
* gm: record IntentionallySkipped testsGravatar epoger@google.com2013-04-10
| | | | | | Review URL: https://codereview.chromium.org/13663003 git-svn-id: http://skia.googlecode.com/svn/trunk@8586 2bbb7eff-a529-9590-31e7-b0007b416f81
* SkFlags now follows proper dashing convention.Gravatar scroggo@google.com2013-04-09
| | | | | | | | | | | | | | | | | | | | | | | | | Two dashes are used for flags with multiple characters, and one dash is used for flags with single characters. In GM, changed '-wp' to '-p' (the command to choose a directory for writing SKPs) to fit with the convention. In render_pictures and bench_pictures, changed the flag for read and write path to have full names (which are consistent) and use the old single character names as their shortcuts. SkCommandLineFlags: Updated the documentation, and only allow -h or --help for help (again, to match the convention). Also enforce the single character limit for the short name, and require the full name to be at least two characters. Provide full names for skhello. BUG=https://code.google.com/p/skia/issues/detail?id=1174 Review URL: https://codereview.chromium.org/12521019 git-svn-id: http://skia.googlecode.com/svn/trunk@8582 2bbb7eff-a529-9590-31e7-b0007b416f81
* re-land r8535, now that src.chromium.org is back upGravatar epoger@google.com2013-04-04
| | | | | | Review URL: https://codereview.chromium.org/13662002 git-svn-id: http://skia.googlecode.com/svn/trunk@8538 2bbb7eff-a529-9590-31e7-b0007b416f81
* Revert r8535Gravatar epoger@google.com2013-04-04
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@8536 2bbb7eff-a529-9590-31e7-b0007b416f81
* gm: report ExpectationsMismatch and RenderModeMismatch results separatelyGravatar epoger@google.com2013-04-04
| | | | | | Review URL: https://codereview.chromium.org/13650002 git-svn-id: http://skia.googlecode.com/svn/trunk@8535 2bbb7eff-a529-9590-31e7-b0007b416f81
* re-land r8514 without breaking 32-bit Mac compilersGravatar epoger@google.com2013-04-03
| | | | | | Review URL: https://codereview.chromium.org/13427004 git-svn-id: http://skia.googlecode.com/svn/trunk@8518 2bbb7eff-a529-9590-31e7-b0007b416f81
* roll out r8514Gravatar epoger@google.com2013-04-03
| | | | | | Review URL: https://codereview.chromium.org/13503003 git-svn-id: http://skia.googlecode.com/svn/trunk@8515 2bbb7eff-a529-9590-31e7-b0007b416f81
* gm: display summary of all results by ErrorTypeGravatar epoger@google.com2013-04-03
| | | | | | | as discussed in https://codereview.chromium.org/12825005/#msg14 Review URL: https://codereview.chromium.org/12851022 git-svn-id: http://skia.googlecode.com/svn/trunk@8514 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2013-04-03
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@8503 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add tile rendering to GMGravatar robertphillips@google.com2013-04-02
| | | | | | | | https://codereview.chromium.org/13165011/ git-svn-id: http://skia.googlecode.com/svn/trunk@8485 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make GM render to render targets that are also textures.Gravatar bsalomon@google.com2013-03-28
| | | | | | Review URL: https://codereview.chromium.org/13211002 git-svn-id: http://skia.googlecode.com/svn/trunk@8438 2bbb7eff-a529-9590-31e7-b0007b416f81
* gm: change ErrorBitfield to ErrorType/ErrorCombinationGravatar epoger@google.com2013-03-22
| | | | | | | This will enable future GM changes to report errors more completely/consistently, like skdiff. Review URL: https://codereview.chromium.org/12992003 git-svn-id: http://skia.googlecode.com/svn/trunk@8344 2bbb7eff-a529-9590-31e7-b0007b416f81
* Change the name of SkFlags to SkCommandLineFlags.Gravatar scroggo@google.com2013-03-21
| | | | | | | | | | | | | | | | | This name is more specific to what it actually does. Also move the code into tools/flags, to (slightly) better organize the massive tools folder. Update the programs that use it to use the new names. No functionality changes. BUG=https://code.google.com/p/skia/issues/detail?id=1173 Review URL: https://codereview.chromium.org/12440067 git-svn-id: http://skia.googlecode.com/svn/trunk@8304 2bbb7eff-a529-9590-31e7-b0007b416f81
* Re-land r8269, fixing build errors/warningsGravatar epoger@google.com2013-03-21
| | | | | | Review URL: https://codereview.chromium.org/12886037 git-svn-id: http://skia.googlecode.com/svn/trunk@8290 2bbb7eff-a529-9590-31e7-b0007b416f81
* Revert r8269 to fix BuildGM. Warnings-as-errors problem, dunno why I didn't ↵Gravatar epoger@google.com2013-03-20
| | | | | | see it locally. git-svn-id: http://skia.googlecode.com/svn/trunk@8271 2bbb7eff-a529-9590-31e7-b0007b416f81
* gm refactor: extract run_multiple_configs() and run_multiple_modes()Gravatar epoger@google.com2013-03-20
| | | | | | Review URL: https://codereview.chromium.org/12536007 git-svn-id: http://skia.googlecode.com/svn/trunk@8269 2bbb7eff-a529-9590-31e7-b0007b416f81
* Write the config options in place in the help string.Gravatar scroggo@google.com2013-03-20
| | | | | | | | | | Fixes for gm. Builds on https://codereview.chromium.org/12955002/ Review URL: https://codereview.chromium.org/12663021 git-svn-id: http://skia.googlecode.com/svn/trunk@8256 2bbb7eff-a529-9590-31e7-b0007b416f81
* Reland r8235 "Switch gm to use SkFlags."Gravatar scroggo@google.com2013-03-20
| | | | | | | | | | | | This time, using #if guards for gpuCacheSize consistently. Also fix some warnings. BUG=https://code.google.com/p/skia/issues/detail?id=1094 Review URL: https://codereview.chromium.org/12440052 git-svn-id: http://skia.googlecode.com/svn/trunk@8254 2bbb7eff-a529-9590-31e7-b0007b416f81
* Reland MSAA changes to GM now that Linux/ATI driver bug has been worked around.Gravatar bsalomon@google.com2013-03-19
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@8240 2bbb7eff-a529-9590-31e7-b0007b416f81
* Revert "Make gm use SkFlags."Gravatar scroggo@google.com2013-03-19
| | | | | | | | | | Fix the build. This reverts commit 6f1fe2aee39308fc56854b062be244a904c209e7. Review URL: https://codereview.chromium.org/12709005 git-svn-id: http://skia.googlecode.com/svn/trunk@8236 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make gm use SkFlags.Gravatar scroggo@google.com2013-03-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make flag parser its own project. It is still in the tools folder, but can now be used by other projects. Some changes to behavior in order to work with SkFlags: enable-missing-warning and disable-missing-warning have become enableMissingWarning and noenableMissingWarning. exclude-config is now excludeConfig --config now can only be listed once, but all configs listed after it will be included (same with excludeConfig and match). In addition, writeJsonSummary has been changed to writeJsonSummaryPath for consistency with other path flags. Provide an option to SkFlags to provide a short name, and use it for the flags which have short names. --tileGridReplayScales now takes space separated arguments, like other inputs. BUG=https://code.google.com/p/skia/issues/detail?id=1094 Review URL: https://codereview.chromium.org/12632015 git-svn-id: http://skia.googlecode.com/svn/trunk@8235 2bbb7eff-a529-9590-31e7-b0007b416f81
* Revert changes to GM MSAA configs, hopefully fixes linux bot gm failures.Gravatar bsalomon@google.com2013-03-19
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@8232 2bbb7eff-a529-9590-31e7-b0007b416f81
* fix angle gmmain buildGravatar bsalomon@google.com2013-03-19
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@8226 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix build break in gmmain.cppGravatar bsalomon@google.com2013-03-19
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@8222 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add config msaa4 to gm (not run by default) and make MSAA configs not run unlessGravatar bsalomon@google.com2013-03-19
| | | | | | | sample count is <= the max. Review URL: https://codereview.chromium.org/12554012 git-svn-id: http://skia.googlecode.com/svn/trunk@8221 2bbb7eff-a529-9590-31e7-b0007b416f81
* In gm, install drawFilter for pipe and tiledPipe.Gravatar scroggo@google.com2013-03-15
| | | | | | | | Fixes bugs in drawing un-antialiased text in gm. Review URL: https://codereview.chromium.org/12812016 git-svn-id: http://skia.googlecode.com/svn/trunk@8173 2bbb7eff-a529-9590-31e7-b0007b416f81
* gm: write all messages to stdout/stderr with "GM:" preamble to distinguish ↵Gravatar epoger@google.com2013-03-13
| | | | | | | | from various debug messages coming from elsewhere Review URL: https://codereview.chromium.org/12691009 git-svn-id: http://skia.googlecode.com/svn/trunk@8126 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix sign/unsigned warnings in benchmain.cpp and gmmain.cppGravatar bsalomon@google.com2013-03-11
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@8079 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make GPU cache size cmd line configurable in GM.Gravatar bsalomon@google.com2013-03-11
| | | | | | Review URL: https://codereview.chromium.org/12717002 git-svn-id: http://skia.googlecode.com/svn/trunk@8076 2bbb7eff-a529-9590-31e7-b0007b416f81
* add flag to skip gpu testGravatar reed@google.com2013-03-08
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@8036 2bbb7eff-a529-9590-31e7-b0007b416f81
* re-land r8028 without "special" Microsoft compiler warningsGravatar epoger@google.com2013-03-08
| | | | | | | TBR=borenet Review URL: https://codereview.chromium.org/12685002 git-svn-id: http://skia.googlecode.com/svn/trunk@8035 2bbb7eff-a529-9590-31e7-b0007b416f81
* Revert r8028... warnings-as-errors broke some buildersGravatar epoger@google.com2013-03-07
| | | | | | (I'm developing on Linux, which does not treat warnings as errors.) git-svn-id: http://skia.googlecode.com/svn/trunk@8031 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fixing some warnings on LinuxGravatar sugoi@google.com2013-03-07
| | | | | | | There were only a few warnings left, so I fixed them and enabled the unused variable warning by removing the "-Wno-unused" flag. Only the -Wno-unused-parameter remains for now (could be removed later). Review URL: https://codereview.chromium.org/12480002 git-svn-id: http://skia.googlecode.com/svn/trunk@8030 2bbb7eff-a529-9590-31e7-b0007b416f81
* cleanup: fix gm's use of ErrorBitfield to be consistent (it's a bitfield, ↵Gravatar epoger@google.com2013-03-07
| | | | | | | | not an enum) Review URL: https://codereview.chromium.org/12640004 git-svn-id: http://skia.googlecode.com/svn/trunk@8028 2bbb7eff-a529-9590-31e7-b0007b416f81