Include <algorithm> if you use functions from <algorithm>.

Include <algorithm> if you use functions from <algorithm>.

I'm trying to remove #include <algorithm> from common headers because
it takes a long time to compile, but then code that actually need
<algorithm> need to include that header themselves.

This will bring no functional change.

BUG=563433
Committed: https://crrev.com/e77640ddf41053b6687984b19b161530c26d8755
Cr-Commit-Position: refs/heads/master@{#365830}

[Daniel Bratell, 2015-12-16 12:33:34]

bratell@opera.com changed reviewers:
+ jochen@chromium.org

[Daniel Bratell, 2015-12-16 12:33:35]

Jochen, can you please take a look? I'm just adding #include <algorithm> here to
maybe in the future be able to remove them from common header files.

[jochen (gone - plz use gerrit), 2015-12-17 14:22:18]

lgtm

[Daniel Bratell, 2015-12-17 14:37:08]

The CQ bit was checked by bratell@opera.com

[commit-bot: I haz the power, 2015-12-17 14:37:32]

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

[haraken, 2015-12-17 14:48:20]

haraken@chromium.org changed reviewers:
+ haraken@chromium.org, yutak@chromium.org

[haraken, 2015-12-17 14:48:20]

+yutak

LGTM

[commit-bot: I haz the power, 2015-12-17 16:05:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-17 16:06:06]

Description was changed from

==========
Include <algorithm> if you use functions from <algorithm>.

I'm trying to remove #include <algorithm> from common headers because
it takes a long time to compile, but then code that actually need
<algorithm> need to include that header themselves.

This will bring no functional change.

BUG=563433
==========

to

==========
Include <algorithm> if you use functions from <algorithm>.

I'm trying to remove #include <algorithm> from common headers because
it takes a long time to compile, but then code that actually need
<algorithm> need to include that header themselves.

This will bring no functional change.

BUG=563433

Committed: https://crrev.com/e77640ddf41053b6687984b19b161530c26d8755
Cr-Commit-Position: refs/heads/master@{#365830}
==========

[commit-bot: I haz the power, 2015-12-17 16:06:07]

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

[Yuta Kitamura, 2015-12-18 04:05:08]

This is reasonable, LGTM.