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

Issue 2583753002: [promises] Use TF_BUILTIN (Closed)

Created:
4 years ago by gsathya
Modified:
4 years ago
Reviewers:
Igor Sheludko, jgruber
CC:
v8-reviews_googlegroups.com
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[promises] Use TF_BUILTIN BUG=v8:5343 Review-Url: https://codereview.chromium.org/2583753002 Cr-Commit-Position: refs/heads/master@{#41770} Committed: https://chromium.googlesource.com/v8/v8/+/3f5a60ac17dc1e84bfda6429cc69f7497ed3ea24

Patch Set 1 #

Patch Set 2 : Remove var #

Patch Set 3 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+618 lines, -678 lines) Patch
M src/builtins/builtins-promise.cc View 1 2 2 chunks +618 lines, -678 lines 0 comments Download

Messages

Total messages: 22 (15 generated)
gsathya
4 years ago (2016-12-16 03:32:26 UTC) #10
Igor Sheludko
lgtm
4 years ago (2016-12-16 07:44:44 UTC) #11
jgruber
Nice, lgtm
4 years ago (2016-12-16 07:56:43 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2583753002/20001
4 years ago (2016-12-16 15:06:02 UTC) #14
commit-bot: I haz the power
Try jobs failed on following builders: v8_presubmit on master.tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/30830)
4 years ago (2016-12-16 15:08:54 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2583753002/40001
4 years ago (2016-12-16 15:29:23 UTC) #19
commit-bot: I haz the power
4 years ago (2016-12-16 15:53:47 UTC) #22
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/v8/v8/+/3f5a60ac17dc1e84bfda6429cc69f7497ed...

Powered by Google App Engine
This is Rietveld 408576698