Add skstd::move and skstd::forward.

Add skstd::move and skstd::forward.

Since we can't use the c++11 standard library, re-implement the bits
we want to use now.

TBR=reed@google.com
The next CL is try to move SkTemplates into private.

Committed: https://skia.googlesource.com/skia/+/2bd028590d9db472e5b9ee44ca8961be5c036432

[bungeman-skia, 2015-08-05 14:51:42]

bungeman@google.com changed reviewers:
+ mtklein@google.com, reed@google.com

[bungeman-skia, 2015-08-05 14:51:43]

I haven't actually used these yet, but I think these are the right
implementations.

[bungeman-skia, 2015-08-05 16:15:27]

The CQ bit was checked by bungeman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-08-05 16:15:37]

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

[bungeman-skia, 2015-08-05 16:16:27]

Fixes. Take a look.

[commit-bot: I haz the power, 2015-08-05 16:22:38]

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

[commit-bot: I haz the power, 2015-08-05 16:22:39]

Dry run: This issue passed the CQ dry run.

[bungeman-skia, 2015-08-05 17:30:23]

The CQ bit was checked by bungeman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-08-05 17:30:34]

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

[commit-bot: I haz the power, 2015-08-05 17:37:39]

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

[commit-bot: I haz the power, 2015-08-05 17:37:39]

Dry run: This issue passed the CQ dry run.

[bungeman-skia, 2015-08-05 17:43:11]

I think we should land this, since it fixes our one existing 'forward' use.
However, I'm open to name bike-shedding. While these are global functions,
they're also templated functions which are private to Skia.

SkMove (can always infer template arguments)
SkTForward (may never infer template arguments)

sk_std_move
sk_std_forward

SkStdMove
SkStdForward

namespace skstd {
    move
    forward
}

maybe something else?

[mtklein, 2015-08-05 17:48:01]

lgtm, as do all bikesheds.  I've noted one more bikeshed that'd be good enough
for me.

https://codereview.chromium.org/1273813002/diff/40001/include/core/SkTemplates.h
File include/core/SkTemplates.h (right):

https://codereview.chromium.org/1273813002/diff/40001/include/core/SkTemplate...
include/core/SkTemplates.h:31: template <typename T> inline typename
SkTRemoveReference<T>::type&& SkMove(T&& t) {
I'd personally be satisfied with

// SkMove == std::move
// SkForward == std::forward

[mtklein, 2015-08-05 17:48:44]

On 2015/08/05 17:48:01, mtklein wrote:
> lgtm, as do all bikesheds.  I've noted one more bikeshed that'd be good enough
> for me.
> 
>
https://codereview.chromium.org/1273813002/diff/40001/include/core/SkTemplates.h
> File include/core/SkTemplates.h (right):
> 
>
https://codereview.chromium.org/1273813002/diff/40001/include/core/SkTemplate...
> include/core/SkTemplates.h:31: template <typename T> inline typename
> SkTRemoveReference<T>::type&& SkMove(T&& t) {
> I'd personally be satisfied with
> 
> // SkMove == std::move
> // SkForward == std::forward

I'd be even fine with

namespace std {
   ... move ...
   ... forward ...
}

to really get the point across.

[bungeman-skia, 2015-08-05 18:49:43]

The CQ bit was checked by bungeman@google.com to run a CQ dry run

[bungeman-skia, 2015-08-05 18:49:51]

The CQ bit was unchecked by bungeman@google.com

[bungeman-skia, 2015-08-05 18:51:27]

The CQ bit was checked by bungeman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-08-05 18:51:40]

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

[commit-bot: I haz the power, 2015-08-05 18:59:02]

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

[commit-bot: I haz the power, 2015-08-05 18:59:04]

Dry run: This issue passed the CQ dry run.

[bungeman-skia, 2015-08-05 19:09:09]

The CQ bit was checked by bungeman@google.com

[bungeman-skia, 2015-08-05 19:09:09]

The patchset sent to the CQ was uploaded after l-g-t-m from mtklein@google.com
Link to the patchset: https://codereview.chromium.org/1273813002/#ps80001
(title: "Don't need to change animator anymore.")

[commit-bot: I haz the power, 2015-08-05 19:09:22]

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

[commit-bot: I haz the power, 2015-08-05 19:10:00]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/2bd028590d9db472e5b9ee44ca8961be5c036432