Add #include <algorithm> to code that is missing it.

Add #include <algorithm> to code that is missing it.

Including <algorithm> is compile time expensive but not including it
when it's used causes compiler errors when other code removes their
includes of <algorithm>.

To prevent possible future compilation errors with C++11 stl, I've
also added #include <utility> to headers that use std::swap since
<algorithm> isn't certain to provide std::swap.

R=thakis@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194281

[Daniel Bratell, 2015-04-14 15:49:36]

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

[Daniel Bratell, 2015-04-14 15:49:37]

thakis, this ended up being the opposite of what I talked about. Instead of
removing #include <algorithm> I've added them to the places that had forgotten
it (which became obvious when I tried to remove a few in wtf).

[Daniel Bratell, 2015-04-16 15:01:59]

Ping thakis, just cleanup, but if you have 5 minutes.

[Nico, 2015-04-16 21:16:56]

i stopped saying the same thing in wtf refptr, please self-review the rest based
on my comments

https://codereview.chromium.org/1088843002/diff/20001/Source/platform/geometr...
File Source/platform/geometry/DoublePoint.h (right):

https://codereview.chromium.org/1088843002/diff/20001/Source/platform/geometr...
Source/platform/geometry/DoublePoint.h:12: #include <algorithm> // For
std::min/std::max
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/platform/geometr...
File Source/platform/geometry/FloatBox.h (right):

https://codereview.chromium.org/1088843002/diff/20001/Source/platform/geometr...
Source/platform/geometry/FloatBox.h:34: #include <algorithm> // For
std::min/std::max
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/platform/geometr...
File Source/platform/geometry/LayoutPoint.h (right):

https://codereview.chromium.org/1088843002/diff/20001/Source/platform/geometr...
Source/platform/geometry/LayoutPoint.h:38: #include <algorithm> // For
std::min/std::max
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/platform/graphic...
File Source/platform/graphics/filters/SpotLightSource.h (right):

https://codereview.chromium.org/1088843002/diff/20001/Source/platform/graphic...
Source/platform/graphics/filters/SpotLightSource.h:27: #include <algorithm> //
For std::min/std::max
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/Alignment.h
File Source/wtf/Alignment.h (right):

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/Alignment.h#...
Source/wtf/Alignment.h:24: #include <algorithm> // For std::swap
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/Alignment.h#...
Source/wtf/Alignment.h:24: #include <algorithm> // For std::swap
also include <utility> else the file isn't iwyu-correct in c++11 implementation
:-/

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/ArrayBufferV...
File Source/wtf/ArrayBufferView.h (right):

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/ArrayBufferV...
Source/wtf/ArrayBufferView.h:35: #include <algorithm> // For std::min
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/MathExtras.h
File Source/wtf/MathExtras.h (right):

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/MathExtras.h...
Source/wtf/MathExtras.h:31: #include <cmath> // For std::isinf and std::isnan
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/MathExtras.h...
Source/wtf/MathExtras.h:32: #include <limits> // For std::numeric_limits
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/OwnPtr.h
File Source/wtf/OwnPtr.h (right):

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/OwnPtr.h#new...
Source/wtf/OwnPtr.h:29: #include <algorithm> // For std::swap
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/OwnPtr.h#new...
Source/wtf/OwnPtr.h:29: #include <algorithm> // For std::swap
add <utiility> :-/

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/RawPtr.h
File Source/wtf/RawPtr.h (right):

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/RawPtr.h#new...
Source/wtf/RawPtr.h:36: #include <algorithm> // For std::swap
remove comment

https://codereview.chromium.org/1088843002/diff/20001/Source/wtf/RawPtr.h#new...
Source/wtf/RawPtr.h:37: #include <stdint.h>
add <utility> :-/

[Daniel Bratell, 2015-04-21 09:33:17]

Done!

I was a bit unclear whether <algorithm> will include <utility> or not in C++11.
In the original proposal (implemented by the glib in my Ubuntu 14.04), UK-300,
<algorithm> was specified to include <utility>. That proposal was rejected and
something else was done later to move std::swap to <algorithm> and I have not
been able to see how that one is specified.

So I've added <utility> for now.

[Nico, 2015-04-22 16:55:54]

lgtm :-/

[Daniel Bratell, 2015-04-23 09:48:50]

The CQ bit was checked by bratell@opera.com

[commit-bot: I haz the power, 2015-04-23 09:49:06]

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

[commit-bot: I haz the power, 2015-04-23 11:15:11]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194281