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

Issue 281753004: Drop thenable coercion cache (Closed)

Created:
6 years, 7 months ago by rossberg
Modified:
6 years, 7 months ago
CC:
v8-dev, yhirano
Visibility:
Public.

Description

Drop thenable coercion cache R=dslomov@chromium.org BUG=372788 LOG=Y Committed: https://code.google.com/p/v8/source/detail?r=21301

Patch Set 1 #

Patch Set 2 : Test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -37 lines) Patch
M include/v8.h View 1 2 chunks +4 lines, -4 lines 0 comments Download
M src/promise.js View 1 chunk +8 lines, -17 lines 0 comments Download
A + test/mjsunit/es6/regress/regress-cr372788.js View 1 1 chunk +12 lines, -16 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
rossberg
6 years, 7 months ago (2014-05-13 14:56:30 UTC) #1
Dmitry Lomov (no reviews)
Looks good but needs a test
6 years, 7 months ago (2014-05-13 15:36:37 UTC) #2
rossberg
On 2014/05/13 15:36:37, Dmitry Lomov (chromium) wrote: > Looks good but needs a test Done. ...
6 years, 7 months ago (2014-05-13 16:42:18 UTC) #3
Dmitry Lomov (no reviews)
lgtm
6 years, 7 months ago (2014-05-14 08:28:57 UTC) #4
rossberg
6 years, 7 months ago (2014-05-14 10:44:41 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r21301 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698