Switch //components away from base::ListValue::Append(Value*) overload.

components/domain_reliability/scheduler.cc

 // Copyright 2014 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 "components/domain_reliability/scheduler.h"
 
 #include <stdint.h>
 #include <algorithm>
 #include <utility>
 
@@ skipping 171 matching lines @@
     last->SetBoolean("success", last_upload_success_);
     data->Set("last_upload", std::move(last));
   }
 
   std::unique_ptr<base::ListValue> collectors_value(new base::ListValue());
   for (const auto* collector : collectors_) {
     std::unique_ptr<base::DictionaryValue> value(new base::DictionaryValue());
     value->SetInteger("failures", collector->failure_count());
     value->SetInteger("next_upload",
         (collector->GetReleaseTime() - now).InSeconds());
-    // Using release instead of Pass because Pass can't implicitly upcast.

[danakj, 2016/08/26 22:59:24]

:D

     collectors_value->Append(std::move(value));
   }
   data->Set("collectors", std::move(collectors_value));
 
   return std::move(data);
 }
 
 void DomainReliabilityScheduler::MakeDeterministicForTesting() {
   backoff_policy_.jitter_factor = 0.0;
 }
@@ skipping 58 matching lines @@
       min_index = i;
     }
   }
 
   DCHECK_NE(kInvalidCollectorIndex, min_index);
   *upload_time_out = min_time;
   *collector_index_out = min_index;
 }
 
 }  // namespace domain_reliability