[Webkit-unassigned] [Bug 64731] Add MediaSource API to HTMLMediaElement

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 20 11:48:09 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=64731


Eric Carlson <eric.carlson at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #101481|review?                     |review-
               Flag|                            |




--- Comment #16 from Eric Carlson <eric.carlson at apple.com>  2011-07-20 11:48:08 PST ---
(From update of attachment 101481)
View in context: https://bugs.webkit.org/attachment.cgi?id=101481&action=review

Two meta-issues:

1) I think this is complex enough that is needs to be completely described in a spec before any changes are checked in.

2) Even if this feature will only be enabled in Chromium, the code is in the main WebKit repository so we need tests in the same repository.


> Source/WebCore/html/HTMLMediaElement.cpp:1682
> +    if (m_sourceState == SOURCE_CLOSED) {
> +        scheduleEvent(eventNames().closeEvent);
> +        return;
> +    }

Shouldn't this be "webkitsourcecloseEvent" for symmetry?

> Source/WebCore/platform/graphics/MediaPlayer.cpp:144
> +    virtual bool sourceAppend(const unsigned char* data, unsigned length) { return false; }

These parameter names aren't needed.

> Source/WebCore/platform/graphics/MediaPlayerPrivate.h:151
> +    virtual bool sourceAppend(const unsigned char* data, unsigned length) { return false; }

These parameter names aren't needed.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list