Stop using some scoped_ptr features that unique_ptr doesn't have

webrtc/base/optional_unittest.cc

Index: webrtc/base/optional_unittest.cc
diff --git a/webrtc/base/optional_unittest.cc b/webrtc/base/optional_unittest.cc
index eabf091e17a024924d05c11c1a47ddb5733a10e9..9b9339e6f21d4cdda4baa8e97a017d7d648c9364 100644
--- a/webrtc/base/optional_unittest.cc
+++ b/webrtc/base/optional_unittest.cc
@@ -65,7 +65,7 @@ class Logger {
   void Foo() { Log("Foo()"); }
   void Foo() const { Log("Foo() const"); }
   static rtc::scoped_ptr<std::vector<std::string>> Setup() {
-    auto s = rtc_make_scoped_ptr(new std::vector<std::string>);
+    rtc::scoped_ptr<std::vector<std::string>> s(new std::vector<std::string>);
     Logger::log_ = s.get();

[tommi, 2016/03/17 12:23:26]

Do you mind removing the "Logger::" prefix where it's not necessary in this
file?
Would also be good to put down a TODO to rename the log_ and next_id_ variables
to g_log and g_next_id.

     Logger::next_id_ = 0;
     return s;