Add --open option to git cl issue.

git_cl.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld."""
 
 from distutils.version import LooseVersion
 import glob
 import json
 import logging
 import optparse
 import os
 import Queue
 import re
 import stat
 import sys
 import textwrap
 import threading
 import urllib2
 import urlparse
+import webbrowser
 
 try:
   import readline  # pylint: disable=F0401,W0611
 except ImportError:
   pass
 
 
 from third_party import colorama
 from third_party import upload
 import breakpad  # pylint: disable=W0611
@@ skipping 2106 matching lines @@
   if args:
     # One arg means set upstream branch.
     RunGit(['branch', '--set-upstream', cl.GetBranch(), args[0]])
     cl = Changelist()
     print "Upstream branch set to " + cl.GetUpstreamBranch()
   else:
     print cl.GetUpstreamBranch()
   return 0
 
 
+def CMDweb(parser, args):
+  """Opens the current CL in the web browser."""
+  _, args = parser.parse_args(args)
+  if args:
+    parser.error('Unrecognized args: %s' % ' '.join(args))
+    return 0

[M-A Ruel, 2013/12/02 21:57:57]

Remove this line since it's not needed, parser.error calls sys.exit(2)

[Lei Zhang, 2013/12/02 22:00:34]

Done. I also fixed all the other instances, including the one I copy+pasted
from.

+
+  issue_url = Changelist().GetIssueURL()
+  if not issue_url:
+    print >> sys.stderr, 'ERROR No issue to open'
+    return 1
+
+  webbrowser.open(issue_url)
+  return 0
+
+
 def CMDset_commit(parser, args):
   """Sets the commit bit to trigger the Commit Queue."""
   _, args = parser.parse_args(args)
   if args:
     parser.error('Unrecognized args: %s' % ' '.join(args))
   cl = Changelist()
   cl.SetFlag('commit', '1')
   return 0
 
 
@@ skipping 173 matching lines @@
         ('AppEngine is misbehaving and returned HTTP %d, again. Keep faith '
           'and retry or visit go/isgaeup.\n%s') % (e.code, str(e)))
 
 
 if __name__ == '__main__':
   # These affect sys.stdout so do it outside of main() to simplify mocks in
   # unit testing.
   fix_encoding.fix_encoding()
   colorama.init()
   sys.exit(main(sys.argv[1:]))