Remove current implementation of frame timing events.

Remove current implementation of frame timing events.

The new draft [1] is significantly simpler and may not need frame IDs at
all. Per feedback from igrigorik@, it's probably easier to just revert
the current implementation until someone is ready to pick this up and
work on it again.

BUG=537864
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=haibinlu@chromium.org
R=esprehn@chromium.org,vmpstr@chromium.org

Committed: https://crrev.com/27d40dd253b7d2b51588c2322b75b32e185632cc
Cr-Commit-Position: refs/heads/master@{#388313}

[dcheng, 2016-04-19 00:12:21]

Description was changed from

==========
Remove current implementation of frame timing events.

The new draft [1] is significantly simpler and may not need frame IDs at
all. Per feedback from igrigorik@, it's probably easier to just revert
the current implementation until someone is ready to pick this up and
work on it again.

BUG=537864
==========

to

==========
Remove current implementation of frame timing events.

The new draft [1] is significantly simpler and may not need frame IDs at
all. Per feedback from igrigorik@, it's probably easier to just revert
the current implementation until someone is ready to pick this up and
work on it again.

BUG=537864
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[dcheng, 2016-04-19 00:14:01]

dcheng@chromium.org changed reviewers:
+ esprehn@chromium.org, vmpstr@chromium.org

[dcheng, 2016-04-19 00:14:02]

+vmpstr for cc changes
+esprehn for third_party/WebKit changes

[esprehn, 2016-04-19 00:21:03]

lgtm

[vmpstr, 2016-04-19 00:24:47]

This is awesome, lgtm. Thanks!

[dcheng, 2016-04-19 03:42:58]

+vmpstr, mind confirming the test updates to remote_channel_unittest.cc? I
trimmed a few more lines of code (saving the last sent and last received
BeginFrameArgs) that I believe should no longer be needed.

Also, I noticed there's a bunch of stuff guarded behind FrameTimingSUpport in
Blink: I'll try removing it in a followup patch.

[vmpstr, 2016-04-19 18:05:36]

still lgtm

[dcheng, 2016-04-19 19:41:55]

Description was changed from

==========
Remove current implementation of frame timing events.

The new draft [1] is significantly simpler and may not need frame IDs at
all. Per feedback from igrigorik@, it's probably easier to just revert
the current implementation until someone is ready to pick this up and
work on it again.

BUG=537864
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Remove current implementation of frame timing events.

The new draft [1] is significantly simpler and may not need frame IDs at
all. Per feedback from igrigorik@, it's probably easier to just revert
the current implementation until someone is ready to pick this up and
work on it again.

BUG=537864
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=haibinlu@chromium.org
==========

[dcheng, 2016-04-19 19:41:55]

dcheng@chromium.org changed reviewers:
+ haibinlu@chromium.org

[dcheng, 2016-04-19 19:42:10]

Description was changed from

==========
Remove current implementation of frame timing events.

The new draft [1] is significantly simpler and may not need frame IDs at
all. Per feedback from igrigorik@, it's probably easier to just revert
the current implementation until someone is ready to pick this up and
work on it again.

BUG=537864
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=haibinlu@chromium.org
==========

to

==========
Remove current implementation of frame timing events.

The new draft [1] is significantly simpler and may not need frame IDs at
all. Per feedback from igrigorik@, it's probably easier to just revert
the current implementation until someone is ready to pick this up and
work on it again.

BUG=537864
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=haibinlu@chromium.org
R=esprehn@chromium.org,vmpstr@chromium.org
==========

[dcheng, 2016-04-19 19:42:24]

The CQ bit was checked by dcheng@chromium.org

[dcheng, 2016-04-19 19:42:25]

TBRing //blimp OWNERS for simple code removal.

[dcheng, 2016-04-19 19:42:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
esprehn@chromium.org, vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1897123002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2016-04-19 19:43:05]

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

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

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-04-22 19:16:30]

Description was changed from

==========
Remove current implementation of frame timing events.

The new draft [1] is significantly simpler and may not need frame IDs at
all. Per feedback from igrigorik@, it's probably easier to just revert
the current implementation until someone is ready to pick this up and
work on it again.

BUG=537864
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=haibinlu@chromium.org
R=esprehn@chromium.org,vmpstr@chromium.org
==========

to

==========
Remove current implementation of frame timing events.

The new draft [1] is significantly simpler and may not need frame IDs at
all. Per feedback from igrigorik@, it's probably easier to just revert
the current implementation until someone is ready to pick this up and
work on it again.

BUG=537864
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=haibinlu@chromium.org
R=esprehn@chromium.org,vmpstr@chromium.org

Committed: https://crrev.com/27d40dd253b7d2b51588c2322b75b32e185632cc
Cr-Commit-Position: refs/heads/master@{#388313}
==========

[commit-bot: I haz the power, 2016-04-22 19:16:32]

Patchset 4 (id:??) landed as
https://crrev.com/27d40dd253b7d2b51588c2322b75b32e185632cc
Cr-Commit-Position: refs/heads/master@{#388313}

[haibinlu, 2016-04-25 17:29:49]

lgtm for blimp/ code