diff options
author | bsalomon <bsalomon@google.com> | 2015-07-23 12:22:19 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-07-23 12:22:19 -0700 |
commit | a449779a330123062e752228de8e3f84593a6fd1 (patch) | |
tree | 0a95d9c657a7f04db83ce87877f99bebc9da43ec /tests | |
parent | a56884c17fb02ae416ce875da3b1eddc7e6d81ae (diff) |
Remove test of illegal behavior that modifies a texture backing an SkImage.
Review URL: https://codereview.chromium.org/1244143004
Diffstat (limited to 'tests')
-rw-r--r-- | tests/SurfaceTest.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tests/SurfaceTest.cpp b/tests/SurfaceTest.cpp index b1e233fa3a..3d6e59ed35 100644 --- a/tests/SurfaceTest.cpp +++ b/tests/SurfaceTest.cpp @@ -979,8 +979,12 @@ DEF_GPUTEST(SkImage_NewFromTexture, reporter, factory) { sk_memset32(storage, expected1, w * h); tex->writePixels(0, 0, w, h, kSkia8888_GrPixelConfig, storage, GrContext::kFlushWrites_PixelOp); - // We expect the ref'd image to see the new color, but cpy'd one should still see the old color + // The cpy'd one should still see the old color +#if 0 + // There is no guarantee that refImg sees the new color. We are free to have made a copy. Our + // write pixels call violated the contract with refImg and refImg is now undefined. test_image_color(reporter, refImg, expected1); +#endif test_image_color(reporter, cpyImg, expected0); // Now exercise the release proc |