aboutsummaryrefslogtreecommitdiffhomepage
path: root/tools/skp/webpages_playback.py
Commit message (Collapse)AuthorAge
* Omit "espn" from key_mobile_sites_smooth in RecreateSKPs botGravatar Ravi Mistry2018-05-20
| | | | | | | | | | | | We already have a espn page set in Skia. Removing it from the chromium page set does not lose us coverage. NoTry: true Bug: skia:7917 Change-Id: Ie5456ee7f8053ad904eb858eddfe564e3781249a Reviewed-on: https://skia-review.googlesource.com/129303 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Fix capitalization of "twitter" in RecreateSKPs botGravatar Ravi Mistry2018-05-16
| | | | | | | | | | | Case has suddenly started to matter. NoTry: true Bug: skia:7917 Change-Id: I6020622bd12481a2ac291caf565632451a3c8da5 Reviewed-on: https://skia-review.googlesource.com/128480 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Skip failing pageset from key_mobile_sites_smoothGravatar Ravi Mistry2018-02-26
| | | | | | | | | | | I would normally move the Chromium failing pagesets to be a Skia pageset but we used to have twitter.com/katyperry as a pageset a while ago and decided to remove it. NoTry: true Bug: skia:7348 Change-Id: I005b34b33f3bd2cc5e7948e1caad222156f36216 Reviewed-on: https://skia-review.googlesource.com/110120 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com>
* Blacklist mail.google.com from top25_desk pagesetGravatar Ravi Mistry2017-12-18
| | | | | | | | | | | | This will not lose coverage because we already have mail.google.com in Skia's page sets. NoTry: true Bug: skia:7421 Change-Id: Ie3d335559e736e9177c9000f41dab25581230c8b Reviewed-on: https://skia-review.googlesource.com/86500 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Telemetry no longer supports credential files so remove supportGravatar Ravi Mistry2017-12-13
| | | | | | | | | NoTry: true Bug: skia:7401 Change-Id: I18b2dfe92b6e998d0799ed95cd5d735706428d1c Reviewed-on: https://skia-review.googlesource.com/84540 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Exclude URLs only for some page setsGravatar Ravi Mistry2017-11-29
| | | | | | | | Bug: skia:7348 Change-Id: Iabf4c0d0bf83a0caeeb06027e5617a18900ac5fc Reviewed-on: https://skia-review.googlesource.com/77180 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Make RecreateSKPs bot fail if capturing SKPs fails + exclude failing page setsGravatar Ravi Mistry2017-11-28
| | | | | | | | Bug: skia:7348 Change-Id: I8dd6337d2efeb1aaa71ac41fcb82b9a22bbd7531 Reviewed-on: https://skia-review.googlesource.com/76220 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Update webpages_playback.py to use the new Golang impl of WPRsGravatar Ravi Mistry2017-08-02
| | | | | | | | | NoTry: true Bug: skia:6901 Change-Id: I1a0dc706f9494c261c2f609df81a37cc60970fdd Reviewed-on: https://skia-review.googlesource.com/30060 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Do not move over webpage archives, they are needed for capturing SKPsGravatar Ravi Mistry2017-07-18
| | | | | | | | | NoTry: true Bug: skia:6850 Change-Id: I35ac2c10ae58561ab5424033d9bb5e14b75c526f Reviewed-on: https://skia-review.googlesource.com/24327 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* RecreateSKPs: Fix another .join() issueGravatar borenet2016-10-17
| | | | | | | BUG=skia:5843 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2429513002 Review-Url: https://codereview.chromium.org/2429513002
* RecreateSKPs: Fix GoogleStorageDataStore.delete_pathGravatar borenet2016-10-17
| | | | | | | BUG=skia:5843 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2423093002 Review-Url: https://codereview.chromium.org/2423093002
* More fixes for webpages_playbackGravatar borenet2016-10-10
| | | | | | | BUG=skia:5843 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2409753002 Review-Url: https://codereview.chromium.org/2409753002
* webpages_playback: Fix GS path with extra '/'Gravatar borenet2016-10-10
| | | | | | | BUG=skia:5843 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2409593002 Review-Url: https://codereview.chromium.org/2409593002
* webpages_playback: use gsutil instead of API, no shell_utilsGravatar borenet2016-10-10
| | | | | | | BUG=skia:5843 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2406863002 Review-Url: https://codereview.chromium.org/2406863002
* webpages_playback: use new skia-skps bucketGravatar borenet2016-10-05
| | | | | | | BUG=skia:2969 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2393043003 Review-Url: https://codereview.chromium.org/2393043003
* Further increase run_benchmark timeoutGravatar Ravi Mistry2016-09-08
| | | | | | | | | | | | | | | Looks like the chromium page sets still hit the 10 min timeout. BUG=skia:5700 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2342 # This code is not exercised by trybots. NOTRY=true Change-Id: I57aba4c1541a840d7e4e97112a6485abc20aa3c7 Reviewed-on: https://skia-review.googlesource.com/2342 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ravi Mistry <rmistry@google.com>
* Increase run_benchmarks timeout because the two Chromium page sets can take ↵Gravatar rmistry2016-08-30
| | | | | | | | | | | | a while BUG=skia:5700 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2298453002 # Trybots do not exercise this code. NOTRY=true TBR=borenet Review-Url: https://codereview.chromium.org/2298453002
* Speculative fix for SKP partner-dir uploadGravatar borenet2016-08-08
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2229433002 Review-Url: https://codereview.chromium.org/2229433002
* webpages_playback: Make --upload_to_partner_bucket work without --uploadGravatar borenet2016-07-25
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2182463002 Review-Url: https://codereview.chromium.org/2182463002
* Add "skp" assetGravatar borenet2016-07-21
| | | | | | | | | | Don't get rid of SKP_VERSION and associated stuff just yet. BUG=skia:5427 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2167763002 DOCS_PREVIEW= https://skia.org/?cl=2167763002 Review-Url: https://codereview.chromium.org/2167763002
* Add catapult to RecreateSKPs bot's PYTHONPATHGravatar rmistry2016-02-27
| | | | | | | | | TBR=borenet BUG=skia:5024 NOTRY=true GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1747623002 Review URL: https://codereview.chromium.org/1747623002
* Fix delete_path in webpages_playback.pyGravatar rmistry2016-02-03
| | | | | | | | | | BUG=skia:3751 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1662673003 # Trybots do not exercise this code. NOTRY=true Review URL: https://codereview.chromium.org/1662673003
* Delete skp dir in partner bucket before uploadingGravatar rmistry2016-02-03
| | | | | | | | | | | BUG=skia:3751 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1660363002 # Trybots do not exercise this code. NOTRY=true TBR=borenet Review URL: https://codereview.chromium.org/1660363002
* Recreate SKPs in a single directory for trusted partners.Gravatar rmistry2016-02-02
| | | | | | | | | | | | | Context: https://bugs.chromium.org/p/skia/issues/detail?id=3751#c15 BUG=skia:3751 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1662693002 # Trybots do not run this code NOTRY=true TBR=borenet Review URL: https://codereview.chromium.org/1662693002
* Upload SKPs to chrome-partner-telemetryGravatar rmistry2016-02-01
| | | | | | | | | | | | | | | Context: https://bugs.chromium.org/p/skia/issues/detail?id=3751#c4 and https://bugs.chromium.org/p/skia/issues/detail?id=3751#c6 BUG=skia:3751 # Trybots do not run this code. NOTRY=true GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1658653002 Review URL: https://codereview.chromium.org/1658653002
* Add ability to specify extra browser args and a prefix for SKP names to ↵Gravatar rmistry2015-04-23
| | | | | | | | | webpages_playback.py BUG=skia:3763 NOTRY=true Review URL: https://codereview.chromium.org/1066933006
* Delete no longer needed pagesets, add new pagesets and make names more readableGravatar rmistry2015-04-03
| | | | | | | | | | Deleted all pagesets mentioned in https://code.google.com/p/skia/issues/detail?id=3574#c18 Added new Wikipedia and Unicodetable pagesets from https://code.google.com/p/skia/issues/detail?id=3574#c19 BUG=skia:3574 NOTRY=true Review URL: https://codereview.chromium.org/1053253003
* Turn on Chromium's top 25 page setGravatar rmistry2015-03-31
| | | | | | | | | | | #crskps BUG=skia: BUG=skia:3574 NOTRY=true TBR=borenet Review URL: https://codereview.chromium.org/1045153004
* Fix cr SKPs namingGravatar rmistry2015-03-31
| | | | | | | | | | | | | #crskps BUG=skia: BUG=skia:3574 NOTRY=true TBR=borenet Review URL: https://codereview.chromium.org/1044193003
* Turn on key mobile sites cr page set and delete dashed desktopGravatar rmistry2015-03-31
| | | | | | | | | | #crskps BUG=skia:3574 BUG=skia:3605 NOTRY=true Review URL: https://codereview.chromium.org/1040233004
* Fix gs.upload_dir_contents arguments mismatchGravatar rmistry2015-03-31
| | | | | | | | | | Fixing bug introduced by https://codereview.chromium.org/737123005 BUG=skia: NOTRY=true TBR=borenet Review URL: https://codereview.chromium.org/1041003003
* Add ability to capture SKPs from Chromium's pagesetsGravatar rmistry2015-03-31
| | | | | | | | | | | | | Also added support for pagesets with multiple pages in them. #crskps BUG=skia:1723 BUG=skia:3574 NOTRY=true Review URL: https://codereview.chromium.org/1047703003
* webpages_playback.py: Support storing to local directoriesGravatar kkinnunen2015-03-31
| | | | | | | | | | | | | | | | | Make webpages_playback support storing to local directories as an alternative to Google Storage. Rename argument --dest_gsbase to --data_store=<location>. Make this parameter accept either gs:// url or directory for local file system storage. Rename argument --upload_to_gs to --upload. Implement file existence check, download and upload for the file operation. NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/737123005
* webpages_playback.py: Avoid crash when retrying to capture skpGravatar kkinnunen2015-03-29
| | | | | | | Move the .wpr files after recording, in the recording retry loop. Previously they were moved in the skp capture retry loop. Review URL: https://codereview.chromium.org/1033223005
* Make debugger not fail with webpages_playback script and multiple skp filesGravatar kkinnunen2014-12-03
| | | | | | | | The debugger accepts either a single skp or a directory as a parameter. The webpages_playback script used to pass multiple files using a '*' glob. Review URL: https://codereview.chromium.org/775073002
* Update webpages_playback documentationGravatar kkinnunen2014-12-03
| | | | | | | Update the documentation on how to run webpages_playback. It was out of date. Review URL: https://codereview.chromium.org/778633002
* Remove Google Storage scheme prefix correctly from gs base name in ↵Gravatar kkinnunen2014-12-03
| | | | | | | | | | | | webpages_playback In python, str.lstrip([chars]) means "lstrip any of the chars from str", not "remove prefix". Thus "gs://skia".lstrip("gs://") yields "kia". Replace with custom function stripping the prefix from lstrip()ed gs base name. Review URL: https://codereview.chromium.org/780493002
* Make render_pdfs not crash webpages_playback scriptGravatar kkinnunen2014-12-03
| | | | | | | | | | | The render_pdfs nowadays seems to input the skps from the dir passed as '-r' parameter. The app will not output anything, but it will not crash the webpages_playback script either. Affects only runs that have --skia_tools=<dir> argument. Review URL: https://codereview.chromium.org/776873004
* 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
* 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
* 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
* 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
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-29 15:19:29 +0000