aboutsummaryrefslogtreecommitdiffhomepage
path: root/tools
Commit message (Collapse)AuthorAge
* Increase timeouts of wowwiki and youtubeGravatar Ravi Mistry2014-10-29
|
* Moving weather pageset since it is crashingGravatar Ravi Mistry2014-10-29
|
* Moving wikipedia pageset since it is crashingGravatar Ravi Mistry2014-10-29
|
* Change extension of crashing page setGravatar Ravi Mistry2014-10-29
|
* Moving vnexpress pageset since it is crashingGravatar Ravi Mistry2014-10-29
|
* Increase wait times for androidpolice and nytimesGravatar Ravi Mistry2014-10-29
|
* pagesets are still crashing in RecreateSKPs bot, try adding waits to most ↵Gravatar rmistry2014-10-29
| | | | | | | | | | pagesets BUG=skia:3049 NOTRY=true TBR=borenet Review URL: https://codereview.chromium.org/682573004
* Add wait to amazon page set to avoid tab crashesGravatar rmistry2014-10-29
| | | | | | | | BUG=skia:3049 TBR=borenet NOTRY=true Review URL: https://codereview.chromium.org/686133002
* rename GrTextureDesc->GrSurfaceDesc, GrTextureFlags->GrSurfaceFlagsGravatar bsalomon2014-10-28
| | | | Review URL: https://codereview.chromium.org/682223002
* Updated all page sets to capture new set of webpage archives.Gravatar rmistry2014-10-28
| | | | | | | | | | | | | | | | | | | | | Fixed the record_wpr retry mechanism in webpages_playback.py. Updated all page sets to remove the no longer needed RunSmoothness and RunNavigateSteps. Updated the key_silk_cases pagesets to point to a non svn location: * skia_fontwipe_desktop.py * skia_silkfinance_desktop.py Removed mercurynews, record_wpr was timing out because of a popup. Added back the following pagesets which were previously disabled but are now working again: * skia_cnn_nexus10.py * skia_wordpress_desktop.py * skia_transformice_nexus10.py BUG=skia:3049 Review URL: https://codereview.chromium.org/684863002
* Revert of harden pathops for pathological test (patchset #19 id:410001 of ↵Gravatar hcm2014-10-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/633393002/) Reason for revert: Compile errors on bots Original issue's description: > These tests stress pathops by describing the union of circle-like paths that have tiny line segments embedded and double back to create near-coincident conditions. > > The fixes include > - detect when finding the active top loops between two possible answers > - preflight chasing winding to ensure answer is consistent > - binary search more often when quadratic intersection fails > - add more failure paths when an intersect is missed > > While this fixes the chrome bug, reenabling path ops in svg should be deferred until additional fixes are landed. > > TBR= > BUG=421132 > > Committed: https://skia.googlesource.com/skia/+/6f726addf3178b01949bb389ef83cf14a1d7b6b2 TBR=caryclark@google.com NOTREECHECKS=true NOTRY=true BUG=421132 Review URL: https://codereview.chromium.org/686843002
* These tests stress pathops by describing the union of circle-like paths that ↵Gravatar caryclark2014-10-28
| | | | | | | | | | | | | | | | | have tiny line segments embedded and double back to create near-coincident conditions. The fixes include - detect when finding the active top loops between two possible answers - preflight chasing winding to ensure answer is consistent - binary search more often when quadratic intersection fails - add more failure paths when an intersect is missed While this fixes the chrome bug, reenabling path ops in svg should be deferred until additional fixes are landed. TBR= BUG=421132 Review URL: https://codereview.chromium.org/633393002
* Fix ability to capture archives in webpages_playback.pyGravatar rmistry2014-10-27
| | | | | | | | | Updated all page sets to match the file name. This is to make it easier to call record_wpr with the page set class name. Also updated the wait time for baidu to 60 seconds. BUG=skia:3049 Review URL: https://codereview.chromium.org/683473002
* add gm: colorwheelGravatar halcanary2014-10-26
| | | | | | BUG=skia:3061 Review URL: https://codereview.chromium.org/680533002
* tools: Remove reference to svndiff.Gravatar Thiago Farina2014-10-24
| | | | | | | | | svndiff script was archived by commit https://skia.googlesource.com/skia/+/70242ace3b2ed06b33ee5ea5c9ac3aedea89b194 R=borenet@google.com Review URL: https://codereview.chromium.org/658923002
* Build gpu stats tracking in dev builds.Gravatar bsalomon2014-10-24
| | | | | | | | Fix cache stats, add more info. BUG=skia:2889 Review URL: https://codereview.chromium.org/655263005
* Add flag to skimage for decoding into unpremul.Gravatar scroggo2014-10-23
| | | | Review URL: https://codereview.chromium.org/666073006
* Add back page_sets/skia_googlespreadsheet_desktopGravatar rmistry2014-10-22
| | | | | | | | | Absolutely could not get the old page to work so I just created a new page. NOTRY=true BUG=skia:3008 Review URL: https://codereview.chromium.org/671883002
* Update webpages_playback script and all page_sets to use new ↵Gravatar rmistry2014-10-22
| | | | | | | | | | | | | | | | skpicture_printer benchmark The purpose of this CL is to turn the RecreateSKPs bot green after https://codereview.chromium.org/665193002/ lands. Description of changes in this CL: * After https://codereview.chromium.org/665193002/ lands we will no longer need to copy the archives and pagesets to telemetry's data directory, I removed the code that did this in https://codereview.chromium.org/655313003/ * page_sets now need unique class names. Updated all page_sets to have unique class names. * Using the new --page_set_name and --page_set_base_dir and --also-run-disabled-tests parameters BUG=skia:3046 Review URL: https://codereview.chromium.org/671513009
* Implement maxrss for WindowsGravatar mtklein2014-10-21
| | | | | | | | | | No more -1 in the first column on Windows! E.g. http://build.chromium.org/p/client.skia/builders/Perf-Win8-ShuttleA-HD7770-x86_64-Release-Trybot/builds/0/steps/nanobench/logs/stdio BUG=skia: Review URL: https://codereview.chromium.org/661413004
* Replace SkTypeface::Style with SkFontStyle.Gravatar bungeman2014-10-20
| | | | | | Committed: https://skia.googlesource.com/skia/+/43b8b36b20ae00e2d78421c4cda1f3f922983a20 Review URL: https://codereview.chromium.org/488143002
* Revert of Replace SkTypeface::Style with SkFontStyle. (patchset #9 id:160001 ↵Gravatar mtklein2014-10-20
| | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/488143002/) Reason for revert: CrOS GM failures: [*] 2 ExpectationsMismatch: fontmgr_iter_565.png fontmgr_iter_8888.png Original issue's description: > Replace SkTypeface::Style with SkFontStyle. > > Committed: https://skia.googlesource.com/skia/+/43b8b36b20ae00e2d78421c4cda1f3f922983a20 TBR=reed@google.com,bungeman@google.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/667023002
* Replace SkTypeface::Style with SkFontStyle.Gravatar bungeman2014-10-20
| | | | Review URL: https://codereview.chromium.org/488143002
* Fix clearing issue in render_picturesGravatar robertphillips2014-10-20
| | | | | | This bug only crops up when using the "--writeWholeImage" option in tile mode Review URL: https://codereview.chromium.org/664103003
* bug_chomper: Cleanup template initialization.Gravatar tfarina2014-10-17
| | | | | | | | | | | | Use the same tricks used by webtry and perf. Code seems more robust and easier to check for errors this way. BUG=None TEST=./run_server.sh, then navigate to 127.0.0.1:8000 and 127.0.0.1:8000/res R=borenet@google.com Review URL: https://codereview.chromium.org/661613004
* Fix RecreateSKPs bot (again)Gravatar borenet2014-10-17
| | | | | | | | | | It's killing its own process... BUG=skia:3008 TBR=rmistry NOTRY=true Review URL: https://codereview.chromium.org/661203002
* Fix recreate_skps?Gravatar borenet2014-10-17
| | | | | | | | | | | | run_measurement was removed in https://codereview.chromium.org/655563007. Based on the comments, I think the right thing to do is replace it with run_benchmark. Hopefully this fixes the bot. BUG=skia: TBR=rmistry NOTRY=true Review URL: https://codereview.chromium.org/658513004
* Add recreate_skps scriptGravatar borenet2014-10-17
| | | | | | BUG=skia:3008 Review URL: https://codereview.chromium.org/655313003
* render_pdfs cull rect output formated betterGravatar halcanary2014-10-16
| | | | Review URL: https://codereview.chromium.org/657353004
* Refactor SkGLContext to be actually extendableGravatar kkinnunen2014-10-15
| | | | | | | | | | | | | | | | | | | | | | Refactor SkGLContext to be actually extendable. Before, non-trivial subclass would need to destroy the GL connection upon running the destructor. However, the base class would run GL commands in its own destructor (with destroyed GL connection) Refactor so that SkGLContext subclass object creation is completely done by the factory function. If the factory function returns a non-NULL ptr, it means the context is usable. The destruction is done with the destructor instead of virtual function called upon destruction. Make the destructors not to call virtual functions, for clarity. Remove custom 1x1 FBO setup code from the base class. It appears not to be used anymore. BUG=skia:2992 Review URL: https://codereview.chromium.org/640283004
* bug_chomper: Consolidate public flag in a single place.Gravatar tfarina2014-10-14
| | | | | | | | BUG=None TEST=./run_server.sh --help and ./run_server.sh -public R=borenet@google.com Review URL: https://codereview.chromium.org/649303003
* Archive svndiff script.Gravatar tfarina2014-10-13
| | | | | | | | | | | Nobody is using this script anymore, so to prevent more changes to it, let's remove it now. BUG=None TEST=None R=borenet@google.com Review URL: https://codereview.chromium.org/646043002
* bug_chomper: Make server's port configurable.Gravatar tfarina2014-10-13
| | | | | | | | | | | | | | Make 'port' a flag so you can change it from the command line, making the server more flexible and allowing us to change in which port it listen to requests. $ ./run_server.sh --port :8002 BUG=None TEST=see above R=borenet@google.com Review URL: https://codereview.chromium.org/649663003
* bug_chomper: Use parenthesized declaration style for const and vars.Gravatar tfarina2014-10-10
| | | | | | | | | | | It seems cleaner that way and nicer to read, also avoids repeating 'const' and 'var' everytime you have to add more items. BUG=None TEST=./run_server.sh R=borenet@google.com Review URL: https://codereview.chromium.org/645803003
* Use the function recently added to common in svndiff.pyGravatar Thiago Farina2014-10-10
| | | | | | | | | | | GetModifiedFiles() function was added to common repo in https://skia.googlesource.com/common/+/af1de867422c42855ae54ed3838c62ea445ea6b4 BUG=None TEST=None R=borenet@google.com Review URL: https://codereview.chromium.org/634313006
* Make the Sk GL context class an abstract base classGravatar kkinnunen2014-10-09
| | | | | | | | | | | | | | | | | | | | | | | Make the Sk GL context class, SkGLNativeContext, an abstract base class. Before, it depended on ifdefs to implement the platform dependent polymorphism. Move the logic to subclasses of the various platform implementations. This a step to enable Skia embedders to compile dm and bench_pictures. The concrete goal is to support running these test apps with Chromium command buffer. With this change, Chromium can implement its own version of SkGLNativeContext that uses command buffer, and host the implementation in its own repository. Implements the above by renaming the SkGLContextHelper to SkGLContext and removing the unneeded SkGLNativeContext. Also removes SkGLNativeContext::AutoRestoreContext functionality, it appeared to be unused: no use in Skia code, and no tests. BUG=skia:2992 Committed: https://skia.googlesource.com/skia/+/a90ed4e83897b45d6331ee4c54e1edd4054de9a8 Review URL: https://codereview.chromium.org/630843002
* Update old tools to allow MultiPictureDraw renderingGravatar robertphillips2014-10-09
| | | | | | I'll post a separate patch for nanobench and dm Review URL: https://codereview.chromium.org/639013003
* Revert of Make the Sk GL context class an abstract base class (patchset #4 ↵Gravatar bsalomon2014-10-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | id:60001 of https://codereview.chromium.org/630843002/) Reason for revert: nanobech failing on Android Original issue's description: > Make the Sk GL context class an abstract base class > > Make the Sk GL context class, SkGLNativeContext, an abstract base class. Before, > it depended on ifdefs to implement the platform dependent polymorphism. Move > the logic to subclasses of the various platform implementations. > > This a step to enable Skia embedders to compile dm and bench_pictures. The > concrete goal is to support running these test apps with Chromium command buffer. > > With this change, Chromium can implement its own version of SkGLNativeContext > that uses command buffer, and host the implementation in its own repository. > > Implements the above by renaming the SkGLContextHelper to SkGLContext and > removing the unneeded SkGLNativeContext. Also removes > SkGLNativeContext::AutoRestoreContext functionality, it appeared to be unused: > no use in Skia code, and no tests. > > BUG=skia:2992 > > Committed: https://skia.googlesource.com/skia/+/a90ed4e83897b45d6331ee4c54e1edd4054de9a8 TBR=kkinnunen@nvidia.com NOTREECHECKS=true NOTRY=true BUG=skia:2992 Review URL: https://codereview.chromium.org/639793002
* Make the Sk GL context class an abstract base classGravatar kkinnunen2014-10-08
| | | | | | | | | | | | | | | | | | | | | Make the Sk GL context class, SkGLNativeContext, an abstract base class. Before, it depended on ifdefs to implement the platform dependent polymorphism. Move the logic to subclasses of the various platform implementations. This a step to enable Skia embedders to compile dm and bench_pictures. The concrete goal is to support running these test apps with Chromium command buffer. With this change, Chromium can implement its own version of SkGLNativeContext that uses command buffer, and host the implementation in its own repository. Implements the above by renaming the SkGLContextHelper to SkGLContext and removing the unneeded SkGLNativeContext. Also removes SkGLNativeContext::AutoRestoreContext functionality, it appeared to be unused: no use in Skia code, and no tests. BUG=skia:2992 Review URL: https://codereview.chromium.org/630843002
* Restore a really single-threaded mode to DM.Gravatar mtklein2014-10-06
| | | | | | | | | | | | | | | Used to be: 0 -> run on main thread plus an autodetected number of extra threads (default) N -> run on main thread plus N extra threads Now it's: -1 -> run on main thread plus an autodetected number of extra threads (default) 0 -> run on main thread N -> run on main thread plus N extra threads BUG=skia: Review URL: https://codereview.chromium.org/636593002
* Fix usage of SK_BUILD_* defines.Gravatar tfarina2014-10-06
| | | | | | | | | | | | | | | Since we just 'define' them, but not attribute anything to them, like '1' for example, cpp expands it to nothing and that breaks the "#if" clauses. To fix that, uses "#if defined(...)" which will correctly check if your macro name was defined or not. BUG=skia:2850 TEST=make most R=robertphillips@google.com Review URL: https://codereview.chromium.org/628763005
* Run goimports through 'go' code in bug_chomper.Gravatar tfarina2014-10-03
| | | | | | | | | | | | | $ cd ~/golib $ go get code.google.com/p/go.tools/cmd/goimports $ cd ~/skia/tools/bug_chomper/ $ ~/golib/bin/goimports -w src/ BUG=None TEST=None R=borenet@google.com Review URL: https://codereview.chromium.org/624813002
* Introduce Props to surface (patchset #27 id:520001 of ↵Gravatar reed2014-09-22
| | | | | | | | | | | | https://codereview.chromium.org/551463004/)" This reverts commit 29c857d0f3a1cb837f73406eeb6ba9771879b5e7. TBR= Author: reed@google.com Review URL: https://codereview.chromium.org/588143004
* Revert of introduce Props to surface (patchset #27 id:520001 of ↵Gravatar reed2014-09-21
| | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/551463004/) Reason for revert: Broke call site in WebKit Original issue's description: > introduce Props to surface (work in progress) > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/3716fd067a5621bb94a6cb08d72afec8bf3aceda R=robertphillips@google.com, bsalomon@google.com, jvanverth@google.com, bungeman@google.com, fmalita@google.com, vangelis@chromium.org, reed@google.com TBR=bsalomon@google.com, bungeman@google.com, fmalita@google.com, jvanverth@google.com, reed@google.com, robertphillips@google.com, vangelis@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Author: reed@chromium.org Review URL: https://codereview.chromium.org/583773004
* introduce Props to surface (work in progress)Gravatar reed2014-09-21
| | | | | | | | | BUG=skia: R=robertphillips@google.com, bsalomon@google.com, jvanverth@google.com, bungeman@google.com, fmalita@google.com, vangelis@chromium.org, reed@chromium.org Author: reed@google.com Review URL: https://codereview.chromium.org/551463004
* fail early if coincidence can't be resolvedGravatar caryclark2014-09-19
| | | | | | | | | | | | Bail out if a very large value causes coincidence resolution to fail. TBR= BUG=415866 Author: caryclark@google.com Review URL: https://codereview.chromium.org/585913002
* Add counting of some GL callsGravatar robertphillips2014-09-18
| | | | | | | | | | I would like this facility for tracking FBO switch improvements. R=bsalomon@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/553583008
* remove no-op overrideGravatar reed2014-09-18
| | | | | | | | | | NOTREECHECKS=True NOTRY=True R=egdaniel@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/586463002
* fix battlefield website by disallowing very small coordinatesGravatar caryclark2014-09-18
| | | | | | | | | | | also add and remove comments to document other attempts to fix this that had drawbacks R=fmalita@chromium.org BUG=414409 Author: caryclark@google.com Review URL: https://codereview.chromium.org/575553003
* add lua to histogram text lengthsGravatar reed2014-09-15
| | | | | | | | | NOTRY=True R=bungeman@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/574493002