binary_stream: Simplify away a no-op TypedArray

[kainino0x]

I just noticed this while looking around for TypedArray allocations that could be cleaned up. The code here didn't match the intent. This change is a no-op, but maybe there is a bug to be fixed instead.

I only verified the unittests pass, so far, since I figured I should send this to you first.

Issue: None

---

Requirements for PR author:

• All missing test coverage is tracked with "TODO" or .unimplemented().
• New helpers are /** documented */ and new helper files are found in helper_index.txt.
• Test behaves as expected in a WebGPU implementation. (If not passing, explain above.)

Requirements for reviewer sign-off:

• Tests are properly located in the test tree.
• Test descriptions allow a reader to "read only the test plans and evaluate coverage completeness", and accurately reflect the test code.
• Tests provide complete coverage (including validation control cases). Missing coverage MUST be covered by TODOs.
• Helpers and types promote readability and maintainability.

When landing this PR, be sure to make any necessary issue status updates.

[kainino0x]

new Uint8Array wraps a view around this.view.buffer. Uint8Array.buffer returns the underlying ArrayBuffer of the view. Therefore this was the same as simply this.view.buffer.

[ben-clayton]

This was supposed to be new Uint8Array(this.view.buffer, 0, align(this.offset, 8)).buffer as in: truncate the array to the written size. I actually noticed this bug yesterday, as all the cached files are 33.6MB (!). Fortunately tar.gz does a good job removing all those 0's.

[github-actions]

Previews, as seen when this build job started (fd9212a):
Run tests | View tsdoc