Bug 74738

Summary: Audio file in video element has a size of 0x0 .
Product: WebKit Reporter: Yael <yael>
Component: MediaAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, eric.carlson, kenneth, mitz, simon.fraser, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Screenshot of the bug in Safari.
none
Patch. none

Yael
Reported 2011-12-16 12:23:57 PST
Created attachment 119650 [details] Screenshot of the bug in Safari. When the source of a video element has only audio, the intrinsic size of the video element is 0x0. See the attached screenshot. This is not happening in Firefox and in Opera.
Attachments
Screenshot of the bug in Safari. (27.15 KB, image/png)
2011-12-16 12:23 PST, Yael 		no flags
Details
Patch. (3.95 KB, patch)
2011-12-16 12:34 PST, Yael 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yael
Comment 1 2011-12-16 12:34:51 PST
Created attachment 119652 [details] Patch. When the source of a video element has audio only, the intrinsic size of the video should not be 0x0. Instead, it should be the same as as no media was loaded.
WebKit Review Bot
Comment 2 2011-12-16 13:03:22 PST
Comment on attachment 119652 [details] Patch. Clearing flags on attachment: 119652 Committed r103098: <http://trac.webkit.org/changeset/103098>
WebKit Review Bot
Comment 3 2011-12-16 13:03:26 PST
All reviewed patches have been landed. Closing bug.
Alexey Proskuryakov
Comment 4 2011-12-16 15:08:35 PST
Is compatibility with Firefox and Opera the only rationale for this change? In the future, please CC authors of tests when changing those.
Yael
Comment 5 2011-12-16 17:15:37 PST
(In reply to comment #4) > Is compatibility with Firefox and Opera the only rationale for this change? > You can see from the attached screenshot that it did not make sense to keep the behavior as it was. > In the future, please CC authors of tests when changing those. Sure, will do.
Yael
Comment 6 2011-12-16 17:20:04 PST
Filed https://www.w3.org/Bugs/Public/show_bug.cgi?id=15231 to also make this more clear in the spec.
Note You need to log in before you can comment on or make changes to this bug.