Mark temp paths in SkBaseDevice::drawArc and ::drawDRRect as volatile

Mark temp paths in SkBaseDevice::drawArc and ::drawDRRect as volatile
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2261123004

Committed: https://skia.googlesource.com/skia/+/6bf233176f49ce1b810f0c48b23641ec148da7bd

[bsalomon, 2016-08-22 17:19:54]

Description was changed from

==========
Mark temp paths in SkBaseDevice::drawArc and ::drawDRRect as volatile
==========

to

==========
Mark temp paths in SkBaseDevice::drawArc and ::drawDRRect as volatile
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2261123004
==========

[bsalomon, 2016-08-22 17:20:53]

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

[bsalomon, 2016-08-22 17:20:57]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2016-08-22 17:21:03]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 17:21:08]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2016-08-22 23:20 UTC

[reed1, 2016-08-22 18:37:14]

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

[reed1, 2016-08-22 18:37:14]

Seems like volatile can/should be added everywhere in SkDevice that we make a
temp path, yes?

e.g. drawDDRect, drawAtlas, drawTextOnPath, 

also canvas does this for clips 3 times in all 3 clip variants

[robertphillips, 2016-08-22 18:45:37]

lgtm, although Mike's point that we should do this where ever we can is a good
one.

[commit-bot: I haz the power, 2016-08-22 18:46:37]

Description was changed from

==========
Mark temp paths in SkBaseDevice::drawArc and ::drawDRRect as volatile
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2261123004
==========

to

==========
Mark temp paths in SkBaseDevice::drawArc and ::drawDRRect as volatile
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2261123004

Committed:
https://skia.googlesource.com/skia/+/6bf233176f49ce1b810f0c48b23641ec148da7bd
==========

[commit-bot: I haz the power, 2016-08-22 18:46:38]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/6bf233176f49ce1b810f0c48b23641ec148da7bd

[bsalomon, 2016-08-22 18:47:45]

On 2016/08/22 18:45:37, robertphillips wrote:
> lgtm, although Mike's point that we should do this where ever we can is a good
> one.

Yes, but this completes the set in SkDevice.cpp. The others already do use
volatile.

[reed1, 2016-08-22 18:53:48]

On 2016/08/22 18:47:45, bsalomon wrote:
> On 2016/08/22 18:45:37, robertphillips wrote:
> > lgtm, although Mike's point that we should do this where ever we can is a
good
> > one.
> 
> Yes, but this completes the set in SkDevice.cpp. The others already do use
> volatile.

SkBaseDevice::drawDRRect ?

[reed1, 2016-08-22 19:00:13]

On 2016/08/22 18:53:48, reed1 wrote:
> On 2016/08/22 18:47:45, bsalomon wrote:
> > On 2016/08/22 18:45:37, robertphillips wrote:
> > > lgtm, although Mike's point that we should do this where ever we can is a
> good
> > > one.
> > 
> > Yes, but this completes the set in SkDevice.cpp. The others already do use
> > volatile.
> 
> SkBaseDevice::drawDRRect ?

Doh -- didn't see your #2, thanks.