Phabricator will be unavailable due to database maintenance from 14:00 UTC until 18:00 UTC on Saturday, October 13, 2018.
Bugzilla will remain up during this time. All users have been logged out of Bugzilla

Should DataView.length be 1 ?

ASSIGNED
Assigned to
(NeedInfo from)

Status

()

P3
normal
ASSIGNED
2 years ago
17 days ago

People

(Reporter: arai, Assigned: till, NeedInfo)

Tracking

({triage-deferred})

Trunk
triage-deferred
Points:
---

Firefox Tracking Flags

(firefox54 affected)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
from https://github.com/tc39/ecma262/issues/787

According to the spec, the length of DataView constructor should be 1,
however Firefox, Chrome, WebKit, and es6draft return 3.
only Edge returns 1.
Keywords: triage-deferred
Priority: -- → P3
(Assignee)

Comment 1

7 months ago
Created attachment 8960904 [details] [diff] [review]
Change DataView.length from 3 to 1

We decided on this at the current TC39 meeting.

The patch is of course very straight-forward, except it changes a test262 test. When we do the next update from upstream, this change will be overwritten. The test might've been fixed by then, but if not, we'll get an additional failure. I think that's ok, because this test isn't important at all, so we shouldn't worry about it.
Assignee: nobody → till
Status: NEW → ASSIGNED
Attachment #8960904 - Flags: review?(arai.unmht)
(Reporter)

Comment 2

7 months ago
Comment on attachment 8960904 [details] [diff] [review]
Change DataView.length from 3 to 1

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

Thanks!
Attachment #8960904 - Flags: review?(arai.unmht) → review+
(Reporter)

Comment 3

17 days ago
can you land the patch?
Flags: needinfo?(till)
You need to log in before you can comment on or make changes to this bug.