Bug 92336 - Add a MediaPlayer API to retrieve the description of the current media engine.
Summary: Add a MediaPlayer API to retrieve the description of the current media engine.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jer Noble
URL:
Keywords:
Depends on:
Blocks: 92341
  Show dependency treegraph
 
Reported: 2012-07-25 23:16 PDT by Jer Noble
Modified: 2012-07-26 08:10 PDT (History)
11 users (show)

See Also:


Attachments
Patch (7.59 KB, patch)
2012-07-25 23:55 PDT, Jer Noble
eric.carlson: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jer Noble 2012-07-25 23:16:52 PDT
Add a MediaPlayer API to retrieve the description of the current media engine.
Comment 1 Jer Noble 2012-07-25 23:55:04 PDT
Created attachment 154555 [details]
Patch
Comment 2 Eric Carlson 2012-07-26 05:41:57 PDT
Comment on attachment 154555 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=154555&action=review

How about platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.h? You could also change Source/WebKit/chromium/src/WebMediaPlayerClientImpl.h, but it is probably better to just file a bug.

> Source/WebCore/platform/graphics/qt/MediaPlayerPrivateQt.h:134
> +    virtual String engineDescription() const { return ""; }

Did you mean to leave this blank?
Comment 3 Jer Noble 2012-07-26 07:56:30 PDT
(In reply to comment #2)
> (From update of attachment 154555 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=154555&action=review
> 
> How about platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.h? 

Yep, missed that one. I'll add it.

> You could also change Source/WebKit/chromium/src/WebMediaPlayerClientImpl.h, but it is probably better to just file a bug.

Might as well just do it in this patch.

> > Source/WebCore/platform/graphics/qt/MediaPlayerPrivateQt.h:134
> > +    virtual String engineDescription() const { return ""; }
> 
> Did you mean to leave this blank?

Nope, I'll fill that one in too.

Thanks!
Comment 4 Jer Noble 2012-07-26 07:59:37 PDT
(In reply to comment #3)
> (In reply to comment #2)
> > You could also change Source/WebKit/chromium/src/WebMediaPlayerClientImpl.h, but it is probably better to just file a bug.
> 
> Might as well just do it in this patch.

Nevermind, I'll file a bug.
Comment 5 Jer Noble 2012-07-26 08:10:27 PDT
Committed r123747: <http://trac.webkit.org/changeset/123747>