Ignore anything after '@' in SCMWrapper.DoesRemoteURLMatch

Ignore anything after '@' in SCMWrapper.DoesRemoteURLMatch

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

[borenet, 2014-03-27 15:01:40]

This seems like a reasonable fix to me.  Thoughts?

[iannucci, 2014-03-28 18:46:59]

lgtm

[iannucci, 2014-03-28 18:47:49]

Actually, is there anything we can do to ensure that this isn't splitting a
username? What about using urlparse and the splitting the @ out of the path (if
it's present?)

[borenet, 2014-03-28 20:39:19]

On 2014/03/28 18:47:49, iannucci wrote:
> Actually, is there anything we can do to ensure that this isn't splitting a
> username? What about using urlparse and the splitting the @ out of the path
(if
> it's present?)

How about I use gclient_utils.SplitUrlRevision and if that does the wrong thing
we fix it?

[borenet, 2014-03-31 17:00:35]

On 2014/03/28 20:39:19, borenet wrote:
> On 2014/03/28 18:47:49, iannucci wrote:
> > Actually, is there anything we can do to ensure that this isn't splitting a
> > username? What about using urlparse and the splitting the @ out of the path
> (if
> > it's present?)
> 
> How about I use gclient_utils.SplitUrlRevision and if that does the wrong
thing
> we fix it?

Friendly ping.

[iannucci, 2014-03-31 21:11:07]

lgtm

[borenet, 2014-04-01 16:39:39]

The CQ bit was checked by borenet@google.com

[commit-bot: I haz the power, 2014-04-01 16:39:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/borenet@google.com/214813002/20001

[commit-bot: I haz the power, 2014-04-01 16:41:37]

Change committed as 260887