New issue
Advanced search Search tips

Issue 753927 link

Starred by 1 user
Status: Duplicate
Merged: issue 752722
Owner:
tebbi@chromium.org
Closed: Aug 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Function Array.prototype.shift do not remove the first element

Reported by dantonep...@gmail.com, Aug 9 2017 
UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.90 Safari/537.36

Steps to reproduce the problem:
1. Execute the following javascript:
//When the first element is undefined and the other elements are real numbers.
var array = [, 1.1, 2];
array.shift();
console.log(array);

What is the expected behavior?
(2) [1.1, 2]

What went wrong?
(2) [undefined × 1, 1.1]

Did this work before? Yes 

Chrome version: 60.0.3112.90  Channel: stable
OS Version: 10.0
Flash Version:

Comment 1 by krajshree@chromium.org, Aug 11 2017

Components: Blink>JavaScript
Labels: -Pri-2 hasbisect-per-revision ReleaseBlock-Stable M-62 Needs-Triage-M60 OS-Linux OS-Mac Pri-1
Owner: tebbi@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on Windows 10, Mac 10.12.6 and Ubuntu 14.04 using reported version #60.0.3112.90 but the same is not reproducible in the latest canary #62.0.3181.0.

Reverse Bisect Information:
=====================
Good build: 62.0.3179.0	 Revision(492477)
Bad Build : 62.0.3178.0	 Revision(492239)

Change Log URL: 
https://chromium.googlesource.com/chromium/src/+log/676befd9322eaebac271fe94d79670e8a50a0bff..a07a574eeba872a56611fcf122f1b02fc33b9761

Blink Change Log:
https://chromium.googlesource.com/v8/v8/+log/3f66a940..c9164210

From the above blink change log possible CL that fixed this issue:
Change-Id: I300a3ebcfafb07d6ecebc01fa57c66eb26f349ac
Reviewed-on: https://chromium-review.googlesource.com/603717

tebbi@ - Could you please check and merge the fix to M52 if it is a valid canadidate.

Note: Adding label ReleaseBlock-Stable as it seems to be a recent regression.

Thanks...!!

Comment 2 by e...@chromium.org, Aug 11 2017

Components: -Blink

Comment 3 by tebbi@chromium.org, Aug 16 2017

Mergedinto: 752722
Status: Duplicate (was: Assigned)
@krajshree: This already has merge-approval for M60, will merge it right now. Did you mean M60 when you wrote M52?
►

Sign in to add a comment