Emit partial results if binary_size is interrupted.

Emit partial results if binary_size is interrupted.

Instead of just throwing away everything if the user presses
Ctrl-c, write down whatever data we have to the disk. By asking for
the largest symbols first, it's sometimes possible to get a quite
good picture after only 10-15% of the total runtime.

BUG=377466
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275053

[Primiano Tucci (use gerrit), 2014-05-27 09:31:46]

LGTM % 2 nits.

https://codereview.chromium.org/305503002/diff/1/tools/binary_size/run_binary...
File tools/binary_size/run_binary_size_analysis.py (right):

https://codereview.chromium.org/305503002/diff/1/tools/binary_size/run_binary...
tools/binary_size/run_binary_size_analysis.py:483: print('Interrupting - killing
subprocesses.')
Can you move this (and indent) to line 481, in the execpt block, and avoid
checking the condition here (like you did below on 490)?

https://codereview.chromium.org/305503002/diff/1/tools/binary_size/run_binary...
tools/binary_size/run_binary_size_analysis.py:490: print('Patience you must have
my young padawan.')
:-D

[Andrew Hayden (chromium.org), 2014-05-27 10:38:32]

LGTM % primiano

[Primiano Tucci (use gerrit), 2014-05-27 11:54:34]

On 2014/05/27 10:38:32, Andrew Hayden wrote:
> LGTM % primiano

Does it mean that the CL does, but I don't look good?
/me looking for a hairdresser. :P

[Daniel Bratell, 2014-06-04 13:35:29]

On 2014/05/27 09:31:46, Primiano Tucci wrote:
> LGTM % 2 nits.
> 
>
https://codereview.chromium.org/305503002/diff/1/tools/binary_size/run_binary...
> File tools/binary_size/run_binary_size_analysis.py (right):
> 
>
https://codereview.chromium.org/305503002/diff/1/tools/binary_size/run_binary...
> tools/binary_size/run_binary_size_analysis.py:483: print('Interrupting -
killing
> subprocesses.')
> Can you move this (and indent) to line 481, in the execpt block, and avoid
> checking the condition here (like you did below on 490)?

Yes, that would be better. No idea how it ended up this way. :-)

Fixed.

[Daniel Bratell, 2014-06-05 07:15:12]

The CQ bit was checked by bratell@opera.com

[commit-bot: I haz the power, 2014-06-05 07:17:34]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bratell@opera.com/305503002/20001

[commit-bot: I haz the power, 2014-06-05 09:16:49]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_gn_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_gn_rel...)

[commit-bot: I haz the power, 2014-06-05 09:37:21]

Change committed as 275053