base::ListValue::Append cleanup: pass unique_ptr instead of the released pointer.

extensions/common/event_filter_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "extensions/common/event_filter.h"
 
 #include <memory>
 #include <string>
 #include <utility>
 
@@ skipping 21 matching lines @@
  protected:
   std::unique_ptr<base::Value> HostSuffixDict(const std::string& host_suffix) {
     std::unique_ptr<base::DictionaryValue> dict(new DictionaryValue());
     dict->Set("hostSuffix", new base::StringValue(host_suffix));
     return std::unique_ptr<base::Value>(dict.release());
   }
 
   std::unique_ptr<base::ListValue> ValueAsList(
       std::unique_ptr<base::Value> value) {
     std::unique_ptr<base::ListValue> result(new base::ListValue());
-    result->Append(value.release());
+    result->Append(std::move(value));
     return result;
   }
 
   std::unique_ptr<EventMatcher> AllURLs() {
     return std::unique_ptr<EventMatcher>(
         new EventMatcher(std::unique_ptr<DictionaryValue>(new DictionaryValue),
                          MSG_ROUTING_NONE));
   }
 
   std::unique_ptr<EventMatcher> HostSuffixMatcher(
@@ skipping 79 matching lines @@
   int id = event_filter_.AddEventMatcher("event1",
                                          HostSuffixMatcher("google.com"));
   std::set<int> matches = event_filter_.MatchEvent(
       "event1", info, MSG_ROUTING_NONE);
   ASSERT_EQ(1u, matches.size());
   ASSERT_EQ(1u, matches.count(id));
 }
 
 TEST_F(EventFilterUnittest, TestMultipleURLFiltersMatchOnAny) {
   std::unique_ptr<base::ListValue> filters(new base::ListValue());
-  filters->Append(HostSuffixDict("google.com").release());
-  filters->Append(HostSuffixDict("yahoo.com").release());
+  filters->Append(HostSuffixDict("google.com"));
+  filters->Append(HostSuffixDict("yahoo.com"));
 
   std::unique_ptr<EventMatcher> matcher(
       MatcherFromURLFilterList(std::move(filters)));
   int id = event_filter_.AddEventMatcher("event1", std::move(matcher));
 
   {
     std::set<int> matches = event_filter_.MatchEvent("event1",
         google_event_, MSG_ROUTING_NONE);
     ASSERT_EQ(1u, matches.size());
     ASSERT_EQ(1u, matches.count(id));
@@ skipping 102 matching lines @@
   std::unique_ptr<EventMatcher> matcher(MatcherFromURLFilterList(
       ValueAsList(std::unique_ptr<Value>(new DictionaryValue()))));
   int id = event_filter_.AddEventMatcher("event1", std::move(matcher));
   std::set<int> matches = event_filter_.MatchEvent(
       "event1", empty_url_event_, MSG_ROUTING_NONE);
   ASSERT_EQ(1u, matches.size());
   ASSERT_EQ(1u, matches.count(id));
 }
 
 }  // namespace extensions