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

Issue 2902943004: Update the V8 version for Chromium branch 60. (Closed)

Created:
3 years, 7 months ago by vogelheim
Modified:
3 years, 7 months ago
Reviewers:
Michael Achenbach
CC:
chromium-reviews
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Update the V8 version for Chromium branch 60. This is a roll-back to 6.0.286. This is the most stable candidate out of the latest Canaries. If this commit results in failures in Blink please contact the Blink sheriff. They may need to make changes to Blnik because of the roll-back. When in doubt please try to contact the committer and reviewers of this CL before reverting it. TBR=machenbach@chromium.org Review-Url: https://codereview.chromium.org/2902943004 Cr-Commit-Position: refs/heads/master@{#474293} Committed: https://chromium.googlesource.com/chromium/src/+/44d0d64eb010c6d52b53359aa37d338ea8a9d3b8

Patch Set 1 #

Patch Set 2 : Rebase. #

Patch Set 3 : Use cherry-picked version. #

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

Messages

Total messages: 24 (17 generated)
Michael Achenbach
lgtm
3 years, 7 months ago (2017-05-24 13:14:00 UTC) #9
Michael Achenbach
Please rebase to 789ec38b623c0fa9a57bc598b5447a8e859b4790
3 years, 7 months ago (2017-05-24 13:34:18 UTC) #10
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/2902943004/40001
3 years, 7 months ago (2017-05-24 13:49:31 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/2902943004/40001
3 years, 7 months ago (2017-05-24 14:42:59 UTC) #19
Michael Achenbach
haha, is Blnik part of the template where you copied the cl desc from?
3 years, 7 months ago (2017-05-24 14:43:52 UTC) #20
vogelheim
On 2017/05/24 14:43:52, machenbach (OOO) wrote: > haha, is Blnik part of the template where ...
3 years, 7 months ago (2017-05-24 14:46:48 UTC) #21
commit-bot: I haz the power
3 years, 7 months ago (2017-05-24 15:28:35 UTC) #24
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/44d0d64eb010c6d52b53359aa37d...

Powered by Google App Engine
This is Rietveld 408576698