Update the Pass→std::move() tool to handle nested calls to Pass().

Update the Pass→std::move() tool to handle nested calls to Pass().

The original implementation just replaced the entire text of x.Pass()
with std::move(x). Unfortunately, this doesn't work well with nested
invocations of Pass(), since `x` will not include the nested rewrite to
use std::move().

Instead, the tool is a bit more clever now: it inserts the text
`std::move(` before the expression and deletes the now-unnecessary text
`.Pass(`. This handles nested expressions well: it still fails for
chained calls to Pass(), but meh.

BUG=557422
Committed: https://crrev.com/47428586c0f9c9a1e1ec1aa8848ae3f35777ee41
Cr-Commit-Position: refs/heads/master@{#365913}

[dcheng, 2015-12-17 21:27:55]

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

[Nico, 2015-12-17 21:31:29]

lgtm

[dcheng, 2015-12-17 21:35:36]

The CQ bit was checked by dcheng@chromium.org

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

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

[commit-bot: I haz the power, 2015-12-17 22:19:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-17 22:20:54]

Description was changed from

==========
Update the Pass→std::move() tool to handle nested calls to Pass().

The original implementation just replaced the entire text of x.Pass()
with std::move(x). Unfortunately, this doesn't work well with nested
invocations of Pass(), since `x` will not include the nested rewrite to
use std::move().

Instead, the tool is a bit more clever now: it inserts the text
`std::move(` before the expression and deletes the now-unnecessary text
`.Pass(`. This handles nested expressions well: it still fails for
chained calls to Pass(), but meh.

BUG=557422
==========

to

==========
Update the Pass→std::move() tool to handle nested calls to Pass().

The original implementation just replaced the entire text of x.Pass()
with std::move(x). Unfortunately, this doesn't work well with nested
invocations of Pass(), since `x` will not include the nested rewrite to
use std::move().

Instead, the tool is a bit more clever now: it inserts the text
`std::move(` before the expression and deletes the now-unnecessary text
`.Pass(`. This handles nested expressions well: it still fails for
chained calls to Pass(), but meh.

BUG=557422

Committed: https://crrev.com/47428586c0f9c9a1e1ec1aa8848ae3f35777ee41
Cr-Commit-Position: refs/heads/master@{#365913}
==========

[commit-bot: I haz the power, 2015-12-17 22:20:54]

Patchset 1 (id:??) landed as
https://crrev.com/47428586c0f9c9a1e1ec1aa8848ae3f35777ee41
Cr-Commit-Position: refs/heads/master@{#365913}