Closed Bug 1426457 Opened 6 years ago Closed 6 years ago

JS shell builtins: Support setting the serialize(...).arraybuffer property

Categories

(Core :: JavaScript Engine, enhancement, P3)

Product:
Core
Component:
JavaScript Engine
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla60
Tracking Flags:
Tracking Status
firefox60 --- fixed

People

(Reporter: jorendorff, Assigned: jorendorff)

Details

Attachments

(1 file)

In the js shell, support setting serialize(_).arraybuffer
1.11 KB, patch
sfink
: review+
Details | Diff | Splinter Review 
It's odd that you can set `.clonebuffer` but not `.arraybuffer`.
Assignee: nobody → jorendorff
Status: NEW → ASSIGNED
Comment on attachment 8938088 [details] [diff] [review]
In the js shell, support setting serialize(_).arraybuffer

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

Hah! You're absolutely right. I think I got distracted by fantasizing about ripping out all of the string-based stuff, so I didn't see the easy small win here.
Attachment #8938088 - Flags: review?(sphink) → review+
Status: ASSIGNED → NEW
Priority: -- → P3
https://hg.mozilla.org/integration/mozilla-inbound/rev/79c46157cc8a82c8f66a18f956b77978df2e529c
Bug 1426457 - In the js shell, support setting serialize(_).arraybuffer. r=sfink.
https://hg.mozilla.org/mozilla-central/rev/79c46157cc8a
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox60: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: