Convert Pass()→std::move() in //chrome

Convert Pass()→std::move() in //chrome

☃

BUG=557422
R=avi@chromium.org
TBR=thestig@chromium.org

Committed: https://crrev.com/e73d8520cc74376a5ca1fdfeb6a60eef0f0ddd7b
Cr-Commit-Position: refs/heads/master@{#366918}

[dcheng, 2015-12-26 23:38:03]

+avi for general review / sanity check
+thestig for OWNERS review

[dcheng, 2015-12-26 23:38:12]

The CQ bit was checked by dcheng@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-26 23:38:19]

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

[Avi (use Gerrit), 2015-12-27 00:14:14]

lgtm

stampity stamp

[commit-bot: I haz the power, 2015-12-27 00:42:36]

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

[commit-bot: I haz the power, 2015-12-27 00:42:38]

Dry run: This issue passed the CQ dry run.

[dcheng, 2015-12-27 01:06:13]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2015-12-27 01:06:25]

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

[commit-bot: I haz the power, 2015-12-27 01:19:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-27 01:20:37]

Description was changed from

==========
Convert Pass()→std::move() in //chrome

☃

BUG=557422
R=avi@chromium.org
TBR=thestig@chromium.org
==========

to

==========
Convert Pass()→std::move() in //chrome

☃

BUG=557422
R=avi@chromium.org
TBR=thestig@chromium.org

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

[commit-bot: I haz the power, 2015-12-27 01:20:40]

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

[Lei Zhang, 2015-12-27 22:35:06]

lgtm

https://codereview.chromium.org/1551503002/diff/1/chrome/browser/sessions/tab...
File chrome/browser/sessions/tab_restore_service_factory.cc (right):

https://codereview.chromium.org/1551503002/diff/1/chrome/browser/sessions/tab...
chrome/browser/sessions/tab_restore_service_factory.cc:65: return new
sessions::InMemoryTabRestoreService(client.Pass(), nullptr);
Did you miss one here?

[dcheng, 2015-12-27 23:09:36]

https://codereview.chromium.org/1551503002/diff/1/chrome/browser/sessions/tab...
File chrome/browser/sessions/tab_restore_service_factory.cc (right):

https://codereview.chromium.org/1551503002/diff/1/chrome/browser/sessions/tab...
chrome/browser/sessions/tab_restore_service_factory.cc:65: return new
sessions::InMemoryTabRestoreService(client.Pass(), nullptr);
On 2015/12/27 at 22:35:06, Lei Zhang wrote:
> Did you miss one here?

This will be updated when I make a pass over the Android build.