Sort issues by their age in the commit queue.

context.py

 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 
 class Context(object):
   """Class to hold context about a the current code review and checkout."""
-  def __init__(self, rietveld, checkout, status, server_hooks_missing=False):
+  def __init__(self, rietveld, checkout, status, user,
+               server_hooks_missing=False):
     """
     Args:
       rietveld: Instance of rietveld.Rietveld.
       checkout: Instance of checkout.SvnCheckout
       status: Instance of async_push.AsyncPush.
+      user: The --user option to commit_queue

[Paweł Hajdan Jr., 2014/01/29 02:38:24]

Oh, since this also has Rietveld, does using just rietveld.email work?

If it doesn't, how about changing the documentation of the user argument to say
"Rietveld user" instead?

[Sergey Berezin, 2014/01/29 17:41:43]

Actually, in a dry run, Rietveld receives an empty email (to signal "no
authentication"), while commit queue still has the specified (or default) user
name. We can discuss if this discrepancy should be eliminated later, but for now
I can't rely on rietveld.email.

On 2014/01/29 02:38:24, Paweł Hajdan Jr. wrote:

[Paweł Hajdan Jr., 2014/01/29 18:49:47]

Will it break in dry run? I think that unless it'd be broken it's fine - using
rietveld.email would be simpler it seems.

       server_hooks_missing: True if the project's SVN repository does not have
                             server-side hooks configured.
     """
     self.rietveld = rietveld
     self.checkout = checkout
     self.status = status
     self.server_hooks_missing = server_hooks_missing
+    self.user = user