Urls with /@ won't be processed correctly when used as a requirements.

tests/gclient_test.py

 #!/usr/bin/env python
 # 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.
 
 """Unit tests for gclient.py.
 
 See gclient_smoketest.py for integration tests.
 """
 
@@ skipping 163 matching lines @@
 
   def _get_processed(self):
     items = []
     try:
       while True:
         items.append(self.processed.get_nowait())
     except Queue.Empty:
       pass
     return items
 
+  def testAutofix(self):
+    # Invalid urls causes pain when specifying requirements. Make sure it's
+    # auto-fixed.
+    d = gclient.Dependency(
+        None, 'name', 'proto://host/path/@revision', None, None,
+        None, '', True)
+    self.assertEquals('proto://host/path@revision', d.url)
 
 if __name__ == '__main__':
   logging.basicConfig(
       level=[logging.ERROR, logging.WARNING, logging.INFO, logging.DEBUG][
         min(sys.argv.count('-v'), 3)],
       format='%(asctime).19s %(levelname)s %(filename)s:'
               '%(lineno)s %(message)s')
   unittest.main()