Commit e0cc162d authored by Andreas Haas's avatar Andreas Haas Committed by Commit Bot

[webcodecs] Replace use of deprecated ArrayBuffer

This CL replaces the use of ArrayBuffer in EncodedVideoChunk by
DOMArrayBuffer. ArrayBuffer is deprecated.

R=yukishiino@chromium.org

Bug: chromium:1008840
Change-Id: I3b1bdefce6a7e60e6b30ddaf6f55dbe1b748da3d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2102584Reviewed-by: default avatarYuki Shiino <yukishiino@chromium.org>
Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Commit-Queue: Andreas Haas <ahaas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#750489}
parent b3ce2d25
...@@ -27,12 +27,12 @@ EncodedVideoChunk* EncodedVideoChunk::Create(String type, ...@@ -27,12 +27,12 @@ EncodedVideoChunk* EncodedVideoChunk::Create(String type,
if (duration) if (duration)
metadata.duration = base::TimeDelta::FromMicroseconds(duration); metadata.duration = base::TimeDelta::FromMicroseconds(duration);
return MakeGarbageCollected<EncodedVideoChunk>( return MakeGarbageCollected<EncodedVideoChunk>(
metadata, ArrayBuffer::Create(data.Bytes(), data.ByteLengthAsSizeT())); metadata, DOMArrayBuffer::Create(data.Bytes(), data.ByteLengthAsSizeT()));
} }
EncodedVideoChunk::EncodedVideoChunk(EncodedVideoMetadata metadata, EncodedVideoChunk::EncodedVideoChunk(EncodedVideoMetadata metadata,
scoped_refptr<ArrayBuffer> buffer) DOMArrayBuffer* buffer)
: metadata_(metadata), buffer_(std::move(buffer)) {} : metadata_(metadata), buffer_(buffer) {}
String EncodedVideoChunk::type() const { String EncodedVideoChunk::type() const {
return metadata_.key_frame ? "key" : "delta"; return metadata_.key_frame ? "key" : "delta";
...@@ -58,7 +58,7 @@ uint64_t EncodedVideoChunk::duration(bool* is_null) const { ...@@ -58,7 +58,7 @@ uint64_t EncodedVideoChunk::duration(bool* is_null) const {
} }
DOMArrayBuffer* EncodedVideoChunk::data() const { DOMArrayBuffer* EncodedVideoChunk::data() const {
return DOMArrayBuffer::Create(buffer_); return buffer_;
} }
} // namespace blink } // namespace blink
...@@ -12,15 +12,13 @@ ...@@ -12,15 +12,13 @@
namespace blink { namespace blink {
class ArrayBuffer;
class DOMArrayBuffer; class DOMArrayBuffer;
class MODULES_EXPORT EncodedVideoChunk final : public ScriptWrappable { class MODULES_EXPORT EncodedVideoChunk final : public ScriptWrappable {
DEFINE_WRAPPERTYPEINFO(); DEFINE_WRAPPERTYPEINFO();
public: public:
EncodedVideoChunk(EncodedVideoMetadata metadata, EncodedVideoChunk(EncodedVideoMetadata metadata, DOMArrayBuffer* buffer);
scoped_refptr<ArrayBuffer> buffer);
static EncodedVideoChunk* Create(String type, static EncodedVideoChunk* Create(String type,
uint64_t timestamp, uint64_t timestamp,
...@@ -41,9 +39,14 @@ class MODULES_EXPORT EncodedVideoChunk final : public ScriptWrappable { ...@@ -41,9 +39,14 @@ class MODULES_EXPORT EncodedVideoChunk final : public ScriptWrappable {
} }
DOMArrayBuffer* data() const; DOMArrayBuffer* data() const;
void Trace(Visitor* visitor) override {
visitor->Trace(buffer_);
ScriptWrappable::Trace(visitor);
}
private: private:
EncodedVideoMetadata metadata_; EncodedVideoMetadata metadata_;
scoped_refptr<ArrayBuffer> buffer_; Member<DOMArrayBuffer> buffer_;
}; };
} // namespace blink } // namespace blink
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment