New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 706004 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

The video animation is clearly visible around 33s in the attached trace: just look for 'VideoPlayback' below the 'Mousewheel Animation' RAIL stage.

Project Member Reported by charliea@chromium.org, Mar 28 2017

Issue description

The video animation is clearly visible around 33s in the this trace (https://console.developers.google.com/m/cloudstorage/b/chrome-telemetry-output/o/trace-file-id_49-2017-03-22_01-54-17-98522.html): just look for 'VideoPlayback' below the 'Mousewheel Animation' RAIL stage. That should clearly be showing up as a Video expectation, but it's not.
 

Comment 1 Deleted

It looks like the ProtoExpectations are indeed getting added for the VideoPlayback, but the video proto expectations are somehow getting filtered out in mergeIntersectingAnimations. The bug has to be somewhere in that code path, because multiple video playback expectations should never get filtered down to zero of them (which is what's happening).
Interesting: the way that we identify a video protoexpectation is that we call pe.containsTypeNames([INITIATOR_TYPE.VIDEO]), which seems to be returning false even for ProtoExpectations. I wonder why?
It looks like the way that we were detecting whether something was a video expectation when determining whether to merge intersecting expectations:

var isVideo = pe.containsTypeNames([INITIATOR_TYPE.VIDEO]);

always returned false, because INITIATOR_TYPE.VIDEO isn't a type name, its the initiator type.

More broadly, we should always be able to just look at the initiator type of the proto expectation to determine what type of proto expectation it is. I'm going to send out a quick CL fixing this.
This was fixed in https://codereview.chromium.org/2778993004/
Status: Fixed (was: Assigned)

Sign in to add a comment