aboutsummaryrefslogtreecommitdiffhomepage
path: root/infra/bots/jobs.json
Commit message (Collapse)AuthorAge
...
* Output calmbench json in nanobench formatGravatar Yuqian Li2017-10-30
| | | | | | | | | | | Meanwhile, we also make the bot name more similar to the perf bot. Bug: skia: Change-Id: I8b0e6aa2d927aa4a0ce57606345589adb6e920b3 Reviewed-on: https://skia-review.googlesource.com/64082 Reviewed-by: Eric Boren <borenet@google.com> Reviewed-by: Joe Gregorio <jcgregorio@google.com> Commit-Queue: Yuqian Li <liyuqian@google.com>
* Remove Nexus10 GPU jobs.Gravatar Ben Wagner2017-10-26
| | | | | | | | | | | | | | The latest OS is Lollipop, and it was originally released November 2012. The GPU is not in the top 26 according to UMA stats (I haven't collected data for lower than 26). We test the same GPU with the Chromebook_303C12, and bsalomon says "Personally, I'm OK with just relying on ChromeOS for testing." No-Try: true Change-Id: Ib17033153faab9d99613e7d2d069524c9435f65e Reviewed-on: https://skia-review.googlesource.com/64066 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Move FxAA jobs to Win2k8, Clang, GCE, 64-bit; run fewer configs.Gravatar Ben Wagner2017-10-26
| | | | | | | | | | | These jobs were originally added in https://skia-review.googlesource.com/c/skia/+/60540. No-Try: true Change-Id: I38677933dbb05dac8df9ea236af9a5d7b8227623 Reviewed-on: https://skia-review.googlesource.com/63742 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Yuqian Li <liyuqian@google.com>
* Add full skpbench jobs for Pixel phoneGravatar Chris Dalton2017-10-25
| | | | | | | | Bug: skia: Change-Id: I1b2ae213f332a3597e834e036efb8dad8df12b8b Reviewed-on: https://skia-review.googlesource.com/63541 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com>
* Move most remaining ANGLE bots to Clang.Gravatar Ben Wagner2017-10-24
| | | | | | | | | | | Sort job names. Docs-Preview: https://skia.org/?cl=63080 No-Try: true Change-Id: Ia72c055c4ddb903063b8a4be5ebd6ba92fc8b6d6 Reviewed-on: https://skia-review.googlesource.com/63080 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add compiler configuration to calmbench botsGravatar Yuqian Li2017-10-23
| | | | | | | | | | | | Now the calmbench bot has the same name schema as the build bots and we're testing using Clang instead of GCC. In the future, we may also import some configuration schemas from perf bots. Bug: skia: Change-Id: Idd9556cff36b749a6c9139d34f70ac50ca48fbba Reviewed-on: https://skia-review.googlesource.com/62920 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Yuqian Li <liyuqian@google.com>
* add -SafeStack botsGravatar Mike Klein2017-10-23
| | | | | | | | | | | | | | | | These bots use https://clang.llvm.org/docs/SafeStack.html. This separates the safe (constant size?) parts of the stack like return address and some local variables apart from the unsafe over- and underflow prone array uses. These go on the "unsafe stack". In theory this makes sense on {Linux,Mac}x{Debug,Release}, but let's just start with Linux/Debug. Change-Id: I9f5cbeb7081e7d9ac8e1298ca0d86880a6085cfa Reviewed-on: https://skia-review.googlesource.com/62743 Reviewed-by: Yuqian Li <liyuqian@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Add skpbench jobs for CCPRGravatar Chris Dalton2017-10-20
| | | | | | | | | | | Adds new runs that benchmark just a handful of skps with CCPR. This will also be Pixel's small debut on skpbench. Bug: skia: Change-Id: I7e4c3aeaa083ffe29c275f1c021ee62d177b6172 Reviewed-on: https://skia-review.googlesource.com/62302 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Chris Dalton <csmartdalton@google.com>
* Fix FxAA job names.Gravatar Ben Wagner2017-10-17
| | | | | | | | | | | | | Fixes a conflict between https://skia-review.googlesource.com/c/skia/+/60540 and https://skia-review.googlesource.com/59961. No-Try: true Change-Id: Ie2731679f4471eff51ac468a794c4c17c2f931df Reviewed-on: https://skia-review.googlesource.com/60861 Reviewed-by: Kevin Lubick <kjlubick@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Shard GCE Coverage bot to decrease latencyGravatar Kevin Lubick2017-10-17
| | | | | | | | Bug: skia: Change-Id: I28d28f514b46ad0c30109beba90ae8a1bd8fc4e3 Reviewed-on: https://skia-review.googlesource.com/59961 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Add tests that force a specific CPU scan converterGravatar Yuqian Li2017-10-17
| | | | | | | | | | | | This would allow us to have more test coverage on specific scan converters which may help us detect some wierd bugs such as crbug.com/774008 Bug: chromium:774008 Change-Id: I9213b11bb47ce48a6eff8da494e1b2699d386ae9 Reviewed-on: https://skia-review.googlesource.com/60540 Reviewed-by: Mike Klein <mtklein@google.com> Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Yuqian Li <liyuqian@google.com>
* Try calmbench on botsGravatar Yuqian Li2017-10-16
| | | | | | | | Bug: skia: Change-Id: I869776e3d9d47b4a1a11089f729a0b17581eb9f7 Reviewed-on: https://skia-review.googlesource.com/57400 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Yuqian Li <liyuqian@google.com>
* Add in test_filter to build configurationGravatar Kevin Lubick2017-10-16
| | | | | | | | | | | | | The first "sharding" technique we will try is just by test config (e.g. 8888, gles, etc). Thus, for backwards compatibility, the default "shard" is All, as in all configs Bug: skia: NOTRY=true Change-Id: Ia02362477a5d97f8f74d688b5f0c4f45fc129375 Reviewed-on: https://skia-review.googlesource.com/59563 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Add Win Clang ANGLE test and perf jobs.Gravatar Ben Wagner2017-10-13
| | | | | | | | No-Try: true Change-Id: I95a29b1abfd84e51f5a31d74644e5305779f0d15 Reviewed-on: https://skia-review.googlesource.com/57100 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
* Switch most Win Vulkan jobs to Clang.Gravatar Ben Wagner2017-10-11
| | | | | | | | | Update examples in recipes. Change-Id: I096f259e8daa46af8626158a820fe226e99cd159 Reviewed-on: https://skia-review.googlesource.com/58200 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add Linux GPU Coverage botGravatar Kevin Lubick2017-10-10
| | | | | | | | | | | Adds it for Vulkan and non-Vulkan Bug: skia:7080 NOTRY=true Change-Id: I5c4b30b91b96ad6ce9694c42550c4e909de4ba41 Reviewed-on: https://skia-review.googlesource.com/57560 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
* Add Linux CPU Coverage BotGravatar Kevin Lubick2017-10-09
| | | | | | | | | | | This simply uploads the results of an LLVM coverage to GCS for later ingestion/display. Bug: skia:7080 Change-Id: I7dcfa2307a239734a614990aca899ea37129126b Reviewed-on: https://skia-review.googlesource.com/53880 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Cleanup jobs.Gravatar Ben Wagner2017-10-09
| | | | | | | | | | | | | | | NUC6i7KYK with GTX960 is an experimental eGPU setup that will be replaced with the NUC7i5BNK. IntelHD4600 has been replaced with IntelHD4400. Move remainder of *AbandonGpuContext jobs to QuadroP400. No-Try: true Change-Id: Ic81392ec162cb88500c9da7609471dbdc4a64f84 Reviewed-on: https://skia-review.googlesource.com/57320 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add Win Clang Vulkan jobs.Gravatar Ben Wagner2017-10-08
| | | | | | | | No-Try: true Change-Id: I0b9f07e9057892443934da399f60762005495bde Reviewed-on: https://skia-review.googlesource.com/57081 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add Win Clang Vulkan builds.Gravatar Ben Wagner2017-10-07
| | | | | | | | | Fix Clang compiler warning. Change-Id: Ic6f5b161058d1ed2df45e12b72dbff3b486ec986 Reviewed-on: https://skia-review.googlesource.com/56740 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Remove Perf SurfacePro2017 jobs.Gravatar Ben Wagner2017-10-06
| | | | | | | | | | | Perf numbers track closely with IntelIris540, although Perf numbers from SurfacePro2017 are more noisy. No-Try: true Change-Id: I9dfefd6daf69644a1c0850453334876269cd7942 Reviewed-on: https://skia-review.googlesource.com/56540 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Switch many jobs to Clang.Gravatar Ben Wagner2017-10-03
| | | | | | | | No-Try: true Change-Id: Ie255d46ff50d13d25d045791c1c3066f06ab9243 Reviewed-on: https://skia-review.googlesource.com/53601 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
* Add Linux nVidia Vulkan jobs.Gravatar Ben Wagner2017-10-02
| | | | | | | Change-Id: Ib323136e4c6f660923433a2508e1e0965712d698 Reviewed-on: https://skia-review.googlesource.com/53460 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com>
* Add AVX512 jobs.Gravatar Ben Wagner2017-09-29
| | | | | | | | | Bug: skia:6326 No-Try: true Change-Id: Iaffc0cbbe14237e7ac2d83cc9da94a4c7bea3154 Reviewed-on: https://skia-review.googlesource.com/52764 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* [WASM] Add POC compile bot for WebAssemblyGravatar Kevin Lubick2017-09-21
| | | | | | | | | | | | | | | | | | | | Fix core.gni to use not use Assembler for none cpu. Right now, there are no outputs because we aren't compiling dm or nanobench. However, this still compiles the skia library and creates two executables, so it's a good canary for a real WASM build. Additional note: the two executables in question don't draw anything to the screen via GL, which is still not possible with Skia+WASM. Bug: skia: Change-Id: I0d767467e94e40d01070e34223dd90e96f1c96f2 Reviewed-on: https://skia-review.googlesource.com/49540 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org> Reviewed-by: Eric Boren <borenet@google.com>
* add Mac TSAN Test botsGravatar Mike Klein2017-09-19
| | | | | | | | | The build should suceeed, but the test bots should fail miserably. Change-Id: I1b525255f01fd001b1cc8a45532ab5a911b36bd6 Reviewed-on: https://skia-review.googlesource.com/48581 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* Remove redundant jobs.Gravatar Ben Wagner2017-09-14
| | | | | | | | | | | | | Replacements added in: https://skia-review.googlesource.com/c/skia/+/26365 https://skia-review.googlesource.com/c/skia/+/31143 Bug: skia:6881 No-Try: true Change-Id: I35a212f55b9f2cd1240f2f072321ee118a3feecc Reviewed-on: https://skia-review.googlesource.com/46306 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add new task to run the float-cast-overflow sanitizer.Gravatar Ben Wagner2017-09-13
| | | | | | | | | Bug: skia:4632 No-Try: true Change-Id: I85a0b23037d5885e5d762788d8bf5b7cc6fc19b2 Reviewed-on: https://skia-review.googlesource.com/45980 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Mike Klein <mtklein@google.com>
* Add Clang GPU tasks.Gravatar Ben Wagner2017-09-12
| | | | | | | | No-Try: true Change-Id: I5290942396a903fcaac94feaab83a5db31af1f93 Reviewed-on: https://skia-review.googlesource.com/45844 Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
* Add Correctness tests for ChromecastGravatar Kevin Lubick2017-08-31
| | | | | | | | | Bug: skia: NOTRY=true Change-Id: Ib604bad8eda6ba4382b850f7c5f318fb0f114401 Reviewed-on: https://skia-review.googlesource.com/20051 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Remove SurfacePro Test tasks.Gravatar Ben Wagner2017-08-18
| | | | | | | | | | | | | | Test tasks were added in https://skia-review.googlesource.com/c/35321, but dm results are identical to to IntelIris540; just some differences in fonts. Removing the Test tasks will allow the bot to run Perf more frequently, providing better data. Bug: skia:6978 Change-Id: Icb8b6af7267a879b2d761d0e7fca6064f53281f8 No-Try: true Reviewed-on: https://skia-review.googlesource.com/36161 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Rename Mac bots to match reality.Gravatar Ben Wagner2017-08-18
| | | | | | | | | | | | Doing this separately from https://skia-review.googlesource.com/c/36040 to avoid confusion. No-Try: true Change-Id: I2d9e03c6da6096f9e14145d9654b9da7a8670036 Reviewed-on: https://skia-review.googlesource.com/35823 Reviewed-by: Brian Salomon <bsalomon@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add SurfacePro2017 tasks.Gravatar Ben Wagner2017-08-17
| | | | | | | | | | | | Not adding Vulkan configs because it mostly produces garbage -- see skia:6398. Bug: skia:6978 NoTry: true Change-Id: I8f9fa7ecc5d195658944f8f29a91ddaf47c066fc Reviewed-on: https://skia-review.googlesource.com/35321 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
* Add tasks for Acer R13Gravatar Kevin Lubick2017-08-15
| | | | | | | | | Bug: skia: NOTRY=true Change-Id: Iafc9a00ff2a41919214cdea89251ceb0c5b443c5 Reviewed-on: https://skia-review.googlesource.com/34721 Reviewed-by: Ben Wagner <benjaminwagner@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>
* Add new Linux IntelIris640 jobs.Gravatar Ben Wagner2017-08-11
| | | | | | | | | Bug: skia:6761 Change-Id: I635958a482a16fc7027d33152db431cb8c5bbe2c No-Try: true Reviewed-on: https://skia-review.googlesource.com/33740 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add duplicate Valgrind tasks on QuadroP400.Gravatar Ben Wagner2017-08-09
| | | | | | | | | | | | | | | | | Valgrind 3.13.0 only supports up to AVX2, so I used SK_CPU_LIMIT_SSE41 to avoid failing with "Unrecognised instruction." (The existing Valgrind tasks run on a ShuttleA machine, whose CPU only supports AVX.) Needed to enable verbose output on all Valgrind tasks to avoid Swarming I/O timeout. Opportunistically removed verbose output for Linux Intel bots that are no longer failing. Bug: skia:6881 Change-Id: I2ffa6efe901c97bd2e0bbc9b26632aafbb3cf9a6 No-Try: true Reviewed-on: https://skia-review.googlesource.com/31143 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
* Bot updates for SkiaCT tasks.Gravatar Ravi Mistry2017-08-08
| | | | | | | | | | | | | | | | | | | | | Use gcc-4.8 when compiling for Ubuntu14. Rename the CT tasks to make it clear their child tasks run on Ubuntu14. Move to Debian-9.1 bots created with the changes in https://skia-review.googlesource.com/c/31281 and https://skia-review.googlesource.com/c/31820 Change the cache dir used by the ct_skps recipe so that changing the bot name doesn't invalidate the cache. Bug: skia:6921 Bug: skia:6818 No-Try: true Change-Id: Idaac8187427b5c2c7bffe66ad6674496b8cd0c0e Reviewed-on: https://skia-review.googlesource.com/31242 Commit-Queue: Ravi Mistry <rmistry@google.com> Reviewed-by: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Ravi Mistry <rmistry@google.com>
* Add duplicate XSAN tasks on QuadroP400.Gravatar Ben Wagner2017-08-04
| | | | | | | | | Bug: skia:6881 Change-Id: I53356a19aaa488f0f739f8ddb687bbe0ea4994fe No-Try: true Reviewed-on: https://skia-review.googlesource.com/26365 Commit-Queue: Ben Wagner <benjaminwagner@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Add Linux QuadroP400 bots.Gravatar Ben Wagner2017-08-04
| | | | | | | | | Bug: skia:6882 Change-Id: I9514af6aa710f1722be277a529acd1d88ed32bc9 No-Try: true Reviewed-on: https://skia-review.googlesource.com/26364 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add Win10 QuadroP400 bots.Gravatar Ben Wagner2017-08-03
| | | | | | | | | Bug: skia:6883 Change-Id: I0812bc0715f10e3722535af2cfc6fdf812cc487f No-Try: true Reviewed-on: https://skia-review.googlesource.com/26366 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
* Add Perf-Win2k8-Clang bots.Gravatar Mike Klein2017-08-01
| | | | | | | Change-Id: I8108f48bb55cfb39532ef05e8e263769ff8fb10f Reviewed-on: https://skia-review.googlesource.com/29420 Reviewed-by: Yuqian Li <liyuqian@google.com> Commit-Queue: Mike Klein <mtklein@chromium.org>
* add more win/clang botsGravatar Mike Klein2017-07-31
| | | | | | | | | | | | - add a Debug builder - add Debug and Release CPU testers on GCE CQ_INCLUDE_TRYBOTS=skia.primary:Build-Win-Clang-x86_64-Debug,Test-Win2k8-Clang-GCE-CPU-AVX2-x86_64-Debug,Test-Win2k8-Clang-GCE-CPU-AVX2-x86_64-Release Change-Id: I4f2d7309b36a532683f68fbdde9c7a9aaaa861ea Reviewed-on: https://skia-review.googlesource.com/29023 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Mike Klein <mtklein@chromium.org>
* add a Win/Clang build botGravatar Mike Klein2017-07-31
| | | | | | | | | | BUG=skia:2679 CQ_INCLUDE_TRYBOTS=skia.primary:Build-Win-Clang-x86_64-Release Change-Id: I861366ae1340abcc67e87041904d08337b465bbe Reviewed-on: https://skia-review.googlesource.com/28864 Reviewed-by: Eric Boren <borenet@google.com>
* Add Metal build botGravatar Greg Daniel2017-07-28
| | | | | | | Bug: skia:6896 Change-Id: If2ed083703e6300b9d2a86cfb298011bbf54e018 Reviewed-on: https://skia-review.googlesource.com/28180 Reviewed-by: Eric Boren <borenet@google.com>
* Add a bot to verify that generated SKSL files have not been hand-editedGravatar Eric Boren2017-07-28
| | | | | | | | Bug: skia:6893 Change-Id: I54480ddd1ad1989661393500044f9164a99ce7b0 Reviewed-on: https://skia-review.googlesource.com/27700 Reviewed-by: Ravi Mistry <rmistry@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* Remove Ci20 MIPS devicesGravatar Kevin Lubick2017-07-24
| | | | | | | | | Bug: skia: NOTRY=true Change-Id: I125154a1e217fcaf085bdac704e3e761bf1e61d2 Reviewed-on: https://skia-review.googlesource.com/26380 Commit-Queue: Kevin Lubick <kjlubick@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
* Add new Android builder which uses API_LEVEL 26Gravatar Eric Boren2017-07-20
| | | | | | | | Bug: skia:6870 Change-Id: Idd9a27518c1123c0a729b4d07461047f5530aae3 Reviewed-on: https://skia-review.googlesource.com/24960 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
* Add two test bots for coverage counting path rendererGravatar Chris Dalton2017-07-18
| | | | | | | | Bug: skia: Change-Id: Ib8e89e5e9a75d24b69c23873168fcc23f9eb5c45 Reviewed-on: https://skia-review.googlesource.com/24063 Commit-Queue: Chris Dalton <csmartdalton@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* Stop adding Android-specific defines to SkUserConfigGravatar Leon Scroggins III2017-07-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove #defines that lived in gn_to_bp.py and android_framework_defines.gni. These have been moved into a new file in Android, SkUserConfigManual.h, in https://googleplex-android-review.git.corp.google.com/#/c/2519600/ Update gn_to_bp.py to include SkUserConfigManual.h, so it will still result in using the same #defines. Lately, we've found it difficult to guard changes behind a flag. e.g. a change to drawing causes a CTS failure in Android, so we have to do the following: - put the change behind a flag, and add it to gn_to_bp.py or android_framework_defines.gni - generate new images on Android (by running CTS with external/skia modified to not define the flag) - create a CL in CTS that uses the new images - land a CL in Skia that stops defining the flag - when the Skia change lands, wait for the auto-roller to create a CL that includes the change, stop the auto-roller, add the topic to the CTS CL so the two can land at the same time - land both Android changes (with TreeHugger) - restart the Android auto-roller With SkUserConfigManual.h (which lives in Android), the process will be similar to Chromium: - land a CL in Android's external/skia that defines a flag e.g. SK_SUPPORT_LEGACY_FEATURE. Land without TreeHugger because it isn't used in Skia and does not do anything - land a change in Skia that changes behavior unless SK_SUPPORT_LEGACY_FEATURE is defined. This will safely go through the Android roll and not change any behavior for Android - create two Android CLs - one in CTS to use the new images, and one in external/skia to delete SK_SUPPORT_LEGACY_FEATURE. Set them to the same topic and land them with TreeHugger In the new process, there is no need to mess with the Android roll. A downside to the new process is that we cannot test the android framework defines without checking in to Android. But given how much we've progressed in automating Android testing, this is fine. Bug: b/63429612 Change-Id: Idfbaef2f4cae641a75fb6e7bf70428733a441336 Reviewed-on: https://skia-review.googlesource.com/22072 Commit-Queue: Leon Scroggins <scroggo@google.com> Reviewed-by: Eric Boren <borenet@google.com>
* [infra] Golo bots have been upgraded to Ubuntu 17.04Gravatar Eric Boren2017-07-11
| | | | | | | Bug: skia: Change-Id: Ia18c02660b2ab75559400f26ecf610dd610b6540 Reviewed-on: https://skia-review.googlesource.com/22202 Reviewed-by: Ravi Mistry <rmistry@google.com>
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-08 14:51:30 +0000