commit | f978f12af553f899cfed7078ab63d3a71c6fbcb1 | [log] [tgz] |
---|---|---|
author | Mike Klein <mtklein@chromium.org> | Fri Nov 11 14:36:17 2016 -0500 |
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | Fri Nov 11 20:16:30 2016 +0000 |
tree | c8ba88c68c333d213eea3953079c00fb0b122216 | |
parent | ce3fe23c04e93d2aa8d4c09c0c99088207edd509 [diff] |
Annotate SkPath::fCovexity benign race in the other constructor too. I overlooked the copy constructor last time. BUG=skia:5776 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4730 Change-Id: I30bc89e1472dd48badf57664cfae8899f44bca9e Reviewed-on: https://skia-review.googlesource.com/4730 Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Mike Klein <mtklein@chromium.org>
diff --git a/src/core/SkPath.cpp b/src/core/SkPath.cpp index 6e201bd..c1276bf 100644 --- a/src/core/SkPath.cpp +++ b/src/core/SkPath.cpp
@@ -148,6 +148,8 @@ : fPathRef(SkRef(that.fPathRef.get())) { this->copyFields(that); SkDEBUGCODE(that.validate();) + AnnotateBenignRaceSized(__FILE__, __LINE__, &fConvexity, sizeof(fConvexity), + "We've tried a few times to make this thread safe, and failed."); } SkPath::~SkPath() {