Closed Bug 1340523 Opened 7 years ago Closed 7 years ago

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

Categories

(Toolkit :: Video/Audio Controls, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID
Tracking Status
firefox51 --- unaffected
firefox52 --- unaffected
firefox53 --- affected
firefox54 --- affected

People

(Reporter: JuliaC, Unassigned)

References

Details

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
Can you add a screenshot? What's the actual effect here?
Flags: needinfo?(iulia.cristescu)
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.
Flags: needinfo?(iulia.cristescu)
(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?
Flags: needinfo?(iulia.cristescu)
(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.
(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.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Clear the needinfo, since the spec documentation has been updated (see https://bugzilla.mozilla.org/show_bug.cgi?id=1271765#c166).
Flags: needinfo?(iulia.cristescu)
You need to log in before you can comment on or make changes to this bug.