Remove -f and swap -s flag from nm call in verify_order.

Remove -f and swap -s flag from nm call in verify_order.

Xcode 7.3 changed the flags for nm, but removing -f and swapping the
order of -s makes it work for both pre-7.3 and 7.3.

BUG=

Committed: https://crrev.com/2034c17e1b8a17785b55f32c4f34512e15c4032d
Cr-Commit-Position: refs/heads/master@{#382570}

[justincohen, 2016-03-07 17:20:32]

Description was changed from

==========
Remove -f from verify_order

BUG=
==========

to

==========
Remove -f from verify_order.

Xcode 7.3 removed the -f flag from nm.  It appears this isn't necessary
anymore.

BUG=
==========

[justincohen, 2016-03-07 18:27:21]

justincohen@chromium.org changed reviewers:
+ thakis@chromium.org

[justincohen, 2016-03-07 18:27:22]

Any ideas on how to test this more completely, other than git try ?

PTAL

[Nico, 2016-03-10 22:46:49]

lgtm if you checked locally than -f doesn't make a difference on nm's output
with recent Xcodes (7.3b and 7.2 at least, I suppose).

[Nico, 2016-03-10 22:46:55]

...and thanks!

[justincohen, 2016-03-22 01:31:04]

I waited for Xcode 7.3 to run another test and had to swap the -s flag around.

Both Xcode 7.2 and 7.2 both output this:
__CTFontManagerUnregisterFontForData
_ChromeAppModeStart_v4
_ChromeMain

PTAL!

[justincohen, 2016-03-22 01:32:14]

Description was changed from

==========
Remove -f from verify_order.

Xcode 7.3 removed the -f flag from nm.  It appears this isn't necessary
anymore.

BUG=
==========

to

==========
Remove -f and swap -s flag from nm call in verify_order.

Xcode 7.3 changed the flags for nm, but removing -f and swapping the
order of -s makes it work for both pre-7.3 and 7.3.

BUG=
==========

[Nico, 2016-03-22 02:22:36]

Lgtm

I'm surprised patch set 2 is necessary.

[justincohen, 2016-03-22 13:59:43]

The CQ bit was checked by justincohen@chromium.org

[commit-bot: I haz the power, 2016-03-22 13:59:49]

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

[commit-bot: I haz the power, 2016-03-22 14:40:27]

Description was changed from

==========
Remove -f and swap -s flag from nm call in verify_order.

Xcode 7.3 changed the flags for nm, but removing -f and swapping the
order of -s makes it work for both pre-7.3 and 7.3.

BUG=
==========

to

==========
Remove -f and swap -s flag from nm call in verify_order.

Xcode 7.3 changed the flags for nm, but removing -f and swapping the
order of -s makes it work for both pre-7.3 and 7.3.

BUG=
==========

[commit-bot: I haz the power, 2016-03-22 14:40:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-22 14:41:45]

Description was changed from

==========
Remove -f and swap -s flag from nm call in verify_order.

Xcode 7.3 changed the flags for nm, but removing -f and swapping the
order of -s makes it work for both pre-7.3 and 7.3.

BUG=
==========

to

==========
Remove -f and swap -s flag from nm call in verify_order.

Xcode 7.3 changed the flags for nm, but removing -f and swapping the
order of -s makes it work for both pre-7.3 and 7.3.

BUG=

Committed: https://crrev.com/2034c17e1b8a17785b55f32c4f34512e15c4032d
Cr-Commit-Position: refs/heads/master@{#382570}
==========

[commit-bot: I haz the power, 2016-03-22 14:41:46]

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