diff options
| author |  Mike Klein <mtklein@chromium.org> | 2016-10-04 15:45:56 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-10-04 20:13:31 +0000 |
| commit | 511f2d79bf6ea0ee858a3468d3aa8dcbfca58648 (patch) | |
| tree | 50d33801abe2245cd147ca4bc477f6f205c0a494 /samplecode | |
| parent | 1834242ec6e3cd62669227d394bc79e1cd66dcfb (diff) | |
Make global use-analytic-AA bit threadsafe.
I also had to cut it down to just a global atomic bool... as a field in a global singleton accessed through instance(), it's very hard to make threadsafe.
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-Clang-GCE-CPU-AVX2-x86_64-Release-TSAN-Trybot
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2937
Change-Id: If80be987906dd521fbe644d1d0d577009f06d0e3
Reviewed-on: https://skia-review.googlesource.com/2937
Reviewed-by: Yuqian Li <liyuqian@google.com>
Commit-Queue: Yuqian Li <liyuqian@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'samplecode')
| -rw-r--r-- | samplecode/SampleApp.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/samplecode/SampleApp.cpp b/samplecode/SampleApp.cpp index 5d73162ae0..52e063d367 100644 --- a/samplecode/SampleApp.cpp +++ b/samplecode/SampleApp.cpp @@ -1802,8 +1802,7 @@ bool SampleWindow::onHandleChar(SkUnichar uni) { } break; case 'A': - GlobalAAConfig::getInstance().fUseAnalyticAA = - !GlobalAAConfig::getInstance().fUseAnalyticAA; + gSkUseAnalyticAA = !gSkUseAnalyticAA.load(); this->inval(nullptr); break; case 'B': |