[Telemetry] Remove obsolete dependency. Remove py_trace.py since it's not used any where.

[Telemetry] Remove obsolete dependency. Remove py_trace.py since it's not used any where.

BUG=472672
Committed: https://crrev.com/9d1576e4edddb590bf43df6a08e8fe39e72f0448
Cr-Commit-Position: refs/heads/master@{#343308}

[nednguyen, 2015-08-13 19:53:49]

Patchset #1 (id:1) has been deleted

[nednguyen, 2015-08-13 19:53:55]

Patchset #1 (id:20001) has been deleted

[nednguyen, 2015-08-13 19:55:18]

nednguyen@google.com changed reviewers:
+ aiolos@chromium.org, sullivan@chromium.org

[nednguyen, 2015-08-13 19:55:19]

https://codereview.chromium.org/1290113003/diff/40001/tools/telemetry/telemet...
File tools/telemetry/telemetry/TELEMETRY_DEPS (right):

https://codereview.chromium.org/1290113003/diff/40001/tools/telemetry/telemet...
tools/telemetry/telemetry/TELEMETRY_DEPS:8: "third_party/android_testrunner/",
This is used by pylib.

[nednguyen, 2015-08-13 19:55:19]

nednguyen@google.com changed reviewers:
+ aiolos@chromium.org, sullivan@chromium.org

[sullivan, 2015-08-13 19:56:22]

lgtm

[nednguyen, 2015-08-13 19:57:55]

The CQ bit was checked by nednguyen@google.com

[nednguyen, 2015-08-13 19:57:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
sullivan@chromium.org
Link to the patchset: https://codereview.chromium.org/1290113003/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2015-08-13 19:58:26]

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

[nednguyen, 2015-08-13 21:39:27]

The CQ bit was checked by nednguyen@google.com

[nednguyen, 2015-08-13 21:39:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
sullivan@chromium.org
Link to the patchset: https://codereview.chromium.org/1290113003/#ps80001
(title: "Add chromite LICENSE")

[commit-bot: I haz the power, 2015-08-13 21:40:05]

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

[commit-bot: I haz the power, 2015-08-14 00:14:29]

Committed patchset #3 (id:80001)

[commit-bot: I haz the power, 2015-08-14 00:15:34]

Patchset 3 (id:??) landed as
https://crrev.com/9d1576e4edddb590bf43df6a08e8fe39e72f0448
Cr-Commit-Position: refs/heads/master@{#343308}

[binjin, 2015-08-14 08:52:50]

This breaks the Mac bots:
http://build.chromium.org/p/chromium.mac/builders/Mac10.8%20Tests/builds/6574

[1/1]
telemetry.telemetry_dependencies_unittest.TelemetryDependenciesTest.testNoNewTelemetryDependencies
failed unexpectedly 10.0118s:
  Traceback (most recent call last):
    File
"/Volumes/data/b/build/slave/Mac10_8_Tests/build/src/tools/telemetry/telemetry/telemetry_dependencies_unittest.py",
line 57, in testNoNewTelemetryDependencies
      'Extra dependencies:\n%s' % '\n'.join(extra_dep_paths))
  AssertionError: Your patch adds new dependencies to telemetry. Please contact
aiolos@,dtu@, or nednguyen@ on how to proceed with this change. Extra
dependencies:
 
/Volumes/data/b/build/slave/Mac10_8_Tests/build/src/third_party/ply/README.chromium
 
/Volumes/data/b/build/slave/Mac10_8_Tests/build/src/third_party/ply/license.patch
  /Volumes/data/b/build/slave/Mac10_8_Tests/build/src/third_party/ply/LICENSE
  /Volumes/data/b/build/slave/Mac10_8_Tests/build/src/third_party/ply/yacc.py
  /Volumes/data/b/build/slave/Mac10_8_Tests/build/src/third_party/ply/README
 
/Volumes/data/b/build/slave/Mac10_8_Tests/build/src/third_party/ply/__init__.py
  /Volumes/data/b/build/slave/Mac10_8_Tests/build/src/third_party/ply/lex.py

[binjin, 2015-08-14 09:54:37]

Created a CL that partially reverts this one, let me know if anything goes
wrong.

https://codereview.chromium.org/1293743002

[nednguyen, 2015-08-14 15:29:41]

On 2015/08/14 09:54:37, binjin wrote:
> Created a CL that partially reverts this one, let me know if anything goes
> wrong.
> 
> https://codereview.chromium.org/1293743002

+1 to binjin for not reverting the whole thing. I am a bit confused why the cq
bot failed to catch this.