Closed Bug 1287021 Opened 4 years ago Closed 4 years ago

[MSE] delaying-the-load-event-flag should be set to false when mediasource is attached to element

Categories

(Core :: Audio/Video: Playback, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: jya, Assigned: jya)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

As per new MSE spec made on June 6th 2016 and discussed in https://github.com/w3c/media-source/issues/24
https://reviewboard.mozilla.org/r/64498/#review61488

::: dom/media/mediasource/MediaSource.cpp:435
(Diff revision 1)
>    }
>    MOZ_ASSERT(!mMediaElement);
>    mMediaElement = aDecoder->GetOwner()->GetMediaElement();
>    MOZ_ASSERT(!mDecoder);
>    mDecoder = aDecoder;
> +  mMediaElement->ChangeDelayLoadStatus(false);

Can you call ChangeDelayLoadStatus() here https://hg.mozilla.org/mozilla-central/file/88bebcaca249aeaca9197382e89d35b02be8292e/dom/html/HTMLMediaElement.cpp#l1516 without making it public?
https://reviewboard.mozilla.org/r/64498/#review61488

> Can you call ChangeDelayLoadStatus() here https://hg.mozilla.org/mozilla-central/file/88bebcaca249aeaca9197382e89d35b02be8292e/dom/html/HTMLMediaElement.cpp#l1516 without making it public?

but then the order of operations isn't as per spec.
That is a long text to read. Can you give a summary about what should happen in order in the spec?
Do you mean https://github.com/w3c/media-source/issues/24#issuecomment-221993352
(3) just set the delaying-the-load-event-flag to false immediately upon MediaSource attachment to the HTMLMediaElement?
yes
I can't see how it is not per spec if we say:

if (mMediaSource) {
  RefPtr<MediaSourceDecoder> decoder = new MediaSourceDecoder(this);
  if (!mMediaSource->Attach(decoder)) {
    decoder->Shutdown();
    return NS_ERROR_FAILURE;
  }
  ChangeDelayLoadStatus(false); // <--
  // ...
}
true... I had assumed that the event could only be fired after the loadstatus had changed, but it doesn't matter
Comment on attachment 8771243 [details]
Bug 1287021: [MSE] Set delaying-the-load-event-flag to false when attaching media source.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/64498/diff/1-2/
Attachment #8771243 - Flags: review?(jwwang) → review+
Comment on attachment 8771243 [details]
Bug 1287021: [MSE] Set delaying-the-load-event-flag to false when attaching media source.

https://reviewboard.mozilla.org/r/64498/#review61728
Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1b3826196e7e
[MSE] Set delaying-the-load-event-flag to false when attaching media source. r=jwwang
https://hg.mozilla.org/mozilla-central/rev/1b3826196e7e
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.