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

Issue 887123004: Revert of Roll src/third_party/WebKit 465217f:d693046 (svn 189778:189793) (Closed)

Created:
5 years, 10 months ago by jiayl
Modified:
5 years, 10 months ago
CC:
chromium-reviews, caseq
Base URL:
https://chromium.googlesource.com/chromium/src@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Roll src/third_party/WebKit 465217f:d693046 (svn 189778:189793) (patchset #1 id:1 of https://codereview.chromium.org/912483003/) Reason for revert: Suspected to cause a huge number of tests failing: http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29%2832%29/builds/9538 Original issue's description: > Roll src/third_party/WebKit 465217f:d693046 (svn 189778:189793) > > Summary of changes available at: > https://chromium.googlesource.com/chromium/blink/+log/465217f..d693046 > > TBR=dpranke@chromium.org,caseq@chromium.org > > Committed: https://crrev.com/3a40a6413424402ac41db8184a398ec46934c206 > Cr-Commit-Position: refs/heads/master@{#315299} TBR=caseq@chromium.org,dpranke@chromium.org,blink-deps-roller@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true

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: 25 (6 generated)
jiayl
Created Revert of Roll src/third_party/WebKit 465217f:d693046 (svn 189778:189793)
5 years, 10 months ago (2015-02-09 19:46:34 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/887123004/1
5 years, 10 months ago (2015-02-09 19:47:46 UTC) #2
Dirk Pranke
On 2015/02/09 19:47:46, I haz the power (commit-bot) wrote: > CQ is trying da patch. ...
5 years, 10 months ago (2015-02-09 19:48:05 UTC) #4
Dirk Pranke
On 2015/02/09 19:48:05, Dirk Pranke wrote: > On 2015/02/09 19:47:46, I haz the power (commit-bot) ...
5 years, 10 months ago (2015-02-09 19:49:06 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/887123004/1
5 years, 10 months ago (2015-02-09 19:49:24 UTC) #9
commit-bot: I haz the power
Failed to apply the patch.
5 years, 10 months ago (2015-02-09 19:50:17 UTC) #11
jiayl
On 2015/02/09 19:49:06, Dirk Pranke wrote: > On 2015/02/09 19:48:05, Dirk Pranke wrote: > > ...
5 years, 10 months ago (2015-02-09 19:51:07 UTC) #13
Dirk Pranke
On Mon, Feb 9, 2015 at 11:51 AM, <jiayl@chromium.org> wrote: > On 2015/02/09 19:49:06, Dirk ...
5 years, 10 months ago (2015-02-09 19:55:09 UTC) #14
Dirk Pranke
Looks like this was reverted by caseq@ 100 minutes ago: https://codereview.chromium.org/907953002 -- Dirk On Mon, ...
5 years, 10 months ago (2015-02-09 19:56:25 UTC) #15
Dirk Pranke
Ah, I see, the failures are only on the 32-bit bot. On Mon, Feb 9, ...
5 years, 10 months ago (2015-02-09 19:58:26 UTC) #16
Ryan Hamilton
Yes, I pingd caseq to revert manually after the CQ revert failed. On Mon, Feb ...
5 years, 10 months ago (2015-02-09 19:59:03 UTC) #17
jiayl
OK, it should be picked up in the next run. On Mon, Feb 9, 2015 ...
5 years, 10 months ago (2015-02-09 19:59:46 UTC) #18
Dirk Pranke
I suspect that https://codereview.chromium.org/840223002 is the cause. I'm going to revert that change and try ...
5 years, 10 months ago (2015-02-09 20:03:50 UTC) #19
Ryan Hamilton
Agreed. That was the plan caseq was going to implement. Can we wait to re-roll ...
5 years, 10 months ago (2015-02-09 20:10:45 UTC) #20
Dirk Pranke
So, the new Linux (dbg)(32) build is running now, so I expect it will complete ...
5 years, 10 months ago (2015-02-09 20:19:44 UTC) #21
Dirk Pranke
argh, hit return too soon (juggling too many message threads). The new roll -- https://codereview.chromium.org/909983002/ ...
5 years, 10 months ago (2015-02-09 20:21:15 UTC) #22
Ryan Hamilton
Great! On Mon, Feb 9, 2015 at 12:20 PM, Dirk Pranke <dpranke@chromium.org> wrote: > argh, ...
5 years, 10 months ago (2015-02-09 20:21:48 UTC) #23
Dirk Pranke
... and the Linux 32 (dbg) bot cycled green: http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29%2832%29/builds/9543 I'll restart the roll in ...
5 years, 10 months ago (2015-02-09 20:23:51 UTC) #24
Ryan Hamilton
5 years, 10 months ago (2015-02-09 20:25:33 UTC) #25
\o/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

Powered by Google App Engine
This is Rietveld 408576698