Fix QuadF::ContainsPoint and gfx::CrossProduct on arm64

Fix QuadF::ContainsPoint and gfx::CrossProduct on arm64

These methods require the use of doubles to not give incorrect
answers. Tests were failing on arm64 due to precision mistakes
with floats.

BUG=662561, 662556
Committed: https://crrev.com/1f740dbd1a15d3a904f40a4175eaad8c536becd7
Cr-Commit-Position: refs/heads/master@{#432685}

[danakj, 2016-11-16 01:48:44]

Description was changed from

==========
quadtest

BUG=662561
==========

to

==========
Reenable all of QuadTest.ContainsPoint for arm

BUG=662561
==========

[danakj, 2016-11-16 22:27:59]

Description was changed from

==========
Reenable all of QuadTest.ContainsPoint for arm

BUG=662561
==========

to

==========
Fix QuadF::ContainsPoint and gfx::CrossProduct on arm64

These methods require the use of doubles to not give incorrect
answers. Tests were failing on arm64 due to precision mistakes
with floats.

BUG=662561
==========

[danakj, 2016-11-16 22:27:59]

danakj@chromium.org changed reviewers:
+ enne@chromium.org

[danakj, 2016-11-16 22:28:37]

Description was changed from

==========
Fix QuadF::ContainsPoint and gfx::CrossProduct on arm64

These methods require the use of doubles to not give incorrect
answers. Tests were failing on arm64 due to precision mistakes
with floats.

BUG=662561
==========

to

==========
Fix QuadF::ContainsPoint and gfx::CrossProduct on arm64

These methods require the use of doubles to not give incorrect
answers. Tests were failing on arm64 due to precision mistakes
with floats.

BUG=662561,662556
==========

[danakj, 2016-11-16 22:28:40]

The CQ bit was checked by danakj@chromium.org to run a CQ dry run

[danakj, 2016-11-16 22:28:48]

The CQ bit was unchecked by danakj@chromium.org

[danakj, 2016-11-16 22:29:01]

The CQ bit was checked by danakj@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-16 22:29:37]

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

[Nico, 2016-11-16 22:36:15]

thakis@chromium.org changed reviewers:
+ thakis@chromium.org

[Nico, 2016-11-16 22:36:16]

lgtm, thanks!

https://codereview.chromium.org/2504583003/diff/20001/ui/gfx/geometry/vector3...
File ui/gfx/geometry/vector3d_unittest.cc (right):

https://codereview.chromium.org/2504583003/diff/20001/ui/gfx/geometry/vector3...
ui/gfx/geometry/vector3d_unittest.cc:230: SCOPED_TRACE(i);
remove?

[danakj, 2016-11-16 22:36:48]

https://codereview.chromium.org/2504583003/diff/20001/ui/gfx/geometry/vector3...
File ui/gfx/geometry/vector3d_unittest.cc (right):

https://codereview.chromium.org/2504583003/diff/20001/ui/gfx/geometry/vector3...
ui/gfx/geometry/vector3d_unittest.cc:230: SCOPED_TRACE(i);
On 2016/11/16 22:36:15, Nico (ooo sick Nov 16) wrote:
> remove?

No, I think this should be here. It will report if a failure happened on which
case it happened.

[enne (OOO), 2016-11-16 22:38:39]

lgtm

[commit-bot: I haz the power, 2016-11-16 23:50:34]

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

[commit-bot: I haz the power, 2016-11-16 23:50:35]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[danakj, 2016-11-16 23:51:03]

The CQ bit was unchecked by danakj@chromium.org

[danakj, 2016-11-16 23:51:07]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2016-11-16 23:52:21]

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-11-17 01:42:39]

Description was changed from

==========
Fix QuadF::ContainsPoint and gfx::CrossProduct on arm64

These methods require the use of doubles to not give incorrect
answers. Tests were failing on arm64 due to precision mistakes
with floats.

BUG=662561,662556
==========

to

==========
Fix QuadF::ContainsPoint and gfx::CrossProduct on arm64

These methods require the use of doubles to not give incorrect
answers. Tests were failing on arm64 due to precision mistakes
with floats.

BUG=662561,662556
==========

[commit-bot: I haz the power, 2016-11-17 01:42:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 01:48:25]

Description was changed from

==========
Fix QuadF::ContainsPoint and gfx::CrossProduct on arm64

These methods require the use of doubles to not give incorrect
answers. Tests were failing on arm64 due to precision mistakes
with floats.

BUG=662561,662556
==========

to

==========
Fix QuadF::ContainsPoint and gfx::CrossProduct on arm64

These methods require the use of doubles to not give incorrect
answers. Tests were failing on arm64 due to precision mistakes
with floats.

BUG=662561,662556

Committed: https://crrev.com/1f740dbd1a15d3a904f40a4175eaad8c536becd7
Cr-Commit-Position: refs/heads/master@{#432685}
==========

[commit-bot: I haz the power, 2016-11-17 01:48:26]

Patchset 2 (id:??) landed as
https://crrev.com/1f740dbd1a15d3a904f40a4175eaad8c536becd7
Cr-Commit-Position: refs/heads/master@{#432685}