Change Rietveld.latest_try_jobs to return try job result details.

Change Rietveld.latest_try_jobs to return try job result details.

In this CL:
 - Make latest_try_jobs return a dict mapping builds to result dicts
   and return that instead of just a list of builds.
 - Move and refactor filter_latest_builds and mark as protected.
 - Remove unused method get_latest_try_job_results.
 - Update tests and other callers.

Reason: In order to make rebaseline-cl not start new try jobs for
builders that already have try jobs started (http://crbug.com/654919),
we need access to the current try job state/result information.

Possible changes to consider before submitting:
 - Maybe we don't want to pass on the result dicts from rietveld as-is,
   but perhaps instead we just want to return a couple pieces of
   information about the job (e.g. is it started? is it finished?)

BUG=654919
Committed: https://crrev.com/6b2243fab21e96c0cb75fc342a7a49ea7274f9d0
Cr-Commit-Position: refs/heads/master@{#424761}

[qyearsley, 2016-10-11 23:28:42]

qyearsley@chromium.org changed reviewers:
+ dpranke@chromium.org, wkorman@chromium.org

[Dirk Pranke, 2016-10-11 23:39:05]

lgtm

[qyearsley, 2016-10-12 00:24:29]

The CQ bit was checked by qyearsley@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-12 00:25:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-12 01:16:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-12 01:16:07]

Dry run: This issue passed the CQ dry run.

[qyearsley, 2016-10-12 16:27:58]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-10-12 16:28:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-12 16:33:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-12 16:37:11]

Description was changed from

==========
Change Rietveld.latest_try_jobs to return try job result details.

In this CL:
 - Make latest_try_jobs return a dict mapping builds to result dicts
   and return that instead of just a list of builds.
 - Move and refactor filter_latest_builds and mark as protected.
 - Remove unused method get_latest_try_job_results.
 - Update tests and other callers.

Reason: In order to make rebaseline-cl not start new try jobs for
builders that already have try jobs started (http://crbug.com/654919),
we need access to the current try job state/result information.

Possible changes to consider before submitting:
 - Maybe we don't want to pass on the result dicts from rietveld as-is,
   but perhaps instead we just want to return a couple pieces of
   information about the job (e.g. is it started? is it finished?)

BUG=654919
==========

to

==========
Change Rietveld.latest_try_jobs to return try job result details.

In this CL:
 - Make latest_try_jobs return a dict mapping builds to result dicts
   and return that instead of just a list of builds.
 - Move and refactor filter_latest_builds and mark as protected.
 - Remove unused method get_latest_try_job_results.
 - Update tests and other callers.

Reason: In order to make rebaseline-cl not start new try jobs for
builders that already have try jobs started (http://crbug.com/654919),
we need access to the current try job state/result information.

Possible changes to consider before submitting:
 - Maybe we don't want to pass on the result dicts from rietveld as-is,
   but perhaps instead we just want to return a couple pieces of
   information about the job (e.g. is it started? is it finished?)

BUG=654919

Committed: https://crrev.com/6b2243fab21e96c0cb75fc342a7a49ea7274f9d0
Cr-Commit-Position: refs/heads/master@{#424761}
==========

[commit-bot: I haz the power, 2016-10-12 16:37:12]

Patchset 1 (id:??) landed as
https://crrev.com/6b2243fab21e96c0cb75fc342a7a49ea7274f9d0
Cr-Commit-Position: refs/heads/master@{#424761}