Put patches to refs/patches/<username>/<patchname>

Put patches to refs/patches/<user>/<name>

When using Git, trychange.py used to put patches to refs/patches/<name>
refs, where <name> is a name of a patch. Now the using the username as a part
of the path.

This way we have better control over permissions per user basis

R=agable@chromium.org, iannucci@chromium.org
TEST=Sent a change to quickoffice try server. Master and slave do not
     need to be modified because the ref file in master branch points to
     the correct branch name

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

[nodir, 2014-04-09 22:44:10]

PTAL

[agable, 2014-04-11 20:53:10]

https://codereview.chromium.org/231663005/diff/1/trychange.py
File trychange.py (right):

https://codereview.chromium.org/231663005/diff/1/trychange.py#newcode514
trychange.py:514: def _PrepareDescriptionAndPatchFiles(description, options,
use_username=True):
Why add this complexity? The ref is not actually a path to the file -- just
because the ref contains the username doesn't mean the filename shouldn't.

https://codereview.chromium.org/231663005/diff/1/trychange.py#newcode648
trychange.py:648: os.path.basename(patch_filename).replace(' ','-'))
use underscores instead of hyphens

[nodir, 2014-04-11 23:49:59]

Done!

https://codereview.chromium.org/231663005/diff/1/trychange.py
File trychange.py (right):

https://codereview.chromium.org/231663005/diff/1/trychange.py#newcode514
trychange.py:514: def _PrepareDescriptionAndPatchFiles(description, options,
use_username=True):
Good point

https://codereview.chromium.org/231663005/diff/1/trychange.py#newcode648
trychange.py:648: os.path.basename(patch_filename).replace(' ','-'))
On 2014/04/11 20:53:11, agable wrote:
> use underscores instead of hyphens

Done.

[agable, 2014-04-12 00:56:38]

LGTM

[nodir, 2014-04-12 00:59:40]

The CQ bit was checked by nodir@chromium.org

[commit-bot: I haz the power, 2014-04-12 00:59:55]

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

[commit-bot: I haz the power, 2014-04-12 01:01:35]

Change committed as 263432