Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to green up headless_…

Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to green up headless_browsertests again.

Also run headless_browsertests with PlzNavigate on the CQ to avoid future regressions.

Lastly, run headless_browsertests on debug Linux bot. This is needed otherwise a DCHECK will break the CQ but won't show up on the main waterfall.

BUG=504347
R=nasko@chromium.org

Review-Url: https://codereview.chromium.org/2727913006
Cr-Original-Commit-Position: refs/heads/master@{#454922}
Committed: https://chromium.googlesource.com/chromium/src/+/ebeb36a0ed4b69c72ce8f2c803bf4233d1cea9f1
Review-Url: https://codereview.chromium.org/2727913006 .
Cr-Commit-Position: refs/heads/master@{#455039}
Committed: https://chromium.googlesource.com/chromium/src/+/04eb2294fb3baa85315da61ef38a3ce225794f0a

[jam, 2017-03-03 19:50:39]

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

[commit-bot: I haz the power, 2017-03-03 19:51:04]

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

[jam, 2017-03-03 19:51:27]

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

[commit-bot: I haz the power, 2017-03-03 20:22:27]

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

[commit-bot: I haz the power, 2017-03-03 20:22:28]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jam, 2017-03-03 20:30:43]

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

[commit-bot: I haz the power, 2017-03-03 20:31:01]

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

[jam, 2017-03-03 20:36:19]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2017-03-03 21:26:06]

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

[commit-bot: I haz the power, 2017-03-03 21:26:07]

Dry run: This issue passed the CQ dry run.

[jam, 2017-03-03 21:29:41]

Description was changed from

==========
Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to
green up headless_browsertests again.

Also run headless_browsertests with PlzNavigate on the CQ to avoid future
regressions.

BUG=504347
==========

to

==========
Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to
green up headless_browsertests again.

Also run headless_browsertests with PlzNavigate on the CQ to avoid future
regressions.

Lastly, run headless_browsertests on debug Linux bot. This is needed otherwise a
DCHECK will break the CQ but won't show up on the main waterfall.

BUG=504347
==========

[jam, 2017-03-06 15:33:04]

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

[jam, 2017-03-06 15:33:05]

redirecting to Nasko since Sami is OOO

[jam, 2017-03-06 18:27:59]

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

[commit-bot: I haz the power, 2017-03-06 18:28:35]

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

[nasko, 2017-03-06 18:38:49]

LGTM

https://codereview.chromium.org/2727913006/diff/20001/headless/public/util/pr...
File headless/public/util/protocol_handler_request_id_browsertest.cc (right):

https://codereview.chromium.org/2727913006/diff/20001/headless/public/util/pr...
headless/public/util/protocol_handler_request_id_browsertest.cc:189:
run_loop.Run();
The explanation of why this is needed is at
https://codereview.chromium.org/2720543002/#msg25.

[jam, 2017-03-06 18:53:36]

The CQ bit was unchecked by jam@chromium.org

[jam, 2017-03-06 18:53:45]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-03-06 18:53:55]

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, 2017-03-06 19:45:10]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488826425508570,
"parent_rev": "11a3425fff09d249101f8fb0dcd9c00ddb80283b", "commit_rev":
"ebeb36a0ed4b69c72ce8f2c803bf4233d1cea9f1"}

[commit-bot: I haz the power, 2017-03-06 19:45:49]

Description was changed from

==========
Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to
green up headless_browsertests again.

Also run headless_browsertests with PlzNavigate on the CQ to avoid future
regressions.

Lastly, run headless_browsertests on debug Linux bot. This is needed otherwise a
DCHECK will break the CQ but won't show up on the main waterfall.

BUG=504347
==========

to

==========
Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to
green up headless_browsertests again.

Also run headless_browsertests with PlzNavigate on the CQ to avoid future
regressions.

Lastly, run headless_browsertests on debug Linux bot. This is needed otherwise a
DCHECK will break the CQ but won't show up on the main waterfall.

BUG=504347

Review-Url: https://codereview.chromium.org/2727913006
Cr-Commit-Position: refs/heads/master@{#454922}
Committed:
https://chromium.googlesource.com/chromium/src/+/ebeb36a0ed4b69c72ce8f2c803bf...
==========

[commit-bot: I haz the power, 2017-03-06 19:45:50]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/ebeb36a0ed4b69c72ce8f2c803bf...

[sclittle, 2017-03-06 23:27:26]

A revert of this CL (patchset #1 id:20001) has been created in
https://codereview.chromium.org/2732873003/ by sclittle@chromium.org.

The reason for reverting is: Broke Linux debug builder:

https://uberchromegw.corp.google.com/i/chromium.linux/builders/Linux%20Tests%...

https://uberchromegw.corp.google.com/i/chromium.linux/builders/Linux%20Tests%....

[jam1, 2017-03-07 00:09:22]

Description was changed from

==========
Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to
green up headless_browsertests again.

Also run headless_browsertests with PlzNavigate on the CQ to avoid future
regressions.

Lastly, run headless_browsertests on debug Linux bot. This is needed otherwise a
DCHECK will break the CQ but won't show up on the main waterfall.

BUG=504347

Review-Url: https://codereview.chromium.org/2727913006
Cr-Commit-Position: refs/heads/master@{#454922}
Committed:
https://chromium.googlesource.com/chromium/src/+/ebeb36a0ed4b69c72ce8f2c803bf...
==========

to

==========
Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to
green up headless_browsertests again.

Also run headless_browsertests with PlzNavigate on the CQ to avoid future
regressions.

Lastly, run headless_browsertests on debug Linux bot. This is needed otherwise a
DCHECK will break the CQ but won't show up on the main waterfall.

BUG=504347

Review-Url: https://codereview.chromium.org/2727913006
Cr-Commit-Position: refs/heads/master@{#454922}
Committed:
https://chromium.googlesource.com/chromium/src/+/ebeb36a0ed4b69c72ce8f2c803bf...
==========

[jam, 2017-03-07 00:16:13]

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

[jam1, 2017-03-07 00:16:22]

The CQ bit was unchecked by jam@google.com

[jam1, 2017-03-07 00:16:47]

The CQ bit was checked by jam@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-07 00:17:17]

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

[jam1, 2017-03-07 00:59:54]

The CQ bit was checked by jam@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-07 01:00:43]

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

[jam, 2017-03-07 01:39:57]

The CQ bit was unchecked by jam@chromium.org

[jam, 2017-03-07 01:40:37]

The CQ bit was checked by jam@chromium.org

[jam, 2017-03-07 01:40:37]

The patchset sent to the CQ was uploaded after l-g-t-m from nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/2727913006/#ps60001
(title: "attempted fix since can't repro locally")

[commit-bot: I haz the power, 2017-03-07 01:41:59]

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, 2017-03-07 03:55:54]

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

[commit-bot: I haz the power, 2017-03-07 03:55:55]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jam, 2017-03-07 05:39:15]

Description was changed from

==========
Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to
green up headless_browsertests again.

Also run headless_browsertests with PlzNavigate on the CQ to avoid future
regressions.

Lastly, run headless_browsertests on debug Linux bot. This is needed otherwise a
DCHECK will break the CQ but won't show up on the main waterfall.

BUG=504347

Review-Url: https://codereview.chromium.org/2727913006
Cr-Commit-Position: refs/heads/master@{#454922}
Committed:
https://chromium.googlesource.com/chromium/src/+/ebeb36a0ed4b69c72ce8f2c803bf...
==========

to

==========
Disable ProtocolHandlerRequestIdCorrelationTest.RunAsyncTest with PlzNavigate to
green up headless_browsertests again.

Also run headless_browsertests with PlzNavigate on the CQ to avoid future
regressions.

Lastly, run headless_browsertests on debug Linux bot. This is needed otherwise a
DCHECK will break the CQ but won't show up on the main waterfall.

BUG=504347
R=nasko@chromium.org

Review-Url: https://codereview.chromium.org/2727913006
Cr-Original-Commit-Position: refs/heads/master@{#454922}
Committed:
https://chromium.googlesource.com/chromium/src/+/ebeb36a0ed4b69c72ce8f2c803bf...
Review-Url: https://codereview.chromium.org/2727913006 .
Cr-Commit-Position: refs/heads/master@{#455039}
Committed:
https://chromium.googlesource.com/chromium/src/+/04eb2294fb3baa85315da61ef38a...
==========

[jam, 2017-03-07 05:39:17]

Committed patchset #3 (id:60001) manually as
04eb2294fb3baa85315da61ef38a3ce225794f0a (presubmit successful).