init atomic bool in SkImage_Gpu

init atomic bool in SkImage_Gpu

BUG=chromium:516860
TBR=reed@google.com

NOTREECHECKS=true

Committed: https://skia.googlesource.com/skia/+/1cd63111d599f8c690c56b43502b6d9f3bee1ef2

[bsalomon, 2015-08-05 13:44:46]

bsalomon@google.com changed reviewers:
+ reed@google.com

[bsalomon, 2015-08-05 13:45:03]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2015-08-05 13:45:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1269243004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1269243004/1

[mtklein, 2015-08-05 13:54:52]

mtklein@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2015-08-05 13:54:53]

Were we assuming SkAtomic's default constructor was
  SkAtomic() : fVal((T)0) {}
?

That seems like a fine defensive change to make too.

(lgtm)

[mtklein, 2015-08-05 13:55:58]

On 2015/08/05 13:54:53, mtklein wrote:
> Were we assuming SkAtomic's default constructor was
>   SkAtomic() : fVal((T)0) {}
> ?
> 
> That seems like a fine defensive change to make too.
> 
> (lgtm)

Actually I think just
   SkAtomic() : fVal(0) {}
would work fine.

[mtklein, 2015-08-05 13:58:03]

The CQ bit was unchecked by mtklein@google.com

[mtklein, 2015-08-05 13:58:05]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2015-08-05 13:58:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1269243004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1269243004/1

[commit-bot: I haz the power, 2015-08-05 13:58:42]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/1cd63111d599f8c690c56b43502b6d9f3bee1ef2