Move atomicinc in GrTracing to behind branch

Move atomicinc in GrTracing to behind branch

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1571603003

Committed: https://skia.googlesource.com/skia/+/4897ff20fc68242d8a3469721f381f03987a0723

[joshua.litt, 2016-01-08 16:48:04]

Description was changed from

==========
Allow runtime toggling of tracing in Skia

BUG=skia:
==========

to

==========
Allow runtime toggling of tracing in Skia

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[joshualitt, 2016-01-08 16:49:11]

joshualitt@google.com changed reviewers:
+ egdaniel@google.com, joshualitt@google.com

[joshualitt, 2016-01-08 16:49:11]

Thoughts?

[egdaniel, 2016-01-08 17:00:39]

lgtm with caps cleanup

https://codereview.chromium.org/1571603003/diff/1/include/gpu/GrConfig.h
File include/gpu/GrConfig.h (left):

https://codereview.chromium.org/1571603003/diff/1/include/gpu/GrConfig.h#oldc...
include/gpu/GrConfig.h:192: #if !defined(GR_FORCE_GPU_TRACE_DEBUGGING)
this is also in GrCaps

[joshualitt, 2016-01-08 19:28:08]

On 2016/01/08 17:00:39, egdaniel wrote:
> lgtm with caps cleanup
> 
> https://codereview.chromium.org/1571603003/diff/1/include/gpu/GrConfig.h
> File include/gpu/GrConfig.h (left):
> 
>
https://codereview.chromium.org/1571603003/diff/1/include/gpu/GrConfig.h#oldc...
> include/gpu/GrConfig.h:192: #if !defined(GR_FORCE_GPU_TRACE_DEBUGGING)
> this is also in GrCaps

k, feel free to give it a once over if you like.  I will reland in a little bit
if I dont hear any objections.

[joshualitt, 2016-01-11 14:19:59]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2016-01-11 14:20:01]

The patchset sent to the CQ was uploaded after l-g-t-m from egdaniel@google.com
Link to the patchset: https://codereview.chromium.org/1571603003/#ps20001
(title: "updated")

[commit-bot: I haz the power, 2016-01-11 14:20:09]

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

[commit-bot: I haz the power, 2016-01-11 14:21:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-11 14:21:13]

Try jobs failed on following builders:
  skia_presubmit-Trybot on client.skia.fyi (JOB_FAILED,
http://build.chromium.org/p/client.skia.fyi/builders/skia_presubmit-Trybot/bu...)

[joshualitt, 2016-01-11 14:21:51]

Description was changed from

==========
Allow runtime toggling of tracing in Skia

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Allow runtime toggling of tracing in Skia

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[joshualitt, 2016-01-11 16:06:58]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2016-01-11 16:06:58]

The patchset sent to the CQ was uploaded after l-g-t-m from egdaniel@google.com
Link to the patchset: https://codereview.chromium.org/1571603003/#ps40001
(title: "tweaks")

[commit-bot: I haz the power, 2016-01-11 16:07:11]

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

[joshualitt, 2016-01-11 16:23:34]

The CQ bit was unchecked by joshualitt@google.com

[joshualitt, 2016-01-11 16:23:55]

Description was changed from

==========
Allow runtime toggling of tracing in Skia

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Move atomicinc in GrTracing to behind branch

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[joshualitt, 2016-01-11 16:24:02]

The CQ bit was checked by joshualitt@google.com

[commit-bot: I haz the power, 2016-01-11 16:24:10]

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

[commit-bot: I haz the power, 2016-01-11 16:30:49]

Description was changed from

==========
Move atomicinc in GrTracing to behind branch

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Move atomicinc in GrTracing to behind branch

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/4897ff20fc68242d8a3469721f381f03987a0723
==========

[commit-bot: I haz the power, 2016-01-11 16:30:50]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/4897ff20fc68242d8a3469721f381f03987a0723