Refactoring auto-bisect bot

Refactoring auto-bisect bot

 - BisectPerformanceMetrics class:
   - Added src_cwd to constructor to allow easier mocking.
     Using os.getcwd() creates non-obvious expectations from
     the caller.
   - Exposed printer field to allow it being reused outside.
   - Added a comment explaining arguments to the constructor.
 - BisectOptions class:
   - Changed from deprecated optparse to argparse
   - Split _CreateCommandLineParser
   - Moved logic from ParseCommandLine to argparse where
     possible
 - Removed code related to ChromeOS

BUG=425652
R=qyearsley@chromium.org
CC=ojan@chromium.org

Committed: https://crrev.com/9cf2803cdd2ae9bb74a2e42231d9bcb1f8b8e4f5
Cr-Commit-Position: refs/heads/master@{#302256}

[qyearsley, 2014-10-31 05:41:40]

On 2014/10/29 15:29:24, Sergiy Byelozyorov wrote:

LGTM :-D

This CL is quite large and involves a few different changes, so in general I
might recommend splitting it up; but anyway, after looking over the changes, I
agree with all of them and I think that all of them will make the code easier to
deal with in the future. Also, thanks for changing to argparse as well.

[Sergiy Byelozyorov, 2014-10-31 13:11:02]

On 2014/10/31 05:41:40, qyearsley wrote:
> On 2014/10/29 15:29:24, Sergiy Byelozyorov wrote:
> 
> LGTM :-D
> 
> This CL is quite large and involves a few different changes, so in general I
> might recommend splitting it up; but anyway, after looking over the changes, I
> agree with all of them and I think that all of them will make the code easier
to
> deal with in the future. Also, thanks for changing to argparse as well.

Yes, sorry for dumping so much into it, but when I start refactoring one aspect
of the code I can't resist the urge of refactoring others. Most importantly it
is often easier to make multiple refactorings at the same time as they are
connected and would otherwise require to re-refactor same code multiple times.
For the same reason it would be hard to split this CL into multiple ones.

[Sergiy Byelozyorov, 2014-10-31 13:11:08]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2014-10-31 13:11:38]

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

[commit-bot: I haz the power, 2014-10-31 13:52:20]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2014-10-31 13:53:04]

Patchset 4 (id:??) landed as
https://crrev.com/9cf2803cdd2ae9bb74a2e42231d9bcb1f8b8e4f5
Cr-Commit-Position: refs/heads/master@{#302256}