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

Issue 459373002: Update V8 to version 3.28.71 (based on bleeding_edge revision r23062). (Closed)

Created:
6 years, 4 months ago by Michael Achenbach
Modified:
6 years, 4 months ago
CC:
chromium-reviews
Project:
chromium
Visibility:
Public.

Description

Update V8 to version 3.28.71 (based on bleeding_edge revision r23062). TBR=jochen@chromium.org NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288989

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 12 (0 generated)
Michael Achenbach
6 years, 4 months ago (2014-08-12 07:12:50 UTC) #1
Michael Achenbach
The CQ bit was checked by machenbach@chromium.org
6 years, 4 months ago (2014-08-12 07:12:51 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/machenbach@chromium.org/459373002/1
6 years, 4 months ago (2014-08-12 07:14:16 UTC) #3
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). The failing builders are: win_chromium_rel on tryserver.chromium.win ...
6 years, 4 months ago (2014-08-12 11:06:39 UTC) #4
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 4 months ago (2014-08-12 13:12:00 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: win_chromium_x64_rel on tryserver.chromium.win (http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_rel/builds/5840)
6 years, 4 months ago (2014-08-12 13:12:01 UTC) #6
Michael Achenbach
The win failure shows up on the main waterfall as well. Will set notry.
6 years, 4 months ago (2014-08-12 13:19:56 UTC) #7
Michael Achenbach
The CQ bit was checked by machenbach@chromium.org
6 years, 4 months ago (2014-08-12 13:20:11 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/machenbach@chromium.org/459373002/1
6 years, 4 months ago (2014-08-12 13:21:48 UTC) #9
jochen (gone - plz use gerrit)
lgtm
6 years, 4 months ago (2014-08-12 14:49:11 UTC) #10
commit-bot: I haz the power
Change committed as 288989
6 years, 4 months ago (2014-08-12 15:46:51 UTC) #11
lushnikov
6 years, 4 months ago (2014-08-13 21:06:29 UTC) #12
Message was sent while issue was closed.
A revert of this CL (patchset #1) has been created in
https://codereview.chromium.org/471493003/ by lushnikov@chromium.org.

The reason for reverting is: This broke developer tools editor. Please see
https://code.google.com/p/chromium/issues/detail?id=403521.

Powered by Google App Engine
This is Rietveld 408576698