Make CollateRevisionHistory handle git and svn at the same time.

Make CollateRevisionHistory handle git and svn at the same time.

BUG=337035
R=agable@chromium.org

Committed: https://chromium.googlesource.com/infra/infra/+/b0b897c663e2a0b0e2f7553c111db8cd0262a394

[zty, 2014-08-20 04:25:43]

please take a look, thanks.

[zty, 2014-08-20 18:58:50]

ptal

[szager1, 2014-08-20 19:09:55]

https://codereview.chromium.org/491583004/diff/20001/infra/services/lkgr_find...
File infra/services/lkgr_finder/lkgr_lib.py (right):

https://codereview.chromium.org/491583004/diff/20001/infra/services/lkgr_find...
infra/services/lkgr_finder/lkgr_lib.py:124: if r is NOREV:
agable and I can't understand why this is necessary for your project, but it
works fine as is for other projects.  Can you please send the stack trace you
got before you made this change?  This is a public bug, so please send the stack
trace via private email.

[zty, 2014-08-20 19:55:46]

sent stack trace via email.

https://codereview.chromium.org/491583004/diff/20001/infra/services/lkgr_find...
File infra/services/lkgr_finder/lkgr_lib.py (right):

https://codereview.chromium.org/491583004/diff/20001/infra/services/lkgr_find...
infra/services/lkgr_finder/lkgr_lib.py:124: if r is NOREV:
On 2014/08/20 19:09:55, szager1 wrote:
> agable and I can't understand why this is necessary for your project, but it
> works fine as is for other projects.  Can you please send the stack trace you
> got before you made this change?  This is a public bug, so please send the
stack
> trace via private email.

Acknowledged.

[szager1, 2014-08-20 22:51:07]

lgtm

[zty, 2014-08-21 00:15:08]

The CQ bit was checked by zty@chromium.org

[commit-bot: I haz the power, 2014-08-21 00:15:41]

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

[commit-bot: I haz the power, 2014-08-21 00:16:46]

Committed patchset #2 (20001) as b0b897c663e2a0b0e2f7553c111db8cd0262a394