Added common git 'fetch' function

Added common git 'fetch' function

Replaced multiple various invocations of 'git fetch' with calls to a common
fetch function.

BUG=373504
TEST=localtest

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

[dnj, 2014-06-10 20:51:34]

PTAL

Depends on: https://codereview.chromium.org/326153003/

[dnj, 2014-06-16 15:38:21]

ping (+mmoss, +nodir)

[dnj, 2014-06-16 18:33:28]

+maruel, since he knows 'gclient'.

[dnj, 2014-06-23 20:38:51]

PING (maruel@, I'd like you to be primary reviewer since the others don't have
time right now. Please let me know if you don't either!)

[M-A Ruel, 2014-06-24 21:11:52]

https://codereview.chromium.org/327803006/diff/1/gclient_scm.py
File gclient_scm.py (right):

https://codereview.chromium.org/327803006/diff/1/gclient_scm.py#newcode1040
gclient_scm.py:1040: def _Run(self, args, options, _show_header=True, **kwargs):
Why an underscore prefix?

[dnj, 2014-06-24 21:45:23]

https://codereview.chromium.org/327803006/diff/1/gclient_scm.py
File gclient_scm.py (right):

https://codereview.chromium.org/327803006/diff/1/gclient_scm.py#newcode1040
gclient_scm.py:1040: def _Run(self, args, options, _show_header=True, **kwargs):
On 2014/06/24 21:11:52, M-A Ruel wrote:
> Why an underscore prefix?

The thought is that since this is opaquely wrapping 'CheckCallAndFilter', it
avoid a name conflict. This is pretty lame though; I'll remove the underscore.

[M-A Ruel, 2014-06-25 00:29:31]

lgtm

https://codereview.chromium.org/327803006/diff/20001/gclient_scm.py
File gclient_scm.py (right):

https://codereview.chromium.org/327803006/diff/20001/gclient_scm.py#newcode1039
gclient_scm.py:1039: if remote is None:
remote = remote or self.remote

[dnj, 2014-06-25 00:36:56]

Thanks for reviewing this!

https://codereview.chromium.org/327803006/diff/20001/gclient_scm.py
File gclient_scm.py (right):

https://codereview.chromium.org/327803006/diff/20001/gclient_scm.py#newcode1039
gclient_scm.py:1039: if remote is None:
On 2014/06/25 00:29:31, M-A Ruel wrote:
> remote = remote or self.remote

Done.

[dnj, 2014-06-25 00:37:25]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2014-06-25 00:37:50]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dnj@chromium.org/327803006/40001

[commit-bot: I haz the power, 2014-06-25 00:39:35]

Change committed as 279560