Remove run_pie from android/tools now that ICS support is deprecated

Remove run_pie from android/tools now that ICS support is deprecated

BUG=473837, 535071
Committed: https://crrev.com/bf76930aa62c84995ba84327173b1542b83186f3
Cr-Commit-Position: refs/heads/master@{#363272}

[pkotwicz, 2015-12-04 03:40:19]

pkotwicz@chromium.org changed reviewers:
+ primiano@chromium.org

[pkotwicz, 2015-12-04 03:40:24]

primiano@ PTAL

[pkotwicz, 2015-12-04 03:49:40]

Hold up, I am not ready for code review yet. I am going through the CL that
added run_pie and trying to undo everything it added

[pkotwicz, 2015-12-04 04:59:05]

Patchset #1 (id:1) has been deleted

[pkotwicz, 2015-12-04 04:59:38]

All's good to go. primiano@ can you please take a look?

[Primiano Tucci (use gerrit), 2015-12-04 10:15:54]

Oh many thanks for cleaning this up.
Can you also remove:

the various run_pie.sha1 in tools/telemetry/bin/android/
(arm64-v8a/run_pie.sha1 and armeabi-v7a/run_pie.sha1)

LGTM

[pkotwicz, 2015-12-04 16:04:20]

pkotwicz@chromium.org changed reviewers:
+ jbudorick@chromium.org, nednguyen@google.com, yfriedman@chromium.org

[pkotwicz, 2015-12-04 16:04:20]

jbudorick@ for build/
nednguyen@ for tools/telemetry
yfriedman@ for tools/android

I have checked and the changes in common.gypi are already mirrored in GN

[nednguyen, 2015-12-04 16:07:20]

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

[nednguyen, 2015-12-04 16:07:20]

what does the run_pie tool do?

https://codereview.chromium.org/1497743004/diff/40001/tools/telemetry/telemet...
File tools/telemetry/telemetry/internal/platform/android_platform_backend.py
(right):

https://codereview.chromium.org/1497743004/diff/40001/tools/telemetry/telemet...
tools/telemetry/telemetry/internal/platform/android_platform_backend.py:62: def
_SetupPrebuiltTools(device):
Kari: do we have a bug about removing this?

[jbudorick, 2015-12-04 16:09:01]

build/ lgtm

[Yaron, 2015-12-04 16:10:51]

lgtm

[pkotwicz, 2015-12-04 17:05:24]

nednguyen@ http://crbug.com/373219 provides a good summary of what the run_pie
tool does. https://codereview.chromium.org/287513002 is the CL which made the
telemetry changes that I am undoing

[nednguyen, 2015-12-04 17:14:46]

Thanks for the context.

lgtm

[pkotwicz, 2015-12-04 18:22:04]

The CQ bit was checked by pkotwicz@chromium.org

[pkotwicz, 2015-12-04 18:22:04]

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

[commit-bot: I haz the power, 2015-12-04 18:24:00]

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

[commit-bot: I haz the power, 2015-12-04 19:51:12]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-12-04 19:52:10]

Description was changed from

==========
Remove run_pie from android/tools now that ICS support is deprecated

BUG=473837, 535071
==========

to

==========
Remove run_pie from android/tools now that ICS support is deprecated

BUG=473837, 535071

Committed: https://crrev.com/bf76930aa62c84995ba84327173b1542b83186f3
Cr-Commit-Position: refs/heads/master@{#363272}
==========

[commit-bot: I haz the power, 2015-12-04 19:52:11]

Patchset 2 (id:??) landed as
https://crrev.com/bf76930aa62c84995ba84327173b1542b83186f3
Cr-Commit-Position: refs/heads/master@{#363272}