Closed Bug 1343407 Opened 3 years ago Closed 3 years ago

Video Puppeteer stall detection accesses non-existent attribute

Categories

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

enhancement
Not set

Tracking

(firefox54 fixed)

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: bryce, Assigned: bryce)

Details

Attachments

(1 file)

When detecting a stall video puppeteer will log information about the stalled video, however, there's a typo in the logging which attempts to access 'self._last_seen_video_state.video_uri', this should instead be 'self._last_seen_video_state.base_uri'.
Assignee: nobody → bvandyk
Comment on attachment 8842250 [details]
Bug 1343407 - Fix attribute name video_uri -> base_uri when logging stalls.

https://reviewboard.mozilla.org/r/116124/#review117664
Attachment #8842250 - Flags: review?(mjzffr) → review+
Pushed by bvandyk@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c6a4bbb14394
Fix attribute name video_uri -> base_uri when logging stalls. r=maja_zf
https://hg.mozilla.org/mozilla-central/rev/c6a4bbb14394
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.