Disabling WorkerProcessLauncherTest.Crash

Disabling WorkerProcessLauncherTest.Crash

It's failing on Win dbg bots

BUG=659219
TBR=sammc@chromium.org
NOTRY=true

Committed: https://crrev.com/4af028e242ed5275e2241f291df47557ec57101b
Cr-Commit-Position: refs/heads/master@{#427493}

[dmazzoni, 2016-10-25 21:42:55]

Description was changed from

==========
Disabling WorkerProcessLauncherTest.Crash

It's failing on Win dbg bots

BUG=659219
==========

to

==========
Disabling WorkerProcessLauncherTest.Crash

It's failing on Win dbg bots

BUG=659219
TBR=sammc@chromium.org
NOTRY=true
==========

[dmazzoni, 2016-10-25 21:42:55]

dmazzoni@chromium.org changed reviewers:
+ sammc@chromium.org

[dmazzoni, 2016-10-25 21:43:11]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2016-10-25 21:43:43]

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-10-25 21:49:24]

Description was changed from

==========
Disabling WorkerProcessLauncherTest.Crash

It's failing on Win dbg bots

BUG=659219
TBR=sammc@chromium.org
NOTRY=true
==========

to

==========
Disabling WorkerProcessLauncherTest.Crash

It's failing on Win dbg bots

BUG=659219
TBR=sammc@chromium.org
NOTRY=true
==========

[commit-bot: I haz the power, 2016-10-25 21:49:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-25 21:52:18]

Description was changed from

==========
Disabling WorkerProcessLauncherTest.Crash

It's failing on Win dbg bots

BUG=659219
TBR=sammc@chromium.org
NOTRY=true
==========

to

==========
Disabling WorkerProcessLauncherTest.Crash

It's failing on Win dbg bots

BUG=659219
TBR=sammc@chromium.org
NOTRY=true

Committed: https://crrev.com/4af028e242ed5275e2241f291df47557ec57101b
Cr-Commit-Position: refs/heads/master@{#427493}
==========

[commit-bot: I haz the power, 2016-10-25 21:52:18]

Patchset 1 (id:??) landed as
https://crrev.com/4af028e242ed5275e2241f291df47557ec57101b
Cr-Commit-Position: refs/heads/master@{#427493}

[Nico, 2016-10-26 14:06:42]

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

[Nico, 2016-10-26 14:06:43]

Same question as on the bug: Since a recent CL broke this, why not revert the CL
instead of disabling the test it broke?

[dmazzoni, 2016-10-26 15:24:19]

Hmmm, good point. I can't recall what I was thinking when I decided to
disable the test. Since there's been no response from the owner I'll revert
the change and re-enable the test now.

On Wed, Oct 26, 2016 at 7:06 AM <thakis@chromium.org> wrote:

> Same question as on the bug: Since a recent CL broke this, why not revert
> the CL
> instead of disabling the test it broke?
>
> https://codereview.chromium.org/2450063002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[dmazzoni, 2016-10-26 21:31:12]

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

The reason for reverting is: Re-enabling now that "Use ChannelMojo between the
remoting daemon and network processes" is reverted (
https://codereview.chromium.org/2424353002/)  http://crbug.com/659219
.