Fill SkRRect::fType proactively.

Fill SkRRect::fType proactively.

I found no external mentions of SkRRect::kUnknown_Type.

Motivating race found by TSAN:
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu13.10-GCE-NoGPU-x86_64-Release-TSAN-Trybot/builds/48/steps/dm/logs/stdio

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/8aacf2085673f6302a43d590d4d2890789b87192

[mtklein, 2014-12-18 21:05:45]

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

[reed1, 2014-12-18 21:11:01]

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

[reed1, 2014-12-18 21:11:01]

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

[reed1, 2014-12-18 21:11:02]

lgtm...

rob, you think we can afford to always compute this up-front?

[mtklein, 2014-12-18 21:14:01]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2014-12-18 21:14:31]

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

[commit-bot: I haz the power, 2014-12-18 21:14:32]

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 2014-12-19 03:14 UTC

[robertphillips, 2014-12-18 21:28:59]

lgtm - it looks like there's no way we can do anything with the RR w/o computing
the type so it shouldn't slow us down.

[commit-bot: I haz the power, 2014-12-18 21:30:10]

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

[mtklein, 2014-12-18 21:37:30]

On 2014/12/18 21:28:59, robertphillips wrote:
> it looks like there's no way we can do anything with the RR w/o computing
> the type so it shouldn't slow us down.

Agreed.