1340523 - “Video format or MIME type is not supported.” text area is not 350x18px

Status: RESOLVED INVALID

(Toolkit :: Video/Audio Controls, defect)

Description

[Iulia Cristescu, QA [:JuliaC] (away, please needinfo? [e-mail])]

Affected versions]:
- Latest Aurora 53.0a2 (2017-02-16) 
- Latest Nightly 54.0a1 (2017-02-16)

[Affected platforms]:
- Windows 10 x64
- Ubuntu 16.04 x64
- Mac OS X 10.12

[Steps to reproduce]:
1. Launch Firefox 
2. Download https://goo.gl/6XI3pn and drag it to Firefox
3. Disable e10s
4. Open the Developer Tools Settings, go to the "Advanced Settings" section and check the "Enable browser chrome and add-on debugging toolboxes" and "Enable remote debugging" settings in order to enable Browser Toolbox 
5. Open the Browser Toolbox ([Ctrl] + [Shift] + {Alt] + [I] / [Cmd] + [Opt] + [Shift] + [I])
6. Inspect the error text area size 

[Expected result]:
- The size displayed in the Browser Toolbox for the error text area is 350x18px (as is specified in the spec doc https://drive.google.com/file/d/0B4K8q1qWmtAvM3ZXVUlHRkdRVVU/view)

[Actual result]:
- The size displayed in the Browser Toolbox for the error text area is 297x20px (on Windows 10), 280x34px (on Mac OS) or 279x16px (on Ubuntu 16.04)  

[Regression range]:
- The issue is not a regression, because it is reproducible back to 53.0a1 (2016-11-19), when Visual refresh of media controls was implemented

Comment 1

[:Gijs (he/him)]

Can you add a screenshot? What's the actual effect here?

Comment 2

[Iulia Cristescu, QA [:JuliaC] (away, please needinfo? [e-mail])]

There is not a visual issue, but the problem is that the actual error text area hasn't the same size values like those required through the spec doc.

Comment 3

[:Gijs (he/him)]

(In reply to Iulia Cristescu, QA [:IuliaC] from comment #2)
> There is not a visual issue, but the problem is that the actual error text
> area hasn't the same size values like those required through the spec doc.

That doesn't sound like an issue to me... why do you think there's an issue?

Comment 4

[Ray Lin[:ralin]]

(In reply to Iulia Cristescu, QA [:IuliaC] from comment #2)
> There is not a visual issue, but the problem is that the actual error text
> area hasn't the same size values like those required through the spec doc.

Ah, I didn't notice the "text area" spec is still there, that should be deleted... I don't think it's reasonable to specify the region size of text, since it varies according to system default font and its outer container(video)'s size.

Comment 5

[:Gijs (he/him)]

(In reply to Ray Lin[:ralin] from comment #4)
> (In reply to Iulia Cristescu, QA [:IuliaC] from comment #2)
> > There is not a visual issue, but the problem is that the actual error text
> > area hasn't the same size values like those required through the spec doc.
> 
> Ah, I didn't notice the "text area" spec is still there, that should be
> deleted... I don't think it's reasonable to specify the region size of text,
> since it varies according to system default font and its outer
> container(video)'s size.

I concur. Going to close this bug out but leave the needinfo in case Iulia is seeing something that has user impact that we're not aware of.

Comment 6

[Iulia Cristescu, QA [:JuliaC] (away, please needinfo? [e-mail])]

Clear the needinfo, since the spec documentation has been updated (see https://bugzilla.mozilla.org/show_bug.cgi?id=1271765#c166).