Update base::is_member_function_pointer to use variadic templates.

Update base::is_member_function_pointer to use variadic templates.

BUG=

Committed: https://crrev.com/489d67130b22457a2c9ee7b6671cc105bf5204c3
Cr-Commit-Position: refs/heads/master@{#296817}

[Aaron Boodman, 2014-09-25 21:46:47]

aa@chromium.org changed reviewers:
+ thakis@chromium.org

[Aaron Boodman, 2014-09-25 22:01:33]

aa@chromium.org changed reviewers:
+ mark@chromium.org

[Mark Mentovai, 2014-09-25 22:03:04]

This should go to thakis or ajwong. I seem to have been left out of the
styleguide/c++/OWNERS party.

[Aaron Boodman, 2014-09-25 22:10:12]

aa@chromium.org changed reviewers:
+ ajwong@chromium.org
- mark@chromium.org

[Nico, 2014-09-25 22:19:12]

lgtm from SFO, away for realz soon

> This should go to thakis or ajwong. I seem to have been left out of the
styleguide/c++/OWNERS party.

'tis a regular base review, not a change to something below styleguide/

https://codereview.chromium.org/598223003/diff/40001/base/template_util_unitt...
File base/template_util_unittest.cc (left):

https://codereview.chromium.org/598223003/diff/40001/base/template_util_unitt...
base/template_util_unittest.cc:127: IsMemberFunctionPointer);
maybe keep this one (without "!")

[Mark Mentovai, 2014-09-25 22:32:37]

On 2014/09/25 22:19:12, Nico (away until Oct 27) wrote:
> lgtm from SFO, away for realz soon
> 
> > This should go to thakis or ajwong. I seem to have been left out of the
> styleguide/c++/OWNERS party.
> 
> 'tis a regular base review, not a change to something below styleguide/

Yes, but if there are two base OWNERS who are also styleguide OWNERS that just
approved this new language feature today, maybe they should be the ones to pass
judgment on this recent addition.

[Aaron Boodman, 2014-09-25 22:34:41]

https://codereview.chromium.org/598223003/diff/40001/base/template_util_unitt...
File base/template_util_unittest.cc (left):

https://codereview.chromium.org/598223003/diff/40001/base/template_util_unitt...
base/template_util_unittest.cc:127: IsMemberFunctionPointer);
On 2014/09/25 22:19:11, Nico (away until Oct 27) wrote:
> maybe keep this one (without "!")

But there's no reason now based on how this works that you'd expect it to be
different than the two argument case, which is already covered above.

[Aaron Boodman, 2014-09-25 22:34:48]

The CQ bit was checked by aa@chromium.org

[commit-bot: I haz the power, 2014-09-25 22:36:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/598223003/40001

[commit-bot: I haz the power, 2014-09-25 23:06:35]

Committed patchset #3 (id:40001) as 5a2810b27c7e103e0c1cd9cc1d0234c407c67f15

[commit-bot: I haz the power, 2014-09-25 23:07:57]

Patchset 3 (id:??) landed as
https://crrev.com/489d67130b22457a2c9ee7b6671cc105bf5204c3
Cr-Commit-Position: refs/heads/master@{#296817}