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

Issue 2380183003: Reland of PlzNav: make release and debug linux match for content_browsertests, content_unittests, .. (Closed)

Created:
4 years, 2 months ago by scottmg
Modified:
4 years, 2 months ago
Reviewers:
Kunihiko Sakamoto, jam
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reland of PlzNav: make release and debug linux match for content_browsertests, content_unittests, and unit_te… (patchset #1 id:1 of https://codereview.chromium.org/2380243002/ ) Reason for revert: Relanding now that failing tests are filtered out. BUG=475027 Original issue's description: > Revert of PlzNav: make release and debug linux match for content_browsertests, content_unittests, and unit_te… (patchset #1 id:1 of https://codereview.chromium.org/2381203002/ ) > > Reason for revert: > browser_side_navigation_content_browsertests is consistently failing on Linux Tests (dbg)(1) bot > > https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/57646 > through > https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/57651 > > Original issue's description: > > PlzNav: make release and debug linux match for content_browsertests, content_unittests, and unit_tests > > > > R=jam@chromium.org > > > > Committed: https://crrev.com/7093f1ac41f71b3bc2be06b2b9abfd566c0791d7 > > Cr-Commit-Position: refs/heads/master@{#422005} > > TBR=jam@chromium.org,scottmg@chromium.org > # Skipping CQ checks because original CL landed less than 1 days ago. > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > > Committed: https://crrev.com/26fdbb01367adbe24cbe806e29784f894b494942 > Cr-Commit-Position: refs/heads/master@{#422064} TBR=jam@chromium.org,ksakamoto@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. Committed: https://crrev.com/1e02a3f9ba0871a11d0b63622307c66d1300df50 Cr-Commit-Position: refs/heads/master@{#422155}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -0 lines) Patch
M testing/buildbot/chromium.linux.json View 2 chunks +31 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (4 generated)
scottmg
Created Reland of PlzNav: make release and debug linux match for content_browsertests, content_unittests, and unit_te…
4 years, 2 months ago (2016-09-30 16:10:59 UTC) #1
jam
lgtm
4 years, 2 months ago (2016-09-30 16:17:37 UTC) #3
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/2380183003/1
4 years, 2 months ago (2016-09-30 17:19:06 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 2 months ago (2016-09-30 18:15:06 UTC) #7
commit-bot: I haz the power
4 years, 2 months ago (2016-09-30 18:16:36 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/1e02a3f9ba0871a11d0b63622307c66d1300df50
Cr-Commit-Position: refs/heads/master@{#422155}

Powered by Google App Engine
This is Rietveld 408576698