Closed Bug 1082239 Opened 5 years ago Closed 5 years ago

test_encryptedMediaExtensions.html | gizmo-frag-cencinit.mp4-1 Failed to set MediaKeys on <video> element

Categories

(Core :: Audio/Video, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla36
Tracking Status
firefox37 --- fixed
firefox38 --- fixed
firefox39 --- fixed

People

(Reporter: eflores, Assigned: eflores)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 2 obsolete files)

No description provided.
https://hg.mozilla.org/mozilla-central/rev/250146966b12
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Oops, forgot to [leave open]. Still need to fix it properly.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment on attachment 8505161 [details] [diff] [review]
Make MediaSourceResource::GetPrincipal return MediaSource script principal

Review of attachment 8505161 [details] [diff] [review]:
-----------------------------------------------------------------

::: content/media/mediasource/MediaSourceResource.h
@@ +53,5 @@
>    virtual nsresult Open(nsIStreamListener** aStreamListener) MOZ_OVERRIDE { UNIMPLEMENTED(); return NS_ERROR_FAILURE; }
>  
> +  virtual already_AddRefed<nsIPrincipal> GetCurrentPrincipal() MOZ_OVERRIDE
> +  {
> +    nsRefPtr<nsIPrincipal> ret = mPrincipal;

s/ret/principal/

or

return nsRefPtr<nsIPrincipal>(mPrincipal).forget();
Attachment #8505161 - Flags: review?(kinetik) → review+
https://hg.mozilla.org/mozilla-central/rev/4f520729fb38
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Mass update firefox-status to track EME uplift.
You need to log in before you can comment on or make changes to this bug.