Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org

Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Committed: https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614
Cr-Original-Commit-Position: refs/heads/master@{#398278}
Cr-Commit-Position: refs/heads/master@{#398328}

[Fady Samuel, 2016-06-06 22:01:01]

fsamuel@chromium.org changed reviewers:
+ sadrul@chromium.org

[Fady Samuel, 2016-06-06 23:02:08]

fsamuel@chromium.org changed reviewers:
+ tsepez@chromium.org

[Fady Samuel, 2016-06-06 23:02:12]

+tsepez@ for ipc / mojo dirs

[Tom Sepez, 2016-06-06 23:08:02]

lgtm

[sadrul, 2016-06-07 03:06:54]

Thanks! lgtm

[Fady Samuel, 2016-06-07 03:11:15]

Description was changed from

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
==========

to

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org
==========

[Fady Samuel, 2016-06-07 03:12:08]

fsamuel@chromium.org changed reviewers:
+ ben@chromium.org

[Fady Samuel, 2016-06-07 03:12:08]

+ben@: TBR'ing for the content part of this cleanup! Thanks

[Fady Samuel, 2016-06-07 03:12:17]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2016-06-07 03:12:35]

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

[commit-bot: I haz the power, 2016-06-07 03:14:38]

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

[commit-bot: I haz the power, 2016-06-07 03:14:39]

Try jobs failed on following builders:
  ios-device on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)

[Fady Samuel, 2016-06-07 03:46:51]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-06-07 03:46:51]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2039853004/#ps10001
(title: "Rebased")

[commit-bot: I haz the power, 2016-06-07 03:47:10]

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

[commit-bot: I haz the power, 2016-06-07 04:13:18]

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

[commit-bot: I haz the power, 2016-06-07 04:13:19]

Try jobs failed on following builders:
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Fady Samuel, 2016-06-07 04:36:19]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-06-07 04:36:19]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2039853004/#ps30001
(title: "Fix broken fuzzer")

[commit-bot: I haz the power, 2016-06-07 04:36:34]

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

[commit-bot: I haz the power, 2016-06-07 04:55:36]

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

[commit-bot: I haz the power, 2016-06-07 04:55:37]

Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Fady Samuel, 2016-06-07 04:58:31]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-06-07 04:58:31]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2039853004/#ps50001
(title: "Remove extraneous fuzztraits")

[commit-bot: I haz the power, 2016-06-07 04:58:51]

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

[commit-bot: I haz the power, 2016-06-07 05:23:26]

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

[commit-bot: I haz the power, 2016-06-07 05:23:27]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Fady Samuel, 2016-06-07 10:25:44]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-06-07 10:25:45]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2039853004/#ps70001
(title: "Add gfx_ipc_geometry dep in gyp")

[commit-bot: I haz the power, 2016-06-07 10:25:58]

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

[Fady Samuel, 2016-06-07 10:55:19]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-06-07 10:55:20]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2039853004/#ps90001
(title: "Add geometry ipc dep to unit test")

[commit-bot: I haz the power, 2016-06-07 10:55:34]

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

[commit-bot: I haz the power, 2016-06-07 11:56:58]

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

[commit-bot: I haz the power, 2016-06-07 11:56:59]

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

[Fady Samuel, 2016-06-07 12:03:09]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2016-06-07 12:03:30]

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

[commit-bot: I haz the power, 2016-06-07 12:40:00]

Description was changed from

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org
==========

to

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org
==========

[commit-bot: I haz the power, 2016-06-07 12:40:02]

Committed patchset #6 (id:90001)

[commit-bot: I haz the power, 2016-06-07 12:41:41]

Description was changed from

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org
==========

to

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org

Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Cr-Commit-Position: refs/heads/master@{#398278}
==========

[commit-bot: I haz the power, 2016-06-07 12:41:43]

Patchset 6 (id:??) landed as
https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Cr-Commit-Position: refs/heads/master@{#398278}

[Geoff Lang, 2016-06-07 15:42:30]

A revert of this CL (patchset #6 id:90001) has been created in
https://codereview.chromium.org/2048673002/ by geofflang@chromium.org.

The reason for reverting is: This appears to have broken the Windows GPU debug
builder.  First broken build:
https://build.chromium.org/p/chromium.gpu.fyi/builders/GPU%20Win%20Builder%20....

[Fady Samuel, 2016-06-07 16:09:55]

Description was changed from

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org

Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Cr-Commit-Position: refs/heads/master@{#398278}
==========

to

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org

Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Cr-Commit-Position: refs/heads/master@{#398278}
==========

[Fady Samuel, 2016-06-07 16:10:27]

On 2016/06/07 15:42:30, Geoff Lang wrote:
> A revert of this CL (patchset #6 id:90001) has been created in
> https://codereview.chromium.org/2048673002/ by mailto:geofflang@chromium.org.
> 
> The reason for reverting is: This appears to have broken the Windows GPU debug
> builder.  First broken build:
>
https://build.chromium.org/p/chromium.gpu.fyi/builders/GPU%20Win%20Builder%20....

Looks like I was missing a dep. Fixed. Relanding.

[Fady Samuel, 2016-06-07 16:10:36]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-06-07 16:10:37]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2039853004/#ps110001
(title: "Added missing gyp dep")

[commit-bot: I haz the power, 2016-06-07 16:32:22]

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

[commit-bot: I haz the power, 2016-06-07 17:49:43]

Description was changed from

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org

Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Cr-Commit-Position: refs/heads/master@{#398278}
==========

to

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org

Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Cr-Commit-Position: refs/heads/master@{#398278}
==========

[commit-bot: I haz the power, 2016-06-07 17:49:45]

Committed patchset #7 (id:110001)

[commit-bot: I haz the power, 2016-06-07 17:50:56]

Description was changed from

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org

Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Cr-Commit-Position: refs/heads/master@{#398278}
==========

to

==========
Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG=611802
TBR=ben@chromium.org

Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Committed: https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614
Cr-Original-Commit-Position: refs/heads/master@{#398278}
Cr-Commit-Position: refs/heads/master@{#398328}
==========

[commit-bot: I haz the power, 2016-06-07 17:50:57]

Patchset 7 (id:??) landed as
https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614
Cr-Commit-Position: refs/heads/master@{#398328}