72badbd99e2321bfbcb22f78218bbafa71af4698 |
|
16-Apr-2015 |
cdalton <cdalton@nvidia.com> |
Add ReverseIter to GrTRecorder BUG=skia: Review URL: https://codereview.chromium.org/1035083004
/external/skia/tests/GrTRecorderTest.cpp
|
77d77f446d1685cd21465627a747341c3c3665e4 |
|
21-Nov-2014 |
bsalomon <bsalomon@google.com> |
Add pop_back to GrTRecorder.h Review URL: https://codereview.chromium.org/750613002
/external/skia/tests/GrTRecorderTest.cpp
|
0ee2627026370bbb25e4efda490e14e468ac86fb |
|
07-Nov-2014 |
scroggo <scroggo@google.com> |
When running DM, write test failures to json. Add skiatest::Failure to keep track of data about a test failure. Reporter::reportFailed and ::onReportFailed now take Failure as a parameter. This allows the implementation to treat the failure as it wishes. Provide a helper to format the failure the same as prior to the change. Update the macros for calling reportFailed (REPORTER_ASSERT etc) to create a Failure object. Convert a direct call to reportFailed to the macro ERRORF. Write Failures to Json. Sample output when running dm on the dummy test crrev.com/705723004: { "test_results" : { "failures" : [ { "condition" : "0 > 3", "file_name" : "../../tests/DummyTest.cpp", "line_no" : 10, "message" : "" }, { "condition" : "false", "file_name" : "../../tests/DummyTest.cpp", "line_no" : 4, "message" : "" }, { "condition" : "1 == 3", "file_name" : "../../tests/DummyTest.cpp", "line_no" : 5, "message" : "I can too count!" }, { "condition" : "", "file_name" : "../../tests/DummyTest.cpp", "line_no" : 6, "message" : "seven is 7" }, { "condition" : "1 == 3", "file_name" : "../../tests/DummyTest.cpp", "line_no" : 14, "message" : "I can too count!" } ] } } Report all of the failures from one test. Previously, if one test had multiple failures, only one was reportered. e.g: Failures: test Dummy: ../../tests/DummyTest.cpp:6 seven is 7 test Dummy2: ../../tests/DummyTest.cpp:10 0 > 3 test Dummy3: ../../tests/DummyTest.cpp:14 I can too count!: 1 == 3 3 failures. Now, we get all the messages: Failures: test Dummy: ../../tests/DummyTest.cpp:4 false ../../tests/DummyTest.cpp:5 I can too count!: 1 == 3 ../../tests/DummyTest.cpp:6 seven is 7 test Dummy2: ../../tests/DummyTest.cpp:10 0 > 3 test Dummy3: ../../tests/DummyTest.cpp:14 I can too count!: 1 == 3 3 failures. (Note that we still state "3 failures" because 3 DM::Tasks failed.) BUG=skia:3082 BUG=skia:2454 Review URL: https://codereview.chromium.org/694703005
/external/skia/tests/GrTRecorderTest.cpp
|
6819df36446f6fdcbd17d83a72a03de46e6d0d2d |
|
15-Oct-2014 |
cdalton <cdalton@nvidia.com> |
Adds a GrTRecorder class that GrInOrderDrawBuffer uses to allocate all its commands interleaved in contiguous memory. GrTRecorder also supports extra data associated with objects, so we can store arrays inline without having to call malloc(). Committed: https://skia.googlesource.com/skia/+/360b6801cfd90485891d709e44cf395d527ba69e Review URL: https://codereview.chromium.org/628453002
/external/skia/tests/GrTRecorderTest.cpp
|
f439c77e9cfc13a4c31b8cb580b11d6ff09d4ed4 |
|
14-Oct-2014 |
mtklein <mtklein@google.com> |
Revert of Create a single command buffer for GrInOrderDrawBuffer (patchset #17 id:1240001 of https://codereview.chromium.org/628453002/) Reason for revert: Leaking memory: http://build.chromium.org/p/client.skia/builders/Test-Ubuntu12-ShuttleA-GTX550Ti-x86_64-Release-Valgrind/builds/9/steps/gm/logs/stdio Original issue's description: > Adds a GrTRecorder class that GrInOrderDrawBuffer uses to allocate > all its commands interleaved in contiguous memory. GrTRecorder also > supports extra data associated with objects, so we can store arrays > inline without having to call malloc(). > > Committed: https://skia.googlesource.com/skia/+/360b6801cfd90485891d709e44cf395d527ba69e TBR=bsalomon@google.com,reed@google.com,cdalton@nvidia.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/654863003
/external/skia/tests/GrTRecorderTest.cpp
|
360b6801cfd90485891d709e44cf395d527ba69e |
|
14-Oct-2014 |
cdalton <cdalton@nvidia.com> |
Adds a GrTRecorder class that GrInOrderDrawBuffer uses to allocate all its commands interleaved in contiguous memory. GrTRecorder also supports extra data associated with objects, so we can store arrays inline without having to call malloc(). Review URL: https://codereview.chromium.org/628453002
/external/skia/tests/GrTRecorderTest.cpp
|
07894c4d7dbd539583d7f51f05616e295eef2578 |
|
13-Oct-2014 |
mtklein <mtklein@google.com> |
Revert of Create a single command buffer for GrInOrderDrawBuffer (patchset #14 id:1050001 of https://codereview.chromium.org/628453002/) Reason for revert: New test failing on Android: http://build.chromium.org/p/client.skia.android/builders/Test-Android-Nexus7-Tegra3-Arm7-Release/builds/89/steps/dm/logs/stdio Original issue's description: > Adds a GrTBaseList class that GrInOrderDrawBuffer uses to allocate > all its commands interleaved in contiguous memory. GrTBaseList also > supports extra data associated with objects, so we can store arrays > inline without having to call malloc(). > > Committed: https://skia.googlesource.com/skia/+/47c844aaba81e5a29c773b660e1d6062c766d253 TBR=bsalomon@google.com,reed@google.com,cdalton@nvidia.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/652843002
/external/skia/tests/GrTRecorderTest.cpp
|
47c844aaba81e5a29c773b660e1d6062c766d253 |
|
13-Oct-2014 |
cdalton <cdalton@nvidia.com> |
Adds a GrTBaseList class that GrInOrderDrawBuffer uses to allocate all its commands interleaved in contiguous memory. GrTBaseList also supports extra data associated with objects, so we can store arrays inline without having to call malloc(). Review URL: https://codereview.chromium.org/628453002
/external/skia/tests/GrTRecorderTest.cpp
|