Fix extraneous double-handled event errors.

Fix extraneous double-handled event errors.

The user model will frequently report double-handled event errors when
there are two or more animations overlapping, because the frames in the
animation intervals are considered associatedEvents for those animations,
so if the animations overlap then the frame will be the associatedEvent
of two or more different animations. Since this is expected, we will
no longer report a double-handled event error on a frame event (but will
still report these errors on other types of events)

BUG=catapult:#2664

Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/8be763db2940432ebf6f6d188894e2af845bbf1a

[alexandermont, 2016-08-11 23:17:13]

alexandermont@chromium.org changed reviewers:
+ charliea@chromium.org

[charliea (OOO until 10-5), 2016-08-12 13:35:22]

charliea@chromium.org changed reviewers:
+ benjhayden@chromium.org

[charliea (OOO until 10-5), 2016-08-12 13:35:22]

lgtm, but deferring to benjhayden@ for final approval because he knows this code
a lot better

[benjhayden, 2016-08-15 22:14:55]

https://codereview.chromium.org/2239953002/diff/1/tracing/tracing/importer/fi...
File tracing/tracing/importer/find_input_expectations.html (right):

https://codereview.chromium.org/2239953002/diff/1/tracing/tracing/importer/fi...
tracing/tracing/importer/find_input_expectations.html:74: var FRAME_EVENT_TITLE
= 'BenchmarkInstrumentation::ImplThreadRenderingStats';
Can you reference tr.model.helpers.IMPL_RENDERING_STATS instead of duplicating
this string?

[alexandermont, 2016-08-15 23:08:55]

https://codereview.chromium.org/2239953002/diff/1/tracing/tracing/importer/fi...
File tracing/tracing/importer/find_input_expectations.html (right):

https://codereview.chromium.org/2239953002/diff/1/tracing/tracing/importer/fi...
tracing/tracing/importer/find_input_expectations.html:74: var FRAME_EVENT_TITLE
= 'BenchmarkInstrumentation::ImplThreadRenderingStats';
On 2016/08/15 at 22:14:55, benjhayden wrote:
> Can you reference tr.model.helpers.IMPL_RENDERING_STATS instead of duplicating
this string?

Done

[benjhayden, 2016-08-15 23:09:56]

lgtm

[alexandermont, 2016-08-17 22:53:57]

The CQ bit was checked by alexandermont@chromium.org

[alexandermont, 2016-08-17 22:53:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
charliea@chromium.org
Link to the patchset: https://codereview.chromium.org/2239953002/#ps20001
(title: "fix reference")

[commit-bot: I haz the power, 2016-08-17 22:54:01]

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-08-18 00:47:56]

Description was changed from

==========
Fix extraneous double-handled event errors.

The user model will frequently report double-handled event errors when
there are two or more animations overlapping, because the frames in the
animation intervals are considered associatedEvents for those animations,
so if the animations overlap then the frame will be the associatedEvent
of two or more different animations. Since this is expected, we will
no longer report a double-handled event error on a frame event (but will
still report these errors on other types of events)

BUG=catapult:#2664
==========

to

==========
Fix extraneous double-handled event errors.

The user model will frequently report double-handled event errors when
there are two or more animations overlapping, because the frames in the
animation intervals are considered associatedEvents for those animations,
so if the animations overlap then the frame will be the associatedEvent
of two or more different animations. Since this is expected, we will
no longer report a double-handled event error on a frame event (but will
still report these errors on other types of events)

BUG=catapult:#2664

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-08-18 00:47:57]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...