aboutsummaryrefslogtreecommitdiffhomepage
path: root/tools
Commit message (Collapse)AuthorAge
...
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2013-03-05
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7976 2bbb7eff-a529-9590-31e7-b0007b416f81
* Provide an option in bench_pictures to count pixel ram.Gravatar scroggo@google.com2013-03-04
| | | | | | | | | | | | | | In SkLruImageCache, provide an option to keep all pixels, and document the new behavior. In gm/factory.cpp, set the budget for the Lru image cache to 1, to retain (basically) the old behavior. BUG=https://code.google.com/p/skia/issues/detail?id=1010 Review URL: https://codereview.chromium.org/12378075 git-svn-id: http://skia.googlecode.com/svn/trunk@7972 2bbb7eff-a529-9590-31e7-b0007b416f81
* Convert skhello tool to SkFlags API.Gravatar commit-bot@chromium.org2013-03-04
| | | | | | | | | | | | BUG=https://code.google.com/p/skia/issues/detail?id=1094 R=scroggo@google.com Signed-off-by: Thiago Farina <tfarina@chromium.org> Review URL: https://chromiumcodereview.appspot.com/12381087 git-svn-id: http://skia.googlecode.com/svn/trunk@7968 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add a way to monitor cache hits and misses for deferred decoding.Gravatar scroggo@google.com2013-03-04
| | | | | | | | | Adds a new flag to bench_pictures in order to do this. Also fix a warning. Review URL: https://codereview.chromium.org/12393046 git-svn-id: http://skia.googlecode.com/svn/trunk@7965 2bbb7eff-a529-9590-31e7-b0007b416f81
* Created my own flag parser, based off of gflags.Gravatar scroggo@google.com2013-03-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Share common code between bench_ and render_ to set up the PictureRenderer. Fix an include error in SkPictureRenderer.h. Simplified parameter passing in render_pictures_main. Switch to using an SkAutoTUnref for the PictureRenderer. I also changed the input format somewhat, so the buildbots need to be updated as well: https://codereview.appspot.com/7441044/ Fixed a bug in PictureBenchmark where calling setTimeIndividualTiles(false) sets the member variable to true. Removed setDeviceType from PictureBenchmark, since only the PictureRenderer needs to know which device type to use. Some changes to the input format: '--logPerIter' no longer takes a 1 or 0. Instead, '--logPerIter' turns it on and '--nologPerIter' turns it off (with off as the default). (Note that this is for bench_pictures; bench still uses the old format) Change '--device' to '--config' and 'bitmap' to '8888' to be the same as gm. Requires '--r' before inputs (to match gm), though there can be multiple inputs following it. Changed --enable-deferred-image-decoding (which no one uses but me yet anyway) to --deferImageDecoding, since the former is incompatible with the flag parser. Changes to behavior: Show a short error message on failure (rather than the explanation of all flags). BUG=https://code.google.com/p/skia/issues/detail?id=1094 Review URL: https://codereview.appspot.com/7230053 git-svn-id: http://skia.googlecode.com/svn/trunk@7961 2bbb7eff-a529-9590-31e7-b0007b416f81
* Move initialization to PictureRenderer ctorGravatar robertphillips@google.com2013-03-02
| | | | | | | | https://codereview.appspot.com/7435052/ git-svn-id: http://skia.googlecode.com/svn/trunk@7942 2bbb7eff-a529-9590-31e7-b0007b416f81
* Revert 7932Gravatar robertphillips@google.com2013-03-01
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7938 2bbb7eff-a529-9590-31e7-b0007b416f81
* Initialize grid info in TiledPictureRenderer::initGravatar robertphillips@google.com2013-03-01
| | | | | | | | https://codereview.appspot.com/7444053/ git-svn-id: http://skia.googlecode.com/svn/trunk@7932 2bbb7eff-a529-9590-31e7-b0007b416f81
* Split GrGLContextInfo into GrGLContext & GrGLContextInfoGravatar robertphillips@google.com2013-02-28
| | | | | | | | https://codereview.appspot.com/7436045/ git-svn-id: http://skia.googlecode.com/svn/trunk@7905 2bbb7eff-a529-9590-31e7-b0007b416f81
* Change SkTileGride geometry calculations to match the Chromium compositor.Gravatar junov@chromium.org2013-02-27
| | | | | | | | | | | | | | This patch changes the semantics of tileWidth/Height to include the border region, and uses an offset to take into account the fact that there is no outer border for outer tiles. This patch also fixes a previous bug where the right column and bottom row were considered to be included in bounds that are expressed as an SkIRect. Companion Chromium CL required for roll: https://codereview.chromium.org/12221077/ TEST=TileGrid unit test Review URL: https://codereview.appspot.com/7350050 git-svn-id: http://skia.googlecode.com/svn/trunk@7885 2bbb7eff-a529-9590-31e7-b0007b416f81
* Create SkLazyPixelRef which performs lazy decoding.Gravatar scroggo@google.com2013-02-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new pixel ref behaves similarly to SkImageRef, with some key differences: It does not depend on the images project. It requires an SkImageCache, which handles allocation and caching of the pixel memory. It takes a function signature for decoding which decodes into already allocated pixel memory rather than into an SkBitmap. Add two implementations of SkImageCache: SkLruImageCache and SkAshmemImageCache. Replace SkSerializationHelpers::DecodeBitmap with SkPicture::InstallPixelRefProc, and update sites that referenced it. SkBitmapFactory now sets the pixel ref to a new object of the new class SkLazyPixelRef, provided it has an SkImageCache for caching. Provide an option to do lazy decodes in render_pictures and bench_pictures. SkPicture: Eliminate the default parameters in the constructor. If a proc for decoding bitmaps is installed, use it to decode any encoded data in subpictures. When parsing deserializing subpictures, check for success. When serializing subpictures, pass the picture's bitmap encoder to the subpicture's call to serialize. Update BitmapFactoryTest to test its new behavior. BUG=https://code.google.com/p/skia/issues/detail?id=1008 BUG=https://code.google.com/p/skia/issues/detail?id=1009 Review URL: https://codereview.appspot.com/7060052 git-svn-id: http://skia.googlecode.com/svn/trunk@7835 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make ANGLE bot run only ANGLE configs in bench_picturesGravatar borenet@google.com2013-02-20
| | | | | | Review URL: https://codereview.appspot.com/7397043 git-svn-id: http://skia.googlecode.com/svn/trunk@7799 2bbb7eff-a529-9590-31e7-b0007b416f81
* bench_graph_svg: add r7618 results (NOT per-tile) to self-test source dataGravatar epoger@google.com2013-02-17
| | | | | | Review URL: https://codereview.appspot.com/7305098 git-svn-id: http://skia.googlecode.com/svn/trunk@7760 2bbb7eff-a529-9590-31e7-b0007b416f81
* bench_graph_svg self-test: add non-tile bench dataGravatar epoger@google.com2013-02-15
| | | | | | | TBR=benchen Review URL: https://codereview.appspot.com/7317048 git-svn-id: http://skia.googlecode.com/svn/trunk@7750 2bbb7eff-a529-9590-31e7-b0007b416f81
* tools/tests/run.sh: download test bench data from Google Storage rather than SVNGravatar epoger@google.com2013-02-15
| | | | | | Review URL: https://codereview.appspot.com/7343045 git-svn-id: http://skia.googlecode.com/svn/trunk@7749 2bbb7eff-a529-9590-31e7-b0007b416f81
* bench_graph_svg: generate output in consistently sorted order, to make diffs ↵Gravatar epoger@google.com2013-02-14
| | | | | | | | | more useful TBR=benchen Review URL: https://codereview.appspot.com/7330044 git-svn-id: http://skia.googlecode.com/svn/trunk@7744 2bbb7eff-a529-9590-31e7-b0007b416f81
* disable new bench_graph_svg test in tools/tests/run.sh that fails on housekeeperGravatar epoger@google.com2013-02-14
| | | | | | | TBR=bensong Review URL: https://codereview.appspot.com/7303092 git-svn-id: http://skia.googlecode.com/svn/trunk@7739 2bbb7eff-a529-9590-31e7-b0007b416f81
* tools/tests/run.sh: add self-test for bench_graph_svg.pyGravatar epoger@google.com2013-02-14
| | | | | | Review URL: https://codereview.appspot.com/7319044 git-svn-id: http://skia.googlecode.com/svn/trunk@7735 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2013-02-14
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7730 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix filter optimizations to take the paint's alpha into accountGravatar robertphillips@google.com2013-02-13
| | | | | | | | https://codereview.appspot.com/7312083/ git-svn-id: http://skia.googlecode.com/svn/trunk@7711 2bbb7eff-a529-9590-31e7-b0007b416f81
* Allow debugger to be compiled without GaneshGravatar robertphillips@google.com2013-02-13
| | | | | | | | https://codereview.appspot.com/7311084/ git-svn-id: http://skia.googlecode.com/svn/trunk@7710 2bbb7eff-a529-9590-31e7-b0007b416f81
* Turn off per-iteration tile time reportingGravatar borenet@google.com2013-02-11
| | | | | | | It's overloading the bench graph generator. Review URL: https://codereview.appspot.com/7300088 git-svn-id: http://skia.googlecode.com/svn/trunk@7691 2bbb7eff-a529-9590-31e7-b0007b416f81
* Argv[0] isn't the first argument to anything.Gravatar reed@google.com2013-02-08
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7687 2bbb7eff-a529-9590-31e7-b0007b416f81
* match chromium's approach for locating the most appropriate macos SDK to useGravatar humper@google.com2013-02-08
| | | | | | | | | | By doing this, the ninja generator for gyp works great on macos, resulting in faster (I think) and much, much terser builds. BUG= Review URL: https://codereview.appspot.com/7142047 git-svn-id: http://skia.googlecode.com/svn/trunk@7684 2bbb7eff-a529-9590-31e7-b0007b416f81
* Clean up some out of order initialization warnings.Gravatar scroggo@google.com2013-02-08
| | | | | | Review URL: https://codereview.appspot.com/7321043 git-svn-id: http://skia.googlecode.com/svn/trunk@7681 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add support for using ANGLE in bench_pictures.Gravatar scroggo@google.com2013-02-08
| | | | | | | | | | | | | BUG=https://code.google.com/p/skia/issues/detail?id=1012 Other cleanups: Remove setDeviceType from PictureBenchmark, since it is unnecessary. Dereference PictureRenderer::fGrContext when done with it. Make PictureRenderer::fGrContext and PictureRenderer::fGrContextFactory private, since they are not used by subclasses. Review URL: https://codereview.appspot.com/7314063 git-svn-id: http://skia.googlecode.com/svn/trunk@7677 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2013-02-08
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7659 2bbb7eff-a529-9590-31e7-b0007b416f81
* Only turn on one gpu timer in PictureBenchmark toolGravatar robertphillips@google.com2013-02-07
| | | | | | | | https://codereview.appspot.com/7310060/ git-svn-id: http://skia.googlecode.com/svn/trunk@7651 2bbb7eff-a529-9590-31e7-b0007b416f81
* Enhance GL error checking for non-Ganesh GL callsGravatar robertphillips@google.com2013-02-07
| | | | | | | | https://codereview.appspot.com/7312057/ git-svn-id: http://skia.googlecode.com/svn/trunk@7647 2bbb7eff-a529-9590-31e7-b0007b416f81
* Temporary patch to BenchPictures gpu timing problemGravatar robertphillips@google.com2013-02-07
| | | | | | | | https://codereview.appspot.com/7311059/ git-svn-id: http://skia.googlecode.com/svn/trunk@7641 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fixed warning/error in PictureRenderer.cppGravatar robertphillips@google.com2013-02-07
| | | | | | | | TBR by scroggo@google.com git-svn-id: http://skia.googlecode.com/svn/trunk@7634 2bbb7eff-a529-9590-31e7-b0007b416f81
* In picture rendering tools, draw the entire scaled image.Gravatar scroggo@google.com2013-02-06
| | | | | | | | | | | If --scale is used, limit the size of the image drawn to the scaled picture size, rather than the original picture size. BUG=https://code.google.com/p/skia/issues/detail?id=1095 Review URL: https://codereview.appspot.com/7307054 git-svn-id: http://skia.googlecode.com/svn/trunk@7633 2bbb7eff-a529-9590-31e7-b0007b416f81
* Turn --timeIndividualTiles back on in bench_picturesGravatar borenet@google.com2013-02-06
| | | | | | Review URL: https://codereview.appspot.com/7299049 git-svn-id: http://skia.googlecode.com/svn/trunk@7620 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2013-02-06
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7608 2bbb7eff-a529-9590-31e7-b0007b416f81
* Improved compartmentalization of filter tool's optimizations & add new ↵Gravatar robertphillips@google.com2013-02-05
| | | | | | | | | | optimization https://codereview.appspot.com/7294049/ git-svn-id: http://skia.googlecode.com/svn/trunk@7601 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix master IP in submit_tryGravatar borenet@google.com2013-02-05
| | | | | | | | | Unreviewed. Should have this reference: http://skia.googlecode.com/svn/buildbot/site_config/global_variables.json (SkipBuildbotRuns) Review URL: https://codereview.appspot.com/7314045 git-svn-id: http://skia.googlecode.com/svn/trunk@7598 2bbb7eff-a529-9590-31e7-b0007b416f81
* Change comment to avoid warning (don't end line comment with '\').Gravatar bungeman@google.com2013-02-04
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7542 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2013-01-31
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7499 2bbb7eff-a529-9590-31e7-b0007b416f81
* Update filter tool to allow more flexible filteringGravatar robertphillips@google.com2013-01-31
| | | | | | | | https://codereview.appspot.com/7227055/ git-svn-id: http://skia.googlecode.com/svn/trunk@7492 2bbb7eff-a529-9590-31e7-b0007b416f81
* Change the method for timing individual tiles in bench_pictures.Gravatar scroggo@google.com2013-01-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When timing individual tiles in bench_pictures, keep a timer running across all repeats, and then take the average. The former method of timing each iteration separately runs into precision errors on some platforms. Running on my Mac Pro with OSX 10.8, the cmsecs for the new method and the old method are roughly the same when checking the CPU time. When checking the wall time, the old method often gives me 0ms, while the new method gives me a larger value. I don't think this can be entirely attributed to rounding though, since on occasion I see the old method showing a short time period (.05 - .15ms) while the new method shows .15ms higher (which is in range for the difference I'm seeing for other tiles where the old method reports 0ms). Some other changes: PictureRenderer::resetState now takes a boolean parameter. If called with false, it will only do a flush, while if called with true, it will also call finish. resetState is now called with true everywhere except in between iterations of drawing the same tile (when timing individual tiles). render_pictures_main no longer calls resetState directly, since it already calls end, which calls resetState. BUG=http://code.google.com/p/skia/issues/detail?id=1066 Review URL: https://codereview.appspot.com/7101060 git-svn-id: http://skia.googlecode.com/svn/trunk@7424 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2013-01-26
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7406 2bbb7eff-a529-9590-31e7-b0007b416f81
* Manually parse arguments in submit_tryGravatar borenet@google.com2013-01-25
| | | | | | | Argparse not supported with Python < 2.7. Review URL: https://codereview.appspot.com/7206054 git-svn-id: http://skia.googlecode.com/svn/trunk@7403 2bbb7eff-a529-9590-31e7-b0007b416f81
* Leave one and only one newline at the end of source filesGravatar rmistry@google.com2013-01-25
| | | | | | Review URL: https://codereview.appspot.com/7216043 git-svn-id: http://skia.googlecode.com/svn/trunk@7398 2bbb7eff-a529-9590-31e7-b0007b416f81
* Modify submit_try to work on windowsGravatar borenet@google.com2013-01-24
| | | | | | Review URL: https://codereview.appspot.com/7199053 git-svn-id: http://skia.googlecode.com/svn/trunk@7379 2bbb7eff-a529-9590-31e7-b0007b416f81
* Replace gcl_try with submit_tryGravatar borenet@google.com2013-01-23
| | | | | | | | submit_try supports both git and svn and hooks directly into depot_tools rather than calling it in a subprocess. Review URL: https://codereview.appspot.com/7202044 git-svn-id: http://skia.googlecode.com/svn/trunk@7352 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add wrapper for "gcl try"Gravatar borenet@google.com2013-01-22
| | | | | | | Adds some validation for --bot Review URL: https://codereview.appspot.com/7179048 git-svn-id: http://skia.googlecode.com/svn/trunk@7315 2bbb7eff-a529-9590-31e7-b0007b416f81
* Patch for r7304. Add string in bench_pictures_main's gFilterTypes for oval ↵Gravatar jvanverth@google.com2013-01-22
| | | | | | filter type. git-svn-id: http://skia.googlecode.com/svn/trunk@7307 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2013-01-17
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7240 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fixing expression logic from r7237/r7234Gravatar junov@chromium.org2013-01-16
| | | | | | TBR=borenet git-svn-id: http://skia.googlecode.com/svn/trunk@7238 2bbb7eff-a529-9590-31e7-b0007b416f81
* Compiler warning fixes for r7235Gravatar robertphillips@google.com2013-01-16
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7237 2bbb7eff-a529-9590-31e7-b0007b416f81