Bug 24364

Summary: Add HTMLMediaElement canPlayType method
Product: WebKit Reporter: Eric Carlson <eric.carlson>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
proposed patch
koivisto: review+
revised patch koivisto: review+

Eric Carlson
Reported 2009-03-04 12:19:37 PST
Add the canPlayType method, define in 4.8.10.3 as: The canPlayType(type) method must return the string "no" if type is a type that the user agent knows it cannot render; it must return "probably" if the user agent is confident that the type represents a media resource that it can render if used in with this audio or video element; and it must return "maybe" otherwise. Implementors are encouraged to return "maybe" unless the type can be confidently established as being supported or not. Generally, a user agent should never return "probably" if the type doesn't have a codecs parameter.
Attachments
proposed patch (5.63 KB, patch)
2009-03-04 12:23 PST, Eric Carlson 		koivisto: review+
DetailsFormatted DiffDiff
revised patch (12.84 KB, patch)
2009-03-04 13:58 PST, Eric Carlson 		koivisto: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Eric Carlson
Comment 1 2009-03-04 12:23:32 PST
Created attachment 28280 [details] proposed patch
Antti Koivisto
Comment 2 2009-03-04 12:43:10 PST
Comment on attachment 28280 [details] proposed patch r=me
Eric Carlson
Comment 3 2009-03-04 13:58:36 PST
Created attachment 28283 [details] revised patch Patch revised so MediaPlayer::supportsType now takes a ContentType instead of separate type and codecs parameters.
Antti Koivisto
Comment 4 2009-03-04 14:04:21 PST
Comment on attachment 28283 [details] revised patch r=me > + static MediaPlayer::SupportsType supportsType(ContentType contentType); ContentType can be const here
Eric Carlson
Comment 5 2009-03-04 14:30:39 PST
Committed revision 41432.
Note You need to log in before you can comment on or make changes to this bug.