🎑 Revert to sorting android libraries with readelf

Revert to sorting android libraries with readelf

We were using GN's ordering of libraries, but GN isn't actually
providing them in dependency order. This change adds a TODO to add this
feature to GN, and in the meantime uses readelf to determine library
dependency order (as we used to do).

BUG=624791
Committed: https://crrev.com/fb7accc611d326aaee424a64ab974e3c04abe918
Cr-Commit-Position: refs/heads/master@{#403257}

[agrieve, 2016-06-30 17:51:24]

Description was changed from

==========
Revert to sorting android libraries with readelf

We were using GN's ordering of libraries, but GN isn't actually
providing them in dependency order. This change adds a TODO to add this
feature to GN, and in the meantime uses readelf to determine library
dependency order (as we used to do).

BUG=624791
==========

to

==========
Revert to sorting android libraries with readelf

We were using GN's ordering of libraries, but GN isn't actually
providing them in dependency order. This change adds a TODO to add this
feature to GN, and in the meantime uses readelf to determine library
dependency order (as we used to do).

BUG=624791
==========

[agrieve, 2016-06-30 17:51:24]

agrieve@chromium.org changed reviewers:
+ jbudorick@chromium.org

[agrieve, 2016-06-30 17:51:29]

On 2016/06/30 17:51:24, agrieve wrote:
> mailto:agrieve@chromium.org changed reviewers:
> + mailto:jbudorick@chromium.org

[agrieve, 2016-06-30 17:51:34]

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

[commit-bot: I haz the power, 2016-06-30 17:52:05]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Yaron, 2016-06-30 19:29:52]

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

[Yaron, 2016-06-30 19:29:53]

lgtm

[commit-bot: I haz the power, 2016-06-30 19:36:27]

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

[commit-bot: I haz the power, 2016-06-30 19:36:28]

Dry run: This issue passed the CQ dry run.

[agrieve, 2016-06-30 19:42:40]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-06-30 19:43:26]

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-06-30 19:48:45]

Description was changed from

==========
Revert to sorting android libraries with readelf

We were using GN's ordering of libraries, but GN isn't actually
providing them in dependency order. This change adds a TODO to add this
feature to GN, and in the meantime uses readelf to determine library
dependency order (as we used to do).

BUG=624791
==========

to

==========
Revert to sorting android libraries with readelf

We were using GN's ordering of libraries, but GN isn't actually
providing them in dependency order. This change adds a TODO to add this
feature to GN, and in the meantime uses readelf to determine library
dependency order (as we used to do).

BUG=624791
==========

[commit-bot: I haz the power, 2016-06-30 19:48:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-30 19:50:34]

Description was changed from

==========
Revert to sorting android libraries with readelf

We were using GN's ordering of libraries, but GN isn't actually
providing them in dependency order. This change adds a TODO to add this
feature to GN, and in the meantime uses readelf to determine library
dependency order (as we used to do).

BUG=624791
==========

to

==========
Revert to sorting android libraries with readelf

We were using GN's ordering of libraries, but GN isn't actually
providing them in dependency order. This change adds a TODO to add this
feature to GN, and in the meantime uses readelf to determine library
dependency order (as we used to do).

BUG=624791

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

[commit-bot: I haz the power, 2016-06-30 19:50:36]

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