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

Issue 2387583003: Reland of Add PlzNavigate run of browser_tests to main linux config (Closed)

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

Description

Reland of Add PlzNavigate run of browser_tests to main linux config (patchset #1 id:1 of https://codereview.chromium.org/2378973004/ ) Reason for revert: Relanding now that tests are filtered out. BUG=504347 Original issue's description: > Revert of Add PlzNavigate run of browser_tests to main linux config (patchset #3 id:40001 of https://codereview.chromium.org/2380893002/ ) > > Reason for revert: > browser_side_navigation_browser_test is consistently failing on Linux Tests (dbg)(1) bot > > https://uberchromegw.corp.google.com/i/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/57645 > https://uberchromegw.corp.google.com/i/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/57646 > https://uberchromegw.corp.google.com/i/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/57647 > https://uberchromegw.corp.google.com/i/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/57648 > https://uberchromegw.corp.google.com/i/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/57649 > > Original issue's description: > > Add PlzNavigate run of browser_tests to main linux config > > > > R=dpranke@chromium.org > > BUG=504347 > > > > Committed: https://crrev.com/0c46e36075fcaa458c81fcae19bd321a2fb38bbd > > Cr-Commit-Position: refs/heads/master@{#421998} > > TBR=dpranke@chromium.org,jam@chromium.org,scottmg@chromium.org > # Skipping CQ checks because original CL landed less than 1 days ago. > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=504347 > > Committed: https://crrev.com/038a12f50d1c93adafc3acd12b7df70d3cd61047 > Cr-Commit-Position: refs/heads/master@{#422045} TBR=dpranke@chromium.org,jam@chromium.org,ksakamoto@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=504347 Committed: https://crrev.com/11a0c6e557aeb55a6308252f974d91cc8d6050a2 Cr-Commit-Position: refs/heads/master@{#422150}

Patch Set 1 #

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

Messages

Total messages: 10 (4 generated)
scottmg
Created Reland of Add PlzNavigate run of browser_tests to main linux config
4 years, 2 months ago (2016-09-30 16:12:43 UTC) #1
jam
lgtm
4 years, 2 months ago (2016-09-30 16:17:46 UTC) #3
Dirk Pranke
lgtm.
4 years, 2 months ago (2016-09-30 16:23:19 UTC) #4
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/2387583003/1
4 years, 2 months ago (2016-09-30 17:49:01 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 2 months ago (2016-09-30 17:51:18 UTC) #8
commit-bot: I haz the power
4 years, 2 months ago (2016-09-30 17:52:50 UTC) #10
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/11a0c6e557aeb55a6308252f974d91cc8d6050a2
Cr-Commit-Position: refs/heads/master@{#422150}

Powered by Google App Engine
This is Rietveld 408576698