Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(426)

Issue 13465008: Fix Array.prototype.concat when exceeding array size limit. (Closed)

Created:
7 years, 8 months ago by Yang
Modified:
7 years, 8 months ago
Reviewers:
Toon Verwaest
CC:
v8-dev
Visibility:
Public.

Description

Fix Array.prototype.concat when exceeding array size limit. R=verwaest@chromium.org BUG=v8:581 Committed: https://code.google.com/p/v8/source/detail?r=14154

Patch Set 1 #

Total comments: 1

Patch Set 2 : fixed according to comment #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -7 lines) Patch
M src/runtime.cc View 1 2 4 chunks +16 lines, -2 lines 0 comments Download
A + test/mjsunit/regress/regress-581.js View 1 2 1 chunk +17 lines, -5 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Yang
7 years, 8 months ago (2013-04-02 13:08:10 UTC) #1
Toon Verwaest
lgtm with comment https://chromiumcodereview.appspot.com/13465008/diff/1/src/runtime.cc File src/runtime.cc (right): https://chromiumcodereview.appspot.com/13465008/diff/1/src/runtime.cc#newcode8973 src/runtime.cc:8973: if (i >= JSObject::kMaxElementCount - index_offset_) ...
7 years, 8 months ago (2013-04-04 12:52:24 UTC) #2
Toon Verwaest
As discussed offline, but now for the record: ECMAScript 6 specifies that the length property ...
7 years, 8 months ago (2013-04-04 17:30:05 UTC) #3
Yang
7 years, 8 months ago (2013-04-05 15:13:08 UTC) #4
Message was sent while issue was closed.
Committed patchset #3 manually as r14154 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698