Fix get_landmines builder()

Fix get_landmines builder()

This meant the script wasn't doing anything on most branches of the ifs.

Is it going to cause problems to fix this? I guess basically everything
will clobber.

R=iannucci@chromium.org, sivachandra@chromium.org

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

[scottmg, 2014-01-17 21:52:43]

wdyt?

[Siva Chandra, 2014-01-17 22:02:50]

This is clearly my miss when I refactored the landmines script.

iannucci@ should say more about the consequences of fixing this.

[scottmg, 2014-01-17 22:11:57]

On 2014/01/17 22:02:50, Siva Chandra wrote:
> This is clearly my miss when I refactored the landmines script.

Yup, np, an easy to make copy-paste error. I just wondered if it would/was
causing troubles elsewhere.
 
> iannucci@ should say more about the consequences of fixing this.

Yeah, I'm a bit worried looking at it some more. I think we might want to also
remove a lot of those older conditions to avoid every single bot clobbering at
the same time and DoSing things.

[iannucci, 2014-01-17 23:02:16]

Yeah, this lgtm, though it will cause a sweep of clobbers.

I will CQ it later (maybe tonight/weekend).

[commit-bot: I haz the power, 2014-01-18 07:05:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scottmg@chromium.org/142093003/20001

[commit-bot: I haz the power, 2014-01-18 16:21:15]

Change committed as 245776