1873596 - Use `mDisplay` to determine video's resolution for telemetry

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, task, P1)

Description

[Alastor Wu [:alwu][PTO, back on 12/15]]

We should use mDisplay instead of mImage here . The video height we want to record is actually intrinsic height, which is the height in mDisplay.

Comment 1

[Alastor Wu [:alwu][PTO, back on 12/15]]

Attachment: Bug 1873596 - use 'mDisplay' to determine video's resolution for telemetry.

The video height we want to record is actually intrinsic
height [1] which is the height in mDisplay [2].

[1] https://developer.mozilla.org/en-US/docs/Web/API/HTMLVideoElement/videoHeight#about_intrinsic_width_and_height
[2] https://searchfox.org/mozilla-central/rev/7fc94186f1a759790c56e41a90ab350d78dbda30/dom/html/HTMLVideoElement.cpp#221

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fa7ad042dcfe
use 'mDisplay' to determine video's resolution for telemetry. r=media-playback-reviewers,padenot

Comment 3

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/fa7ad042dcfe