tools/sort_sources.py: sort C++ source file names in build files.

tools/sort_sources.py: sort C++ source file names in build files.

tools/git/move_source_file.py updates C++ source file names in
build files for you, but it does not sort the updated file names.

With this tool, you can sort C++ source file names in build files
(both GYP and GN files supported) automatically. It's not perfect
but should work reasonably well.

BUG=456014
TEST=Run the tool against some build files. The results are included in the patch.

Committed: https://crrev.com/3d23ca0518139be109b6d52a8bf01b55c7ca29e3
Cr-Commit-Position: refs/heads/master@{#315517}

[satorux1, 2015-02-09 08:13:35]

satorux@chromium.org changed reviewers:
+ derat@chromium.org

[Daniel Erat, 2015-02-09 16:30:02]

lgtm

https://codereview.chromium.org/888873003/diff/1/tools/sort_sources.py
File tools/sort_sources.py (right):

https://codereview.chromium.org/888873003/diff/1/tools/sort_sources.py#newcode2
tools/sort_sources.py:2: # Copyright 2014 The Chromium Authors. All rights
reserved.
nit: 2015

https://codereview.chromium.org/888873003/diff/1/tools/sort_sources.py#newcode39
tools/sort_sources.py:39: if re.search(r'^ +[\'"].*\.(c|cc|cpp|h)[\'"],$',
line):
nit: not sure if it's worth doing, but you could also handle places where people
incorrectly use tabs by changing the ' +' to '\s+'

[satorux1, 2015-02-10 06:54:51]

New patchsets have been uploaded after l-g-t-m from derat@chromium.org

[satorux1, 2015-02-10 06:55:05]

https://codereview.chromium.org/888873003/diff/1/tools/sort_sources.py
File tools/sort_sources.py (right):

https://codereview.chromium.org/888873003/diff/1/tools/sort_sources.py#newcode2
tools/sort_sources.py:2: # Copyright 2014 The Chromium Authors. All rights
reserved.
On 2015/02/09 16:30:01, Daniel Erat wrote:
> nit: 2015

Done.

https://codereview.chromium.org/888873003/diff/1/tools/sort_sources.py#newcode39
tools/sort_sources.py:39: if re.search(r'^ +[\'"].*\.(c|cc|cpp|h)[\'"],$',
line):
On 2015/02/09 16:30:01, Daniel Erat wrote:
> nit: not sure if it's worth doing, but you could also handle places where
people
> incorrectly use tabs by changing the ' +' to '\s+'

Done.

[satorux1, 2015-02-10 06:55:26]

The CQ bit was checked by satorux@chromium.org

[commit-bot: I haz the power, 2015-02-10 06:56:01]

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

[commit-bot: I haz the power, 2015-02-10 08:05:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-10 08:05:31]

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