diff options
| author |  mtklein <mtklein@google.com> | 2016-03-10 11:31:27 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-03-10 11:31:27 -0800 |
| commit | 977c07dad26fbb56f6d4b5fc1ac1a324d28bd693 (patch) | |
| tree | 93016a3da1a43ada575ff1557716e97833f5a87d /tests/PictureTest.cpp | |
| parent | 7e6fcf890a1a63249136b8e6d9f4d5a606ef7508 (diff) | |
Revert of Use std::unique_ptr. (patchset #7 id:120001 of https://codereview.chromium.org/1780933003/ )
Reason for revert:
Now we remember! The problem was Clank:
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clang_dbg_recipe/builds/34329
Original issue's description:
> Use std::unique_ptr.
>
> TBR=reed@google.com
>
> Committed: https://skia.googlesource.com/skia/+/20c1e3abfc681771f73eb19fde7284196e028940
TBR=bungeman@google.com,mtklein@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1782973002
Diffstat (limited to 'tests/PictureTest.cpp')
| -rw-r--r-- | tests/PictureTest.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/PictureTest.cpp b/tests/PictureTest.cpp index a33deeb218..448e079958 100644 --- a/tests/PictureTest.cpp +++ b/tests/PictureTest.cpp @@ -1409,7 +1409,7 @@ DEF_TEST(Picture_preserveCullRect, r) { SkAutoTDelete<SkStream> rstream(wstream.detachAsStream()); SkAutoTUnref<SkPicture> deserializedPicture(SkPicture::CreateFromStream(rstream)); - REPORTER_ASSERT(r, deserializedPicture != nullptr); + REPORTER_ASSERT(r, SkToBool(deserializedPicture)); REPORTER_ASSERT(r, deserializedPicture->cullRect().left() == 1); REPORTER_ASSERT(r, deserializedPicture->cullRect().top() == 2); REPORTER_ASSERT(r, deserializedPicture->cullRect().right() == 3); |