Require someone working on PlzNavigate to approve changes to the test filters.

Require someone working on PlzNavigate to approve changes to the test filters.

Since PlzNavigate is close to launching, we want to ensure that someone from the team approves any new additions to the test filters for browser_tests & content_browsertests, which are almost empty now.

BUG=368813
Committed: https://crrev.com/d8a6bf34335f3bebf7beac12493d10ee5e4fa7d3
Cr-Commit-Position: refs/heads/master@{#436398}

[jam, 2016-12-05 18:13:04]

jam@chromium.org changed reviewers:
+ nasko@chromium.org

[jam, 2016-12-05 18:13:12]

The CQ bit was checked by jam@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-05 18:13:32]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[scottmg, 2016-12-05 18:17:49]

scottmg@chromium.org changed reviewers:
+ scottmg@chromium.org

[scottmg, 2016-12-05 18:17:49]

lgtm

[nasko, 2016-12-05 18:18:28]

LGTM with a suggestion.

https://codereview.chromium.org/2548363003/diff/1/testing/buildbot/filters/OW...
File testing/buildbot/filters/OWNERS (right):

https://codereview.chromium.org/2548363003/diff/1/testing/buildbot/filters/OW...
testing/buildbot/filters/OWNERS:10: per-file
browser-side-navigation*=yzshen@chromium.org
I'd suggest adding creis@ too.

[commit-bot: I haz the power, 2016-12-05 19:12:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-05 19:12:14]

Dry run: This issue passed the CQ dry run.

[jam, 2016-12-05 19:27:51]

The CQ bit was checked by jam@chromium.org

[jam, 2016-12-05 19:27:52]

The patchset sent to the CQ was uploaded after l-g-t-m from nasko@chromium.org,
scottmg@chromium.org
Link to the patchset: https://codereview.chromium.org/2548363003/#ps20001
(title: "review comment")

[jam, 2016-12-05 19:28:00]

https://codereview.chromium.org/2548363003/diff/1/testing/buildbot/filters/OW...
File testing/buildbot/filters/OWNERS (right):

https://codereview.chromium.org/2548363003/diff/1/testing/buildbot/filters/OW...
testing/buildbot/filters/OWNERS:10: per-file
browser-side-navigation*=yzshen@chromium.org
On 2016/12/05 18:18:28, nasko wrote:
> I'd suggest adding creis@ too.

Done.

[commit-bot: I haz the power, 2016-12-05 19:28:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-05 20:52:37]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480966071593640,
"parent_rev": "d7b726f7f177ba80cf31408a9607912fa1428ba2", "commit_rev":
"22798c65004ee78d922a82f965591d0c2d2770ff"}

[commit-bot: I haz the power, 2016-12-05 20:53:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-05 20:55:12]

Description was changed from

==========
Require someone working on PlzNavigate to approve changes to the test filters.

Since PlzNavigate is close to launching, we want to ensure that someone from the
team approves any new additions to the test filters for browser_tests &
content_browsertests, which are almost empty now.

BUG=368813
==========

to

==========
Require someone working on PlzNavigate to approve changes to the test filters.

Since PlzNavigate is close to launching, we want to ensure that someone from the
team approves any new additions to the test filters for browser_tests &
content_browsertests, which are almost empty now.

BUG=368813

Committed: https://crrev.com/d8a6bf34335f3bebf7beac12493d10ee5e4fa7d3
Cr-Commit-Position: refs/heads/master@{#436398}
==========

[commit-bot: I haz the power, 2016-12-05 20:55:13]

Patchset 2 (id:??) landed as
https://crrev.com/d8a6bf34335f3bebf7beac12493d10ee5e4fa7d3
Cr-Commit-Position: refs/heads/master@{#436398}