Finally get rid of depot_tools' breakpad.

Finally get rid of depot_tools' breakpad.

R=maruel@chromium.org
BUG=

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

[tandrii(chromium), 2016-02-10 14:07:41]

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

[commit-bot: I haz the power, 2016-02-10 14:07:49]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1689633002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1689633002/1

[commit-bot: I haz the power, 2016-02-10 14:15:58]

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

[commit-bot: I haz the power, 2016-02-10 14:15:58]

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

[tandrii(chromium), 2016-02-10 16:29:23]

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

[commit-bot: I haz the power, 2016-02-10 16:29:27]

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

[tandrii(chromium), 2016-02-10 16:32:42]

PTAL

[commit-bot: I haz the power, 2016-02-10 16:32:44]

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

[commit-bot: I haz the power, 2016-02-10 16:32:45]

Dry run: This issue passed the CQ dry run.

[M-A Ruel, 2016-02-10 16:34:52]

lgtm

[tandrii(chromium), 2016-02-10 16:38:02]

The CQ bit was checked by tandrii@chromium.org

[commit-bot: I haz the power, 2016-02-10 16:38:06]

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

[commit-bot: I haz the power, 2016-02-10 16:40:13]

Description was changed from

==========
Finally get rid of depot_tools' breakpad.

R=maruel@chromium.org
BUG=
==========

to

==========
Finally get rid of depot_tools' breakpad.

R=maruel@chromium.org
BUG=

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

[commit-bot: I haz the power, 2016-02-10 16:40:13]

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

[Nico, 2016-02-10 16:51:25]

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

The reason for reverting is: All chromium bots depend on this: 

________ running '/usr/bin/python src/build/android/play_services/update.py
download' in '/b/build/slave/linux/build'
Traceback (most recent call last):
  File "src/build/android/play_services/update.py", line 29, in <module>
    import find_depot_tools  # pylint: disable=import-error,unused-import
  File "/b/build/slave/linux/build/src/build/find_depot_tools.py", line 49, in
<module>
    import breakpad
ImportError: No module named breakpad

https://www.google.com/url?hl=en&q=http://build.chromium.org/p/tryserver.chro....

[sdefresne, 2016-02-10 16:52:24]

On 2016/02/10 at 16:40:13, commit-bot wrote:
> Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=298710

This broke Chrome on iOS downstream tree as we sync at branch point (last merge
synced up to 6137b72d44c0391a39e2a8f9cad9efb4706880e9) but our bots and
developers got this CL at their latest "gclient" invocation :-(

[Dirk Pranke, 2016-02-10 17:12:35]

I'm even confused as to why we would do this. Do we think breakpad is no longer
useful?

[M-A Ruel, 2016-02-10 17:22:49]

On 2016/02/10 17:12:35, Dirk Pranke wrote:
> I'm even confused as to why we would do this. Do we think breakpad is no
longer
> useful?

This breakpad is not used.

[Dirk Pranke, 2016-02-10 17:25:56]

On 2016/02/10 17:22:49, M-A Ruel wrote:
> On 2016/02/10 17:12:35, Dirk Pranke wrote:
> > I'm even confused as to why we would do this. Do we think breakpad is no
> longer
> > useful?
> 
> This breakpad is not used.

What do you mean? Clearly the other python files were still importing the file.
Do you mean no one was looking
at the exceptions that got posted, or that stopped working, or something?

[tandrii(chromium), 2016-02-10 17:28:43]

On 2016/02/10 17:12:35, Dirk Pranke wrote:
> I'm even confused as to why we would do this. Do we think breakpad is no
longer
> useful?

Because nobody checks its stack traces anyway, though there are also good
privacy concerns for people naming their machine hosts as *.google.com or
*.chromium.org.

[tandrii(chromium), 2016-02-10 17:29:15]

On 2016/02/10 17:28:43, tandrii(chromium) wrote:
> On 2016/02/10 17:12:35, Dirk Pranke wrote:
> > I'm even confused as to why we would do this. Do we think breakpad is no
> longer
> > useful?
> 
> Because nobody checks its stack traces anyway, though there are also good
> privacy concerns for people naming their machine hosts as *.google.com or
> *.chromium.org.

and i filed a bug http://crbug.com/585837

[tandrii(chromium), 2016-02-10 17:29:15]

On 2016/02/10 17:28:43, tandrii(chromium) wrote:
> On 2016/02/10 17:12:35, Dirk Pranke wrote:
> > I'm even confused as to why we would do this. Do we think breakpad is no
> longer
> > useful?
> 
> Because nobody checks its stack traces anyway, though there are also good
> privacy concerns for people naming their machine hosts as *.google.com or
> *.chromium.org.

and i filed a bug http://crbug.com/585837