[Findit] Fix bug when displaying try job result for swamring tests.

[Findit] Fix bug when displaying try job result for swamring tests.

The bug is caused by step names with platform. Make sure using step_name without platform when looking for try jobs.

BUG=596200
Committed: https://chromium.googlesource.com/infra/infra/+/d355f4edca7dba22111569ff730714ff36e2e8e6

[chanli, 2016-03-21 01:02:13]

chanli@chromium.org changed reviewers:
+ lijeffrey@chromium.org, stgao@chromium.org

[chanli, 2016-03-21 01:02:14]

PTAL.

[stgao, 2016-03-21 18:01:17]

lgtm with nits.

https://codereview.chromium.org/1820753002/diff/1/appengine/findit/handlers/h...
File appengine/findit/handlers/handlers_util.py (right):

https://codereview.chromium.org/1820753002/diff/1/appengine/findit/handlers/h...
appengine/findit/handlers/handlers_util.py:149: step =
culprit_info.get('step_no_platform', culprit_info['step'])
nit: is this a step name or a dict containing info about the step?
Same for ``test`` below.

[lijeffrey, 2016-03-21 18:18:21]

lgtm

[chanli, 2016-03-21 18:27:06]

The CQ bit was checked by chanli@chromium.org

[chanli, 2016-03-21 18:27:06]

The patchset sent to the CQ was uploaded after l-g-t-m from stgao@chromium.org,
lijeffrey@chromium.org
Link to the patchset: https://codereview.chromium.org/1820753002/#ps20001
(title: ".")

[commit-bot: I haz the power, 2016-03-21 18:27:08]

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

[commit-bot: I haz the power, 2016-03-21 18:32:25]

Description was changed from

==========
[Findit] Fix bug when displaying try job result for swamring tests.

The bug is caused by step names with platform. Make sure using step_name without
platform when looking for try jobs.

BUG=596200
==========

to

==========
[Findit] Fix bug when displaying try job result for swamring tests.

The bug is caused by step names with platform. Make sure using step_name without
platform when looking for try jobs.

BUG=596200

Committed:
https://chromium.googlesource.com/infra/infra/+/d355f4edca7dba22111569ff73071...
==========

[commit-bot: I haz the power, 2016-03-21 18:32:26]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/infra/infra/+/d355f4edca7dba22111569ff73071...