Fix buildbot console view in order_by_time mode

Fix buildbot console view in order_by_time mode

Applying a rebased version of http://trac.buildbot.net/ticket/2364

R=agable@chromium.org
BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=296438

[ricow1, 2015-08-21 11:30:02]

Aaron, randomly picked you from log. I hope this is the right approach for
landing a patch like this

[agable, 2015-08-25 18:21:54]

LGTM, yep this is the right approach to merging in changes from the buildbot
project itself.

[ricow1, 2015-08-26 05:14:42]

The CQ bit was checked by ricow@google.com

[commit-bot: I haz the power, 2015-08-26 05:14:51]

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

[commit-bot: I haz the power, 2015-08-26 05:16:21]

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

[commit-bot: I haz the power, 2015-08-26 05:16:21]

Try jobs failed on following builders:
  build_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/build_presubmit...)

[ricow1, 2015-08-26 09:17:41]

The CQ bit was checked by ricow@google.com

[ricow1, 2015-08-26 09:17:41]

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

[commit-bot: I haz the power, 2015-08-26 09:17:48]

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

[commit-bot: I haz the power, 2015-08-26 09:20:22]

Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=296438

[msw, 2015-08-26 23:35:47]

On 2015/08/26 09:20:22, commit-bot: I haz the power wrote:
> Committed patchset #2 (id:20001) as
> http://src.chromium.org/viewvc/chrome?view=rev&revision=296438

Is it possible that this is causing build_presubmit errors?
http://build.chromium.org/p/tryserver.chromium.linux/builders/build_presubmit/
http://build.chromium.org/p/tryserver.chromium.linux/builders/build_presubmit...

** Presubmit ERRORS **
Pylint (603 files using ['--disable=cyclic-import'] on 8 cores) (83.47s) failed
************* Module master.chromium_status_bb8
E:235,35: No value for argument 'revisions' in method call
(no-value-for-parameter)

[agable, 2015-08-26 23:48:26]

On 2015/08/26 at 23:35:47, msw wrote:
> On 2015/08/26 09:20:22, commit-bot: I haz the power wrote:
> > Committed patchset #2 (id:20001) as
> > http://src.chromium.org/viewvc/chrome?view=rev&revision=296438
> 
> Is it possible that this is causing build_presubmit errors?
> http://build.chromium.org/p/tryserver.chromium.linux/builders/build_presubmit/
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/build_presubmit...
> 
> ** Presubmit ERRORS **
> Pylint (603 files using ['--disable=cyclic-import'] on 8 cores) (83.47s)
failed
> ************* Module master.chromium_status_bb8
> E:235,35: No value for argument 'revisions' in method call
(no-value-for-parameter)

Yep, 100%.

[msw, 2015-08-26 23:49:29]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1310303005/ by msw@chromium.org.

The reason for reverting is: Causes build_presubmit errors:
http://build.chromium.org/p/tryserver.chromium.linux/builders/build_presubmit/
http://build.chromium.org/p/tryserver.chromium.linux/builders/build_presubmit...

** Presubmit ERRORS **
Pylint (603 files using ['--disable=cyclic-import'] on 8 cores) (83.47s) failed
************* Module master.chromium_status_bb8
E:235,35: No value for argument 'revisions' in method call
(no-value-for-parameter).

[agable, 2015-08-26 23:55:07]

On 2015/08/26 at 23:48:26, agable wrote:
> On 2015/08/26 at 23:35:47, msw wrote:
> > On 2015/08/26 09:20:22, commit-bot: I haz the power wrote:
> > > Committed patchset #2 (id:20001) as
> > > http://src.chromium.org/viewvc/chrome?view=rev&revision=296438
> > 
> > Is it possible that this is causing build_presubmit errors?
> >
http://build.chromium.org/p/tryserver.chromium.linux/builders/build_presubmit/
> >
http://build.chromium.org/p/tryserver.chromium.linux/builders/build_presubmit...
> > 
> > ** Presubmit ERRORS **
> > Pylint (603 files using ['--disable=cyclic-import'] on 8 cores) (83.47s)
failed
> > ************* Module master.chromium_status_bb8
> > E:235,35: No value for argument 'revisions' in method call
(no-value-for-parameter)
> 
> Yep, 100%.

Some quick grepping shows that seems to be the only usage of any of the
functions whose signatures were changed here. Fix here:
https://codereview.chromium.org/1309253003

[agable, 2015-08-26 23:57:35]

Thanks for the revert; I've updated my fix to be a reland+fix.