Remove hooks until Windows performance issues are sorted.

Remove hooks until Windows performance issues are sorted.

R=iannucci@chromium.org,hinoka@chromium.org
BUG=

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

[Ryan Tseng, 2014-06-19 23:22:17]

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

https://codereview.chromium.org/348703002/diff/1/gclient_scm.py#newcode389
gclient_scm.py:389: self._Clone(revision, url, options)
This will still drop hooks on the first run.

[szager1, 2014-06-19 23:28:52]

PTAL

[iannucci, 2014-06-19 23:31:43]

https://chromiumcodereview.appspot.com/348703002/diff/20001/gclient_scm.py
File gclient_scm.py (right):

https://chromiumcodereview.appspot.com/348703002/diff/20001/gclient_scm.py#ne...
gclient_scm.py:333: self._DisableHooks()
Will this still run for unmanaged repos? If so, do we think devs have their own
hooks?

[szager1, 2014-06-20 00:49:39]

https://chromiumcodereview.appspot.com/348703002/diff/20001/gclient_scm.py
File gclient_scm.py (right):

https://chromiumcodereview.appspot.com/348703002/diff/20001/gclient_scm.py#ne...
gclient_scm.py:333: self._DisableHooks()
On 2014/06/19 23:31:42, iannucci wrote:
> Will this still run for unmanaged repos? If so, do we think devs have their
own
> hooks?

I changed it to only run on managed repos.

[Ryan Tseng, 2014-06-20 00:51:02]

lgtm

[iannucci, 2014-06-20 01:01:45]

lgtm

[szager1, 2014-06-20 15:52:14]

The CQ bit was checked by szager@chromium.org

[commit-bot: I haz the power, 2014-06-20 15:52:59]

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

[commit-bot: I haz the power, 2014-06-20 15:55:08]

Change committed as 278717