Remove MediaLargeByteBuffer

RESOLVED FIXED in Firefox 41

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: poiru, Assigned: poiru)

Tracking

Trunk
mozilla41
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox41 fixed)

Details

Attachments

(3 attachments)

FallibleTArray will be removed in bug 968520 so MediaLargeByteBuffer needs to go. All MediaLargeByteBuffer calls already include the `mozilla::fallible` parameter so we can safely replace MediaLargeByteBuffer with MediaByteBuffer. In the future, one should use the `mozilla::fallible` parameter where fallibility is wanted.
All MediaLargeByteBuffer calls already include the `mozilla::fallible`
parameter so we can safely replace MediaLargeByteBuffer with MediaByteBuffer.
In the future, one should use the `mozilla::fallible` parameter where
fallibility is wanted.
Attachment #8621680 - Flags: review?(jyavenard)
Attachment #8621681 - Flags: review?(jyavenard)
Attachment #8621677 - Flags: review?(jyavenard) → review+
Comment on attachment 8621680 [details] [diff] [review]
Part 2: Use MediaByteBuffer instead of MediaLargeByteBuffer

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

::: dom/media/MediaData.h
@@ +21,5 @@
>  class Image;
>  class ImageContainer;
>  }
>  
> +class MediaByteBuffer;

duplicate
Attachment #8621680 - Flags: review?(jyavenard) → review+
Comment on attachment 8621681 [details] [diff] [review]
Part 3: Remove MediaLargeByteBuffer

wish all patches were like that: one scope, one patch.. perfect
Attachment #8621681 - Flags: review?(jyavenard) → review+
You need to log in before you can comment on or make changes to this bug.