Closed
Bug 560059
Opened 15 years ago
Closed 14 years ago
default video dimensions do not follow standard
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
People
(Reporter: silviapfeiffer1, Assigned: Ms2ger)
References
Details
(Keywords: regression)
Attachments
(1 file)
|
(deleted),
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_2; en-us) AppleWebKit/531.22.7 (KHTML, like Gecko) Version/4.0.5 Safari/531.22.7
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3
I noticed that http://www.whatwg.org/specs/web-apps/current-work/multipage/video.html#video says that the default dimensions of <video> are 300px by 150px, but Firefox seems to do 300px by 28px when loading of video src and of poster fail.
Example page:
<!DOCTYPE html>
<html lang="en">
<head>
<title>Test</title>
</head>
<body>
<video src="HelloWorld.ogv">
</video>
</body>
</html>
Reproducible: Always
Steps to Reproduce:
1. Use above example page and load into Firefox (I did it from localhost)
Actual Results:
Dimensions are 300px x 28px
Expected Results:
Dimensions should be 300px x 150px according to standard
|
||
Updated•15 years ago
|
Component: General → Video/Audio
Product: Firefox → Core
QA Contact: general → video.audio
|
||
Comment 1•15 years ago
|
||
This seems to be fallout from bug 449156: the intrinsic height seems to be set to the height of the controls instead of to 150. We should be treating the "not a video element" and "video element but has no video data" cases differently in this code instead of taking the same codepath for both....
|
|
||
Comment 2•15 years ago
|
||
Not sure whether we should fix this on 1.9.2, but definitely should on trunk.
blocking1.9.2: --- → ?
blocking2.0: --- → ?
|
||
Comment 3•15 years ago
|
||
Not sure it's essential to fix on 1.9.2 either, but if the patch is an easy backport, we'd likely take it.
blocking1.9.2: ? → -
status1.9.2:
--- → wanted
blocking2.0: ? → beta1+
|
Assignee | |
Comment 4•14 years ago
|
||
Attachment #451091 -
Flags: review?(roc) → review+
|
|
Assignee | |
Updated•14 years ago
|
Keywords: checkin-needed
|
||
Comment 5•14 years ago
|
||
|
|
Assignee | |
Updated•14 years ago
|
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•