tool --help alphabetizes command line flags

tool --help alphabetizes command line flags

Committed: https://skia.googlesource.com/skia/+/03758b8c4a7f22c74a107905d8e16effab1c0442

[hal.canary, 2015-01-16 15:34:08]

halcanary@google.com changed reviewers:
+ mtklein@google.com

[hal.canary, 2015-01-16 15:34:08]

ptal

[mtklein, 2015-01-18 15:53:10]

lgtm, with some nits

https://codereview.chromium.org/854193003/diff/1/tools/flags/SkCommandLineFla...
File tools/flags/SkCommandLineFlags.cpp (right):

https://codereview.chromium.org/854193003/diff/1/tools/flags/SkCommandLineFla...
tools/flags/SkCommandLineFlags.cpp:239: SkTQSort(&allFlags[0],
&allFlags[allFlags.count() - 1], c);
SkTQSort(&allFlags[0], &allFlags[allFlags.count()-1], CompareFlagsByName()) ?

https://codereview.chromium.org/854193003/diff/1/tools/flags/SkCommandLineFla...
tools/flags/SkCommandLineFlags.cpp:245: while (flag != NULL) {
For consistency, switch this to the for-loop above, or switch the for-loop to
this while-loop?  Weird to see it written two different ways.

[hal.canary, 2015-01-18 16:46:17]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2015-01-18 16:47:10]

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

[commit-bot: I haz the power, 2015-01-18 18:39:29]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/03758b8c4a7f22c74a107905d8e16effab1c0442