Age | Commit message (Collapse) | Author |
|
Bug: 80207024
Change-Id: Ia99833fc799b679110016ef77871a81985ddab4d
Reviewed-on: https://skia-review.googlesource.com/142592
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
(cherry picked from commit 3e34285f2a0fd2aa5f83f51dd65a42fd454ed7ee)
Reviewed-on: https://skia-review.googlesource.com/142761
|
|
Bug: 79208762
Change-Id: I5f1bea3dc4eda203eb4af2a433ec219d258be12a
Reviewed-on: https://skia-review.googlesource.com/134611
Reviewed-by: Derek Sollenberger <djsollen@google.com>
(cherry picked from commit cdd738ccc6308b573c54c2ac64f2f85f0d8f6391)
Reviewed-on: https://skia-review.googlesource.com/134641
Reviewed-by: Hal Canary <halcanary@google.com>
|
|
Bug: 80423218
Change-Id: Ied9cb7c72cddb19430b1475bd436bbc8c3ebd3fd
Reviewed-on: https://skia-review.googlesource.com/130541
Reviewed-by: Derek Sollenberger <djsollen@google.com>
|
|
Bug: 109842281
Test: This
Fix up test names to only include valid characters.
Change-Id: I4709d7a92e6ad822ec1f208823ddb3b37356d142
|
|
The "/" character is not a valid character for method names in the
CTS testing environment and is causing problems with some of the
tooling so we are replacing the "/" with "_".
cherry picked from commit e4201104f1659e5ca283f0126f42403609e379d7
Reviewed-on: https://skia-review.googlesource.com/129620
Bug: 80096930
Test: atest CtsSkQPTestCases
Change-Id: Ia260eed13d590ecc90debfb7050a0ff295a76885
|
|
Devices like the Mali-450 don't translate highp to 32-bit float which
causes rendering errors. We have mitigated many of these issues
for future releases, but instead of regenerating new models for this
release we will simply relax the testing constraints until next release
where we can use both the updated code and generate new models.
This is a cherry-pick of 45537a2efde695a0cce07ffc1f17cd8b3da2bc6e from
https://skia.googlesource.com/skia/+/45537a2efde695a0cce07ffc1f17cd8b3da2bc6e
Test: atest CtsSkQPTestCases
Bug: 79931609
Change-Id: I02230a8d247be6972ca9fa01beb3fe4bdf4df912
|
|
Test: atest CtsSkQPTestCases
Bug: 75981841
Change-Id: I7eca7bae8a13c74aeeb9ccff59f8cf31eeac26df
|
|
aosp/master
Change-Id: I92a7015d9517c83d69685bf30d27cdedf745abb9
|
|
aosp/master
Change-Id: Icb106666d47d54cfbba9551aae91add52cfe2bc8
Exempt-From-Owner-Approval: upstream merge
|
|
TBR: djsollen@google.com, jvanverth@google.com
Bug: skia:
Change-Id: I3c43a8e563a47ef43e64231a1895565fd1c18370
Reviewed-on: https://skia-review.googlesource.com/117622
Reviewed-by: Greg Daniel <egdaniel@google.com>
|
|
The general test creates an AHardwareBuffer. Imports into a backend to file with data.
And then imports it into another backend to read the HWB and draws it to a surface.
We then readback the pixels to make sure they match what we started with.
This version of the test tests all combinations of srcBackends (CPU, EGL, Vulkan) with the
dst backends (EGL, Vulkan).
Also cherry-picks in changes:
Add ctor to GrVkAlloc (https://skia-review.googlesource.com/111660)
Add testing only helper to flush and sync gpu (https://skia-review.googlesource.com/113122)
Bug: skia:
Change-Id: I732f371af7f48f5a590fd161021a69cbc6f2583c
Reviewed-on: https://skia-review.googlesource.com/117080
Reviewed-by: Jim Van Verth <jvanverth@google.com>
|
|
Includes local change to build as a multi-lib
Test: atest CtsSkQPTestCases
Bug: 74442448
Change-Id: I07df9a63227012369b890a7774404c831633fed4
|
|
Exempt-From-Owner-Approval: upstream merge
Change-Id: I221f38a3a0a8263689effad2a574f46c829434cd
|
|
These two tests fail without BGRA support.
TODO: restore not_native32_bitmap_config and give GPU backend ability to
fall back on CPU to fix swizzle when needed. Or document that this might
fail to draw.
Change-Id: Idcd4a2e039e148f32e19b8808fd6d889b845f1e8
Reviewed-on: https://skia-review.googlesource.com/115073
Reviewed-by: Hal Canary <halcanary@google.com>
|
|
NoTry: true
Change-Id: I39562e0aaf8c70cc15484c559c66f13d1ada1498
Reviewed-on: https://skia-review.googlesource.com/114370
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Hal Canary <halcanary@google.com>
|
|
The resources and build files were updated with the following
sequence. Subsequent merges from upstream skqp branches should
preform these steps as part of the merge process.
python tools/skqp/gn_to_bp.py
python tools/skqp/download_model
python tools/skqp/setup_resources
Additionally the following one time step were performed.
1) Added MODULE_LICENSE_BSD file
2) Added empty SkUserConfigManual
3) Added Android.mk and AndroidTest.xml for CtsSkQPTestCases
4) Deleted .gitignore in the assets directory
Bug: 74442448
Change-Id: If247adcca136daa43ed574eb19dfe1fd30f0f6c1
|
|
BUG=skqp:7695
Change-Id: Ifd87db5781542f1f1d37dd540420cf9fbaff37de
Reviewed-on: https://skia-review.googlesource.com/112844
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
also, never execute `atlastext`.
BUG=skia:7691
Change-Id: I190d9603085bc085e54290a522f98b12578032d9
Reviewed-on: https://skia-review.googlesource.com/112402
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Change-Id: I9ff70e1050f5b82d226ee601d090fea2ba874f49
Reviewed-on: https://skia-review.googlesource.com/111461
Reviewed-by: Hal Canary <halcanary@google.com>
|
|
PLEASE NOTE: Instructions for running a single test have changed!
No-Try: true
Change-Id: I1923240e879daa7ff0556737ddd5aa3f58e0097c
Reviewed-on: https://skia-review.googlesource.com/109566
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Derek Sollenberger <djsollen@google.com>
(cherry picked from commit 65a4178ff7c9c094fc9ddf6713ca8122e4950a7f)
Reviewed-on: https://skia-review.googlesource.com/109820
Reviewed-by: Hal Canary <halcanary@google.com>
|
|
PLEASE NOTE: Instructions on running `am instrument` for the
SkQP APK have changed.
To run a single test, see the section "Running a single test"
in `tools/skqp/README.md`.
No-Try: true
Change-Id: I0a2cbc47755929d6c6a927a3591ff98046779c77
Reviewed-on: https://skia-review.googlesource.com/108780
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
(cherry picked from commit f637cc01f8f28d920a1d5b792193e97e8dd63e42)
Reviewed-on: https://skia-review.googlesource.com/109166
Reviewed-by: Hal Canary <halcanary@google.com>
|
|
Change-Id: I00c6eba278f480cfffebc0b7c621eaa1a017609a
Reviewed-on: https://skia-review.googlesource.com/107003
Reviewed-by: Derek Sollenberger <djsollen@google.com>
|
|
- Fix some naming confusion between `x64` and `x86-64`.
- SkQP Universal (multi-arch apk) Build Script
[tools/skqp/make_universal_apk]
- skqp/build.gradle: add `universal` product flavor
- skqp manifest: make debug build optional.
No-Try: true
Change-Id: Ic8cd88719a34786ab2d53771749b2beb9db98be5
Reviewed-on: https://skia-review.googlesource.com/105023
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Stephan Altmueller <stephana@google.com>
(cherry picked from commit c87fbf93d0c9329a1f14da55aa96a27547ae0df6)
Reviewed-on: https://skia-review.googlesource.com/106542
Reviewed-by: Derek Sollenberger <djsollen@google.com>
|
|
No-Try: true
Change-Id: I25ca750c3baa8176b01d3b642a7cdbe67686dd77
Reviewed-on: https://skia-review.googlesource.com/103440
Reviewed-by: Hal Canary <halcanary@google.com>
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Stephan Altmueller <stephana@google.com>
(cherry picked from commit f2579ca3f2e94dfdda8eb19864f4e5487357b456)
Reviewed-on: https://skia-review.googlesource.com/106540
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Change-Id: I7c4d1d451c3e1987c4844347034259a8313326fc
|
|
Change-Id: I0176bcb0a6048adb66c941dfd542841a5a21be5b
|
|
Change-Id: I3de0df6558ed044a06e971f20f99b413993f1999
|
|
Also update model. No slack, add more to 'DoNotScore' list.
Change-Id: I50fee65f814f50c67d67a1bb02b0d811f149d243
|
|
Change-Id: I8e61a986f0b2e3826491e14ee4c7b56d59a982a1
|
|
Change-Id: I2bc87007eb3a4322a1e18694664fa16e13cd64ff
|
|
Change-Id: Ibc5b55c98242c50b62b16134e0a504c04a458f6c
Reviewed-on: https://skia-review.googlesource.com/102320
Reviewed-by: Hal Canary <halcanary@google.com>
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Change-Id: I0fd17ae3ee3fd5ea9f5e1eec345775af1904c6d0
|
|
Also, update SkQP model.
Change-Id: Ieddb5558c798eab73c2a13a7c9bb03d59b8a36c7
|
|
Change-Id: I24ee2fb3a4a4ca03a60ee2f1a90e022825eee409
Reviewed-on: https://skia-review.googlesource.com/101820
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Change-Id: I77936dcfc052f5ee15bc078020179e2eff1ba0e5
Reviewed-on: https://skia-review.googlesource.com/101742
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Change-Id: Ic57a86ec115e3ace52615c0b8e9a22a915988850
Reviewed-on: https://skia-review.googlesource.com/101320
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Also, Merge remote-tracking branch 'origin/master' into skqp/dev
Change-Id: Icea68087ef0244053454da45bf435eefc2365cf2
|
|
Change-Id: I74de5477c3cc9a034553363ac525d8f264478153
Reviewed-on: https://skia-review.googlesource.com/99862
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
- tools/skqp/generate_gn_args.sh - single script to generate
configuration used to build SkQP.
- tools/skqp/make_model.sh - script to execute make_gmkb.go and put
the results in the correct place.
- tools/skqp/{up,down}load_directory_contents.sh - scripts to move
large assets into the cloud and retrieve them.
- tools/skqp/gm_runner.cpp - treat missing text files as empty.
- tools/skqp/make_apk.sh - make use of generate_gn_args.sh.
- tools/skqp/make_{model,known_tests}.sh - better error handling,
and also stop calling `git add`.
Change-Id: Ib1803b4d68e63945f2c47f8eb6cb96375d24e3be
Reviewed-on: https://skia-review.googlesource.com/98842
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
NoScoreInCompatibilityTestMode
Also clean up some things, fix docs, whitelist.
Change-Id: I2818d973978ffe1b8ce0cc9c69f8d91ab4a0ef22
Reviewed-on: https://skia-review.googlesource.com/91805
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Change-Id: If0b093cfe646f5805cc4d0660c581150fbf992f4
Reviewed-on: https://skia-review.googlesource.com/96702
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Stephan Altmueller <stephana@google.com>
|
|
Switches over to writing output to the sd card.
Change-Id: Ic287a741b9af471f823a7e97eb0c6d3d20f639d6
Reviewed-on: https://skia-review.googlesource.com/96380
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Stephan Altmueller <stephana@google.com>
|
|
Bug: skia:
Change-Id: Id26832ccce7d09d3d3d640bcdd11a424f18943f7
Reviewed-on: https://skia-review.googlesource.com/92682
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Adds activities to the skqp app so it can run as an Android
app (as opposed to just instrumentation tests).
A user can trigger the tests via a button.
Adds the an intent receiver so the tests can be triggered on
Firebase Testlab via the gameloop option.
It adds the run_testlab.go script to run an apk across devices
on Firebase Testlab.
Bug: skia:
Change-Id: I3ff5c37d743fa47913a916a0fa1e7db3c2cc79c7
Reviewed-on: https://skia-review.googlesource.com/89163
Commit-Queue: Stephan Altmueller <stephana@google.com>
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Derek Sollenberger <djsollen@google.com>
|
|
* Generate report in app after running all tests.
* Add script to unpack backup.ab file.
* Remove unused index file.
* Streamline instructions in README.md.
Change-Id: I44c80b17332eb4496ee31578287b691bd646d71a
Reviewed-on: https://skia-review.googlesource.com/86742
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Hal Canary <halcanary@google.com>
|
|
Change-Id: Ic32eaec6cce1509f07e7cf610717d3b12d335c89
Reviewed-on: https://skia-review.googlesource.com/83921
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Add a real implementation for gm_knowledge.h This depends on
the presence of files in the form $GMK_DIR/foo/{max,min}.png
The implementation also writes out failures in a report directory.
Add a utility: experimental/make_gmkb which is a stand-alone
go executable that generates the foo/{max,min}.png data.
tools/skqp/README.md has instructions on running SkQP.
Also: add SkFontMgrPriv.h
Change-Id: Ibe1e9a7e7de143d14eee3877f5f2d2d8713f7f49
Reviewed-on: https://skia-review.googlesource.com/65380
Reviewed-by: Yuqian Li <liyuqian@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
Change-Id: I27df422a78209d60647db89a608fc31f9739a429
Reviewed-on: https://skia-review.googlesource.com/83361
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
|
|
This reverts commit b351817588baf94ac725e35f10ee1d5ff8178834.
Reason for revert: Relanding with android fixes
Original change's description:
> Revert "Add define to use customize vulkan header."
>
> This reverts commit a492eb0e1f08311bfa47f46c660144e7bc8a6c0e.
>
> Reason for revert: Breaking Android roll (b/c builds tools?)
>
> Original change's description:
> > Add define to use customize vulkan header.
> >
> > Bug: skia:
> > Change-Id: Ia87c81a54603a02b2f8f51a735bf173a49afe6c7
> > Reviewed-on: https://skia-review.googlesource.com/81121
> > Commit-Queue: Greg Daniel <egdaniel@google.com>
> > Reviewed-by: Brian Salomon <bsalomon@google.com>
>
> TBR=egdaniel@google.com,bsalomon@google.com
>
> Change-Id: I80a685bf88af909865f274ffc61686be57e8c313
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: skia:
> Reviewed-on: https://skia-review.googlesource.com/81740
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Commit-Queue: Brian Salomon <bsalomon@google.com>
TBR=egdaniel@google.com,bsalomon@google.com
Change-Id: Ia701e884bfa3799dc73002f892feb2ecafe9da12
Reviewed-on: https://skia-review.googlesource.com/81742
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Greg Daniel <egdaniel@google.com>
|
|
The NDK has deprecated mips and mips64:
https://developer.android.com/ndk/guides/abis.html
Might as well clean this up now while I remember.
Change-Id: Ie4b2334c75208082067cc16fe355d0349c7e0904
Reviewed-on: https://skia-review.googlesource.com/80560
Reviewed-by: Derek Sollenberger <djsollen@google.com>
|