Refactor gmock_mutant.h

Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as MakeUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which no one use.

BUG=554299
Committed: https://crrev.com/651fa1249144a42032c20e4e82f87e995f5d7dff
Cr-Commit-Position: refs/heads/master@{#374689}

[tzik, 2016-02-08 06:09:02]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-08 06:09:22]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1680643002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1680643002/1

[tzik, 2016-02-08 06:20:23]

Description was changed from

==========
Refactor gmock_mutant.h

This CL contains:
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which is no one use.

BUG=
==========

to

==========
Refactor gmock_mutant.h

This CL contains:
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which is no one use.

BUG=554299
==========

[tzik, 2016-02-08 06:22:22]

Description was changed from

==========
Refactor gmock_mutant.h

This CL contains:
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which is no one use.

BUG=554299
==========

to

==========
Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as GetUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which is no one use.

BUG=554299
==========

[commit-bot: I haz the power, 2016-02-08 06:57:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-08 06:57:20]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[tzik, 2016-02-08 07:40:14]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-08 07:40:29]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1680643002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1680643002/20001

[commit-bot: I haz the power, 2016-02-08 08:31:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-08 08:31:18]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[tzik, 2016-02-08 08:42:29]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-08 08:43:08]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1680643002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1680643002/40001

[tzik, 2016-02-08 09:17:55]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-08 09:18:48]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1680643002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1680643002/60001

[tzik, 2016-02-08 09:21:23]

Description was changed from

==========
Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as GetUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which is no one use.

BUG=554299
==========

to

==========
Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as MakeUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which is no one use.

BUG=554299
==========

[tzik, 2016-02-08 09:27:35]

Description was changed from

==========
Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as MakeUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which is no one use.

BUG=554299
==========

to

==========
Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as MakeUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which no one use.

BUG=554299
==========

[tzik, 2016-02-08 10:29:14]

tzik@chromium.org changed reviewers:
+ dcheng@chromium.org, dmazzoni@chromium.org, dpranke@chromium.org,
kelvinp@chromium.org, mmenke@chromium.org, thakis@chromium.org

[tzik, 2016-02-08 10:29:15]

PTAL

[commit-bot: I haz the power, 2016-02-08 11:48:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-08 11:48:57]

Dry run: This issue passed the CQ dry run.

[tzik, 2016-02-08 12:40:16]

Oops, sorry. I forgot to guide which directory you guys might be interested in.

Could you take a look to them?
thakis: base/bind.h
dmazzoni: chrome/browser/speech/extension_api/tts_extension_apitest.cc
mmenke: net/quic/
kelvinp: remoting/host/
dpranke: testing/

[mmenke, 2016-02-08 12:42:52]

On 2016/02/08 12:40:16, tzik wrote:
> Oops, sorry. I forgot to guide which directory you guys might be interested
in.
> 
> Could you take a look to them?
> thakis: base/bind.h
> dmazzoni: chrome/browser/speech/extension_api/tts_extension_apitest.cc
> mmenke: net/quic/
> kelvinp: remoting/host/
> dpranke: testing/

net/quic/ LGTM

[Nico, 2016-02-08 15:37:12]

lgtm, nice!

[joedow, 2016-02-08 16:22:22]

joedow@chromium.org changed reviewers:
+ joedow@chromium.org

[joedow, 2016-02-08 16:22:22]

remoting/ lgtm.

[Dirk Pranke, 2016-02-08 21:20:03]

I am totally unqualified to review this, so I defer to thakis@. lgtm based on
his approval.

[tzik, 2016-02-10 03:53:40]

dmazzoni: ping. Could you review tts_extension_apitest.cc?

[Nico, 2016-02-10 13:45:24]

tts_extension_apitest.cc lgtm as chrome/OWNER (it's just a refactoring change,
so I feel comfortable stamping this)

[tzik, 2016-02-10 16:36:42]

On 2016/02/10 13:45:24, Nico wrote:
> tts_extension_apitest.cc lgtm as chrome/OWNER (it's just a refactoring change,
> so I feel comfortable stamping this)

Hm, OK. So let me land this now.

[tzik, 2016-02-10 16:36:51]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2016-02-10 16:37:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1680643002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1680643002/60001

[commit-bot: I haz the power, 2016-02-10 16:43:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-10 16:43:36]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[tzik, 2016-02-10 17:03:52]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2016-02-10 17:06:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1680643002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1680643002/60001

[dmazzoni, 2016-02-10 17:43:06]

lgtm

[tzik, 2016-02-10 17:43:45]

On 2016/02/10 17:43:06, dmazzoni wrote:
> lgtm

Thanks!

[commit-bot: I haz the power, 2016-02-10 18:30:34]

Description was changed from

==========
Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as MakeUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which no one use.

BUG=554299
==========

to

==========
Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as MakeUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which no one use.

BUG=554299
==========

[commit-bot: I haz the power, 2016-02-10 18:30:35]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-02-10 18:32:15]

Description was changed from

==========
Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as MakeUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which no one use.

BUG=554299
==========

to

==========
Refactor gmock_mutant.h

This CL contains:
 * Expose resulting type of Bind as MakeUnboundRunType.
 * Refactor testing::CreateFunctor using base::Bind.
 * Remove generate_gmock_mutant.py.
 * Change argument ordering of testing::CreateFunctor to align to
   base::Bind.
 * Remove LATE_OBJECT_BINDING support, which no one use.

BUG=554299

Committed: https://crrev.com/651fa1249144a42032c20e4e82f87e995f5d7dff
Cr-Commit-Position: refs/heads/master@{#374689}
==========

[commit-bot: I haz the power, 2016-02-10 18:32:18]

Patchset 4 (id:??) landed as
https://crrev.com/651fa1249144a42032c20e4e82f87e995f5d7dff
Cr-Commit-Position: refs/heads/master@{#374689}