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

Issue 1153263004: Unship harmony tostring (Closed)

Created:
5 years, 6 months ago by arv (Not doing code reviews)
Modified:
5 years, 6 months ago
Reviewers:
adamk, rossberg
CC:
v8-dev, jochen (gone - plz use gerrit), caitp (gmail)
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

Unship harmony tostring Blink's DOM bindings are not yet ready. BUG=493137, 239915 LOG=N R=rossberg, adamk CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel Committed: https://crrev.com/07c1f277bd93dcbfef3c2dd2b256cdde5665289c Cr-Commit-Position: refs/heads/master@{#28804}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -7 lines) Patch
M src/flag-definitions.h View 1 chunk +7 lines, -6 lines 0 comments Download
M test/mjsunit/es6/built-in-accessor-names.js View 1 chunk +2 lines, -0 lines 0 comments Download
M test/mjsunit/harmony/sharedarraybuffer.js View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 27 (9 generated)
arv (Not doing code reviews)
PTAL
5 years, 6 months ago (2015-06-03 13:55:12 UTC) #1
rossberg
LGTM, but you may want to change the subject to "Unship". ;)
5 years, 6 months ago (2015-06-03 13:59:54 UTC) #2
arv (Not doing code reviews)
On 2015/06/03 13:59:54, rossberg wrote: > LGTM, but you may want to change the subject ...
5 years, 6 months ago (2015-06-03 14:05:08 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1153263004/1
5 years, 6 months ago (2015-06-03 17:06:03 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: v8_presubmit on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/3159)
5 years, 6 months ago (2015-06-03 17:09:18 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1153263004/1
5 years, 6 months ago (2015-06-03 17:55:23 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: linux_blink_rel on tryserver.blink (JOB_FAILED, http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/64792)
5 years, 6 months ago (2015-06-03 20:27:52 UTC) #11
adamk
lgtm I've uploaded a fix for the blink tests in https://codereview.chromium.org/1154523007/
5 years, 6 months ago (2015-06-03 20:44:12 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1153263004/1
5 years, 6 months ago (2015-06-03 23:48:31 UTC) #14
commit-bot: I haz the power
Try jobs failed on following builders: linux_blink_rel on tryserver.blink (JOB_FAILED, http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/64894)
5 years, 6 months ago (2015-06-04 01:03:30 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1153263004/1
5 years, 6 months ago (2015-06-04 01:06:32 UTC) #18
commit-bot: I haz the power
Try jobs failed on following builders: linux_blink_rel on tryserver.blink (JOB_FAILED, http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/64911)
5 years, 6 months ago (2015-06-04 01:45:43 UTC) #20
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1153263004/1
5 years, 6 months ago (2015-06-04 15:53:35 UTC) #22
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 6 months ago (2015-06-04 17:33:31 UTC) #23
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/07c1f277bd93dcbfef3c2dd2b256cdde5665289c Cr-Commit-Position: refs/heads/master@{#28804}
5 years, 6 months ago (2015-06-04 17:33:44 UTC) #24
Michael Achenbach
On 2015/06/04 17:33:44, commit-bot: I haz the power wrote: > Patchset 1 (id:??) landed as ...
5 years, 6 months ago (2015-06-04 19:26:49 UTC) #25
Michael Achenbach
http://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/303 inspector/console/console-format-es6.html
5 years, 6 months ago (2015-06-04 19:27:42 UTC) #26
arv (Not doing code reviews)
5 years, 6 months ago (2015-06-04 19:57:08 UTC) #27
Message was sent while issue was closed.
On 2015/06/04 19:27:42, Michael Achenbach (OOO) wrote:
>
http://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/buil...
> inspector/console/console-format-es6.html

I'm on it.

Powered by Google App Engine
This is Rietveld 408576698