Remove coordinates from LatencyInfo objects.

Remove coordinates from LatencyInfo objects.

BUG=None

Review-Url: https://codereview.chromium.org/2804693002
Cr-Commit-Position: refs/heads/master@{#465991}
Committed: https://chromium.googlesource.com/chromium/src/+/27d3244a31074eeef019e8955c4ecc532fa11735

[tdresser, 2017-04-05 14:06:40]

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

[commit-bot: I haz the power, 2017-04-05 14:06:59]

Dry run: 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, 2017-04-05 14:21:26]

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

[commit-bot: I haz the power, 2017-04-05 14:21:27]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[tdresser, 2017-04-06 13:23:46]

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

[commit-bot: I haz the power, 2017-04-06 13:24:01]

Dry run: 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, 2017-04-06 13:26:11]

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

[commit-bot: I haz the power, 2017-04-06 13:26:12]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)

[tdresser, 2017-04-06 14:08:58]

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

[commit-bot: I haz the power, 2017-04-06 14:09:15]

Dry run: 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, 2017-04-06 14:58:02]

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

[commit-bot: I haz the power, 2017-04-06 14:58:03]

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

[tdresser, 2017-04-06 20:42:37]

tdresser@chromium.org changed reviewers:
+ mfomitchev@chromium.org

[mfomitchev, 2017-04-06 21:40:53]

https://codereview.chromium.org/2804693002/diff/60001/ui/latency/mojo/latency...
File ui/latency/mojo/latency_info_struct_traits.cc (left):

https://codereview.chromium.org/2804693002/diff/60001/ui/latency/mojo/latency...
ui/latency/mojo/latency_info_struct_traits.cc:246:
StructTraits<ui::mojom::LatencyInfoDataView,
You should update .h for struct traits as well.

[tdresser, 2017-04-07 12:17:18]

Yikes - shouldn't that have failed to link on one of the trybots?

[mfomitchev, 2017-04-07 15:16:01]

On 2017/04/07 12:17:18, tdresser wrote:
> Yikes - shouldn't that have failed to link on one of the trybots?

I don't think it tries to link if the method is unused.
LGTM

[tdresser, 2017-04-07 15:55:58]

The CQ bit was checked by tdresser@chromium.org

[commit-bot: I haz the power, 2017-04-07 15:56:23]

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, 2017-04-07 16:05:15]

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

[commit-bot: I haz the power, 2017-04-07 16:05:16]

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

[tdresser, 2017-04-07 16:09:07]

tdresser@chromium.org changed reviewers:
+ cpu@chromium.org, mbarbella@chromium.org

[tdresser, 2017-04-07 16:09:08]

+cpu@ for ipc review.
+mbarbella@ for clusterfuzz review.

[Martin Barbella, 2017-04-07 16:31:04]

ipc and ipc_fuzzer changes lgtm

[cpu_(ooo_6.6-7.5), 2017-04-15 01:53:10]

cpu@chromium.org changed reviewers:
- cpu@chromium.org

[cpu_(ooo_6.6-7.5), 2017-04-15 01:53:11]

I know nothing about tools/ipc_fuzzer.

[tdresser, 2017-04-18 11:59:56]

tdresser@chromium.org changed reviewers:
+ nasko@chromium.org

[tdresser, 2017-04-18 11:59:56]

+nasko@ for for ui/latency/ipc, ui/latency/mojo.

[nasko, 2017-04-19 17:31:44]

Code removal, yay! LGTM

[tdresser, 2017-04-19 17:58:01]

The CQ bit was checked by tdresser@chromium.org

[commit-bot: I haz the power, 2017-04-19 17:58:33]

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, 2017-04-19 18:02:52]

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

[commit-bot: I haz the power, 2017-04-19 18:02:53]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[tdresser, 2017-04-19 19:58:11]

The CQ bit was checked by tdresser@chromium.org

[tdresser, 2017-04-19 19:58:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
mbarbella@chromium.org, mfomitchev@chromium.org, nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/2804693002/#ps100001
(title: "Rebase.")

[commit-bot: I haz the power, 2017-04-19 19:58:36]

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, 2017-04-19 20:20:56]

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

[commit-bot: I haz the power, 2017-04-19 20:20:58]

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

[tdresser, 2017-04-20 12:23:29]

The CQ bit was checked by tdresser@chromium.org

[tdresser, 2017-04-20 12:23:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
mbarbella@chromium.org, mfomitchev@chromium.org, nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/2804693002/#ps120001
(title: "Fix mojo latencyInfo test")

[commit-bot: I haz the power, 2017-04-20 12:23:38]

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, 2017-04-20 13:26:15]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1492691009634220,
"parent_rev": "45d637552defb050205b9a1301b6c540a39dda0c", "commit_rev":
"27d3244a31074eeef019e8955c4ecc532fa11735"}

[commit-bot: I haz the power, 2017-04-20 13:26:25]

Description was changed from

==========
Remove coordinates from LatencyInfo objects.

BUG=None
==========

to

==========
Remove coordinates from LatencyInfo objects.

BUG=None

Review-Url: https://codereview.chromium.org/2804693002
Cr-Commit-Position: refs/heads/master@{#465991}
Committed:
https://chromium.googlesource.com/chromium/src/+/27d3244a31074eeef019e8955c4e...
==========

[commit-bot: I haz the power, 2017-04-20 13:26:27]

Committed patchset #7 (id:120001) as
https://chromium.googlesource.com/chromium/src/+/27d3244a31074eeef019e8955c4e...