diff options
| author |  Mike Klein <mtklein@chromium.org> | 2016-11-12 12:00:44 +0000 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-11-12 12:01:30 +0000 |
| commit | df8b2694d4cd000fcfa664aa07cd7fbde2bf6621 (patch) | |
| tree | d1dd52e54f79dc9ad3f7ed27d3019a80e48bc261 | |
| parent | 2563601fc2b0505619f905f86bd249ae630197cc (diff) | |
Revert "Annotate SkPath::fCovexity benign race in the other constructor too."
This reverts commit f978f12af553f899cfed7078ab63d3a71c6fbcb1.
Reason for revert: slowing the TSAN bot down to the point of timeouts.
Original change's description:
> 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>
>
TBR=mtklein@chromium.org,reed@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: I6e34b1fc80fd98387d79521ce24f4545b990eb25
Reviewed-on: https://skia-review.googlesource.com/4739
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Mike Klein <mtklein@chromium.org>
| -rw-r--r-- | src/core/SkPath.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/core/SkPath.cpp b/src/core/SkPath.cpp index c1276bf179..6e201bd204 100644 --- a/src/core/SkPath.cpp +++ b/src/core/SkPath.cpp @@ -148,8 +148,6 @@ 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() { |