rewrite path.getBounds test to work-around compiler bug

rewrite path.getBounds test to work-around compiler bug

BUG=skia:
TBR=

Committed: https://skia.googlesource.com/skia/+/3b2fb981ab8d011077eb6de7eec5fe1f590906b3

[reed1, 2015-07-29 15:29:18]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2015-07-29 15:29:30]

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

[mtklein, 2015-07-29 15:34:14]

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

[mtklein, 2015-07-29 15:34:14]

https://codereview.chromium.org/1262143002/diff/1/tests/PathTest.cpp
File tests/PathTest.cpp (right):

https://codereview.chromium.org/1262143002/diff/1/tests/PathTest.cpp#newcode582
tests/PathTest.cpp:582: // As written these tests were failing on LLVM 4.2
MacMini Release mysteriously, so we've
I don't think this is an llvm-gcc-4.2 bot.  We haven't had those for >~ 1 year.

Did you mean to write Clang 3.2?  I'm surprised we're still able to compile with
something so old.  I'd have thought these bots were all using 3.4-3.6.

[reed1, 2015-07-29 15:35:21]

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

[reed1, 2015-07-29 15:35:22]

Eric typed > clang --version
and got what I said (I think)...

[commit-bot: I haz the power, 2015-07-29 15:37:16]

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

[mtklein, 2015-07-29 15:46:53]

Ah, so weird.  Apple always makes such strange lies about LLVM and Clang
versions.  I think they just peg them to XCode version.

E.g. my clang --version reads
  Apple LLVM version 7.0.0 (clang-700.0.57.2)
but it's actually LLVM + Clang ~3.7.

[reed1, 2015-07-29 15:47:54]

eric, can you paste the entitreity of your output?

[mtklein, 2015-07-29 15:53:18]

On 2015/07/29 15:47:54, reed1 wrote:
> eric, can you paste the entitreity of your output?

https://en.wikipedia.org/wiki/Xcode#Toolchain_versions

[borenet, 2015-07-29 16:25:17]

On 2015/07/29 15:53:18, mtklein wrote:
> On 2015/07/29 15:47:54, reed1 wrote:
> > eric, can you paste the entitreity of your output?
> 
> https://en.wikipedia.org/wiki/Xcode#Toolchain_versions

skiabot-macmini-10_8-001:~ chrome-bot$ clang --version
Apple LLVM version 4.2 (clang-425.0.24) (based on LLVM 3.2svn)
Target: x86_64-apple-darwin12.5.0
Thread model: posix

[mtklein, 2015-07-29 16:25:50]

On 2015/07/29 16:25:17, borenet wrote:
> On 2015/07/29 15:53:18, mtklein wrote:
> > On 2015/07/29 15:47:54, reed1 wrote:
> > > eric, can you paste the entitreity of your output?
> > 
> > https://en.wikipedia.org/wiki/Xcode#Toolchain_versions
> 
> skiabot-macmini-10_8-001:~ chrome-bot$ clang --version
> Apple LLVM version 4.2 (clang-425.0.24) (based on LLVM 3.2svn)
> Target: x86_64-apple-darwin12.5.0
> Thread model: posix

Thanks!  Wow, good thing we're upgrading these bots...