commit 57ff2734df74aaf331627fd51a8e79966c0d211c
author Danil Chapovalov <danilchap@webrtc.org> Wed Mar 28 11:25:15 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Wed Mar 28 11:58:06 2018 +0000
tree 2347a965d316ed3eb1d6841411879e4b459bd64e
parent 467057ec7fe791e3cd8b9046fd5ba5ba701b3e5f

Remove rtc::Optional::MoveValue

This function is not present in std::optional
The only use of MoveValue doesn't need move since
copying underneath struct is as correct and as fast as moving

Bug: webrtc:9078
Change-Id: Ic0c87e50ffd8f6c024759b14ceeb8922b5d3a6fd
Reviewed-on: https://webrtc-review.googlesource.com/64986
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22646}

api/optional_unittest.cc

diff --git a/api/optional_unittest.cc b/api/optional_unittest.cc
index 2149033..ad700dc 100644
--- a/api/optional_unittest.cc
+++ b/api/optional_unittest.cc
@@ -861,7 +861,7 @@
   {
     Optional<Logger> x(Logger(42));
     log->push_back("---");
-    Logger moved = x.MoveValue();
+    Logger moved = std::move(x.value());
     log->push_back("---");
   }
   EXPECT_EQ(