From df8b2694d4cd000fcfa664aa07cd7fbde2bf6621 Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Sat, 12 Nov 2016 12:00:44 +0000 Subject: [PATCH] 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 > Commit-Queue: Mike Klein > 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 Reviewed-by: Mike Klein --- src/core/SkPath.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/core/SkPath.cpp b/src/core/SkPath.cpp index c1276bf..6e201bd 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() { -- 2.7.4