Remove uses of std::unary_function and std::binary_function.

Remove uses of std::unary_function and std::binary_function.

This patch removes unary and binary function, since those are deprecated
in C++11. They also only provide two typedefs, so this cleans up some
code. In rare cases where the typedefs are actually required, it's
easier to just provide the typedef directly instead of deriving from
one of these functions.

BUG=593407
Committed: https://crrev.com/3abe3303bcbb2b24d7c21228f88114337347b674
Cr-Commit-Position: refs/heads/master@{#380179}

[vmpstr, 2016-03-09 01:47:15]

vmpstr@chromium.org changed reviewers:
+ brettw@chromium.org, jochen@chromium.org, jshin@chromium.org,
mkwst@chromium.org, pkotwicz@chromium.org

[vmpstr, 2016-03-09 01:47:15]

pkotwicz@chromium.org for ash/wm/
jochen@chromium.org for components/ and content/
mkwst@chromium.org for net/cookies/
brettw@chromium.org for tools/gn/
jshin@chromium.org for ui/base/i10n/

Please take a look.

[Mike West, 2016-03-09 07:21:30]

On 2016/03/09 at 01:47:15, vmpstr wrote:
> mkwst@chromium.org for net/cookies/


net/cookies LGTM. Note that I'm removing the method you're poking at in
https://codereview.chromium.org/1767513004, so depending on which lands first,
one of us will need to rebase. :)

[jochen (gone - plz use gerrit), 2016-03-09 15:14:41]

jochen@chromium.org changed reviewers:
+ danakj@chromium.org, thakis@chromium.org

[jochen (gone - plz use gerrit), 2016-03-09 15:14:41]

please file a tracking bug for this

I'd also recommend to add one of the C++11 ninjas to such CLs (thakis / danakj)

[Nico, 2016-03-09 17:54:13]

lgtm

Maybe add to the CL description why they're not useful ("They only provide
typedefs for argument and result types, and we don't need these typedefs
anywhere. In the rare cases where they would be needed, the recommendation is to
manually add these typedefs" I think?)

I think "this removes needless code" is a better motivation then "are
deprecated", but it's the same diff in the end :-)

[vmpstr, 2016-03-09 18:46:29]

Description was changed from

==========
Remove uses of std::unary_function and std::binary_function.

This patch removes unary and binary function, since those are deprecated
in C++11.
==========

to

==========
Remove uses of std::unary_function and std::binary_function.

This patch removes unary and binary function, since those are deprecated
in C++11. They also only provide two typedefs, so this cleans up some
code. In rare cases where the typedefs are actually required, it's
easier to just provide the typedef directly instead of deriving from
one of these functions.

BUG=593407
==========

[vmpstr, 2016-03-09 18:47:06]

On 2016/03/09 17:54:13, Nico wrote:
> lgtm
> 
> Maybe add to the CL description why they're not useful ("They only provide
> typedefs for argument and result types, and we don't need these typedefs
> anywhere. In the rare cases where they would be needed, the recommendation is
to
> manually add these typedefs" I think?)
> 
> I think "this removes needless code" is a better motivation then "are
> deprecated", but it's the same diff in the end :-)

Done. PTAL.

[Nico, 2016-03-09 18:57:00]

lgtm++

[jochen (gone - plz use gerrit), 2016-03-09 18:57:33]

lgtm

[vmpstr, 2016-03-09 19:01:59]

vmpstr@chromium.org changed reviewers:
+ flackr@chromium.org

[vmpstr, 2016-03-09 19:02:01]

+flackr as well for ash/wm/. Please take a look.

[flackr, 2016-03-09 19:29:35]

lgtm thanks!

[flackr, 2016-03-09 19:29:35]

lgtm thanks!

[vmpstr, 2016-03-09 19:31:20]

mkwst@, landing this now, so you might have to rebase :)

[vmpstr, 2016-03-09 19:31:27]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2016-03-09 19:32:23]

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

[commit-bot: I haz the power, 2016-03-09 19:38:26]

Description was changed from

==========
Remove uses of std::unary_function and std::binary_function.

This patch removes unary and binary function, since those are deprecated
in C++11. They also only provide two typedefs, so this cleans up some
code. In rare cases where the typedefs are actually required, it's
easier to just provide the typedef directly instead of deriving from
one of these functions.

BUG=593407
==========

to

==========
Remove uses of std::unary_function and std::binary_function.

This patch removes unary and binary function, since those are deprecated
in C++11. They also only provide two typedefs, so this cleans up some
code. In rare cases where the typedefs are actually required, it's
easier to just provide the typedef directly instead of deriving from
one of these functions.

BUG=593407
==========

[commit-bot: I haz the power, 2016-03-09 19:38:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-09 19:41:20]

Description was changed from

==========
Remove uses of std::unary_function and std::binary_function.

This patch removes unary and binary function, since those are deprecated
in C++11. They also only provide two typedefs, so this cleans up some
code. In rare cases where the typedefs are actually required, it's
easier to just provide the typedef directly instead of deriving from
one of these functions.

BUG=593407
==========

to

==========
Remove uses of std::unary_function and std::binary_function.

This patch removes unary and binary function, since those are deprecated
in C++11. They also only provide two typedefs, so this cleans up some
code. In rare cases where the typedefs are actually required, it's
easier to just provide the typedef directly instead of deriving from
one of these functions.

BUG=593407

Committed: https://crrev.com/3abe3303bcbb2b24d7c21228f88114337347b674
Cr-Commit-Position: refs/heads/master@{#380179}
==========

[commit-bot: I haz the power, 2016-03-09 19:41:21]

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