diff options
| author |  Mike Klein <mtklein@chromium.org> | 2016-11-11 14:36:17 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-11-11 20:16:30 +0000 |
| commit | f978f12af553f899cfed7078ab63d3a71c6fbcb1 (patch) | |
| tree | c8ba88c68c333d213eea3953079c00fb0b122216 /src | |
| 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>
Diffstat (limited to 'src')
| -rw-r--r-- | src/core/SkPath.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/core/SkPath.cpp b/src/core/SkPath.cpp index 6e201bd204..c1276bf179 100644 --- a/src/core/SkPath.cpp +++ b/src/core/SkPath.cpp @@ -148,6 +148,8 @@ SkPath::SkPath(const SkPath& that) : 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() { |