1691886 - Consider removing BufferSize

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P3)

Description

[Jan de Mooij [:jandem]]

This wrapper around size_t was helpful to mark the places that still needed changes, but deprecatedGetUint32 will be removed soon and then it probably makes sense to replace BufferSize with size_t.

Note that BufferSize does not guarantee the wrapped size is <= the max ArrayBuffer byte length, so I don't think it's that useful to keep around going forward.

This can wait a few weeks until we have had more fuzzing.

Comment 1

[Lars T Hansen [:lth]]

Poaching.

Comment 2

[Lars T Hansen [:lth]]

Attachment: Bug 1691886 - Remove BufferSize. r?jandem

Remove the BufferSize abstraction, replace it with size_t everywhere.

Depends on D111100

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/76f9bff99e53
Remove BufferSize. r=jandem

Comment 4

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/76f9bff99e53