Fix compile error on simple chrome.

Fix compile error on simple chrome.

Under C++11, NVRO can happen when a return's expression
has the function's return type.
Here, those are different (although FakeArcBridgeBootstrap
inherits ArcBridgeBootstrap), thus explicit std::move()
should be here.

Note, C++17 or later, the condition is relaxed and NVRO
becomes mandatory, so the old code should work, though.

cf): C++11 n3337/12.8/32, C++17 n4594/12.8/32

BUG=633258
TEST=Ran bots. Build component_unittests under simple chrome env.

Committed: https://crrev.com/778baa7c83b7c45eee055f53ac445cdeba5cfbdd
Cr-Commit-Position: refs/heads/master@{#423436}

[hidehiko, 2016-10-06 02:17:27]

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

[commit-bot: I haz the power, 2016-10-06 02:17:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hidehiko, 2016-10-06 02:18:03]

hidehiko@chromium.org changed reviewers:
+ kinaba@chromium.org

[hidehiko, 2016-10-06 02:18:04]

PTAL.

[kinaba, 2016-10-06 02:19:02]

lgtm

[hidehiko, 2016-10-06 02:19:13]

Description was changed from

==========
Fix compile error on simple chrome.

Under C++11, NVRO can happen when a return's expression
has the function's return type.
Here, those are different (although FakeArcBridgeBootstrap
inherits ArcBridgeBootstrap), thus explicit std::move()
should be here.

Note, C++17 or later, the condition is relaxed and NVRO
becomes mandatory, so the old code should work, though.

BUG=633258
TEST=Ran bots. Build component_unittests under simple chrome env.
==========

to

==========
Fix compile error on simple chrome.

Under C++11, NVRO can happen when a return's expression
has the function's return type.
Here, those are different (although FakeArcBridgeBootstrap
inherits ArcBridgeBootstrap), thus explicit std::move()
should be here.

Note, C++17 or later, the condition is relaxed and NVRO
becomes mandatory, so the old code should work, though.

cf): C++11 n3337/12.8/32, C++17 n4594/12.8/32

BUG=633258
TEST=Ran bots. Build component_unittests under simple chrome env.
==========

[commit-bot: I haz the power, 2016-10-06 02:36:32]

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

[commit-bot: I haz the power, 2016-10-06 02:36:33]

Dry run: This issue passed the CQ dry run.

[hidehiko, 2016-10-06 05:03:40]

The CQ bit was checked by hidehiko@chromium.org

[commit-bot: I haz the power, 2016-10-06 05:04:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-06 05:09:37]

Description was changed from

==========
Fix compile error on simple chrome.

Under C++11, NVRO can happen when a return's expression
has the function's return type.
Here, those are different (although FakeArcBridgeBootstrap
inherits ArcBridgeBootstrap), thus explicit std::move()
should be here.

Note, C++17 or later, the condition is relaxed and NVRO
becomes mandatory, so the old code should work, though.

cf): C++11 n3337/12.8/32, C++17 n4594/12.8/32

BUG=633258
TEST=Ran bots. Build component_unittests under simple chrome env.
==========

to

==========
Fix compile error on simple chrome.

Under C++11, NVRO can happen when a return's expression
has the function's return type.
Here, those are different (although FakeArcBridgeBootstrap
inherits ArcBridgeBootstrap), thus explicit std::move()
should be here.

Note, C++17 or later, the condition is relaxed and NVRO
becomes mandatory, so the old code should work, though.

cf): C++11 n3337/12.8/32, C++17 n4594/12.8/32

BUG=633258
TEST=Ran bots. Build component_unittests under simple chrome env.
==========

[commit-bot: I haz the power, 2016-10-06 05:09:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-06 05:11:34]

Description was changed from

==========
Fix compile error on simple chrome.

Under C++11, NVRO can happen when a return's expression
has the function's return type.
Here, those are different (although FakeArcBridgeBootstrap
inherits ArcBridgeBootstrap), thus explicit std::move()
should be here.

Note, C++17 or later, the condition is relaxed and NVRO
becomes mandatory, so the old code should work, though.

cf): C++11 n3337/12.8/32, C++17 n4594/12.8/32

BUG=633258
TEST=Ran bots. Build component_unittests under simple chrome env.
==========

to

==========
Fix compile error on simple chrome.

Under C++11, NVRO can happen when a return's expression
has the function's return type.
Here, those are different (although FakeArcBridgeBootstrap
inherits ArcBridgeBootstrap), thus explicit std::move()
should be here.

Note, C++17 or later, the condition is relaxed and NVRO
becomes mandatory, so the old code should work, though.

cf): C++11 n3337/12.8/32, C++17 n4594/12.8/32

BUG=633258
TEST=Ran bots. Build component_unittests under simple chrome env.

Committed: https://crrev.com/778baa7c83b7c45eee055f53ac445cdeba5cfbdd
Cr-Commit-Position: refs/heads/master@{#423436}
==========

[commit-bot: I haz the power, 2016-10-06 05:11:35]

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