Remove references to crash_service, which is no longer needed.

Remove references to crash_service, which is no longer needed.

R=jochen@chromium.org,phajdan.jr@chromium.org,scottmg@chromium.org
BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299779

[Dirk Pranke, 2016-04-05 22:37:28]

Jochen, Paweł, please take a look?

Scott and I *think* that the actual crash_service binary is no longer needed and
isn't being used on the Windows bots.

We think the layout tests still use content_tests_crash_service.exe, but it will
start and stop that on demand (and it is built as part of blink_tests).

However, neither Scott nor I is entire certain about this, so hopefully you two
also have some context here and can take a look?

[Dirk Pranke, 2016-04-05 22:38:57]

On 2016/04/05 22:37:28, Dirk Pranke wrote:
> Jochen, Paweł, please take a look?
> 
> Scott and I *think* that the actual crash_service binary is no longer needed
and
> isn't being used on the Windows bots.
> 
> We think the layout tests still use content_tests_crash_service.exe, but it
will
> start and stop that on demand (and it is built as part of blink_tests).

er, make that content_shell_crash_service.exe

> 
> However, neither Scott nor I is entire certain about this, so hopefully you
two
> also have some context here and can take a look?

[Dirk Pranke, 2016-04-05 22:40:33]

see https://codereview.chromium.org/1858883004/ for the matching source-side
change to delete the code.

[jochen (gone - plz use gerrit), 2016-04-06 04:15:30]

On 2016/04/05 at 22:40:33, dpranke wrote:
> see https://codereview.chromium.org/1858883004/ for the matching source-side
change to delete the code.

blink uses, content shell crash service, yes

lgtm

[Paweł Hajdan Jr., 2016-04-06 13:11:31]

LGTM

Awesome!

[Dirk Pranke, 2016-04-07 22:19:02]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2016-04-07 22:19:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1862773003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1862773003/20001

[commit-bot: I haz the power, 2016-04-07 22:24:31]

Description was changed from

==========
Remove references to crash_service, which is no longer needed.

R=jochen@chromium.org,phajdan.jr@chromium.org,scottmg@chromium.org
BUG=
==========

to

==========
Remove references to crash_service, which is no longer needed.

R=jochen@chromium.org,phajdan.jr@chromium.org,scottmg@chromium.org
BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299779
==========

[commit-bot: I haz the power, 2016-04-07 22:24:33]

Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=299779

[Sergiy Byelozyorov, 2016-04-08 17:04:28]

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

[Sergiy Byelozyorov, 2016-04-08 17:04:29]

Reverted in http://codereview.chromium.org/1871583004: see
http://crbug.com/601762 for more tails.