Closed Bug 1210448 Opened 10 years ago Closed 9 years ago

Rewrite YouTubePuppeteer.player_stalled

Categories

(Testing Graveyard :: external-media-tests, defect)

Product:
Testing Graveyard
Component:
external-media-tests
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: impossibus, Unassigned)

Details

It's hard to detect a stall in YouTube videos because they may have ads in the middle, and the |current_time| attribute of the video (or of the player) describes whatever is playing at the moment (whether it's the main video or an ad). YouTubePuppeteer.player_stalled is out-of-date with how YouTube currently reports |current_time| and it does not take into account acceptable amounts of time spent on buffering as described by :rillian. VideoPuppeteer uses a different approach to stall detection; perhaps YouTubePuppeteer can adapt this to use with YouTube.
The media tests have been refactored such that this work is not needed any more.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.