De-client TestRequestTracker

services/test_service/test_request_tracker_impl.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 "base/bind.h"
 #include "services/test_service/test_request_tracker_impl.h"
 
 namespace mojo {
 namespace test {
 
- TrackingContext::TrackingContext() : next_id(1) {}
- TrackingContext::~TrackingContext() {}
+TrackingContext::TrackingContext() : next_id(1) {
+}
+TrackingContext::~TrackingContext() {

[viettrungluu, 2015/02/04 00:27:34]

nit: blank line above

+}
 
- TestRequestTrackerImpl::TestRequestTrackerImpl(TrackingContext* context)
-     : context_(context), weak_factory_(this) {
- }
+TestRequestTrackerImpl::TestRequestTrackerImpl(
+    InterfaceRequest<TestRequestTracker> request,
+    TrackingContext* context)
+    : context_(context), binding_(this, request.Pass()), weak_factory_(this) {
+}
 
- TestRequestTrackerImpl::~TestRequestTrackerImpl() {
- }
+TestRequestTrackerImpl::~TestRequestTrackerImpl() {
+}
 
 void TestRequestTrackerImpl::RecordStats(
       uint64_t client_id,
       ServiceStatsPtr stats) {
   assert(context_->ids_to_names.find(client_id) !=
          context_->ids_to_names.end());
   context_->records[client_id].push_back(*stats);
 }
 
-void TestRequestTrackerImpl::OnConnectionEstablished() {
+void TestRequestTrackerImpl::SetNameAndReturnId(
+    const String& service_name,
+    const Callback<void(uint64_t id)>& callback) {
   uint64_t id = context_->next_id++;
-  client()->SetIdAndReturnName(id,
-      base::Bind(&TestRequestTrackerImpl::UploaderNameCallback,
-                 weak_factory_.GetWeakPtr(),
-                 id));
-}
-
-void TestRequestTrackerImpl::UploaderNameCallback(
-    uint64_t id, const mojo::String& name) {
+  callback.Run(id);
   DCHECK(context_->ids_to_names.find(id) == context_->ids_to_names.end());
-  context_->ids_to_names[id] = name;
+  context_->ids_to_names[id] = service_name;
 }
 
 TestTrackedRequestServiceImpl::TestTrackedRequestServiceImpl(
     TrackingContext* context)
     : context_(context) {
 }
 
 TestTrackedRequestServiceImpl::~TestTrackedRequestServiceImpl() {
 }
 
@@ skipping 15 matching lines @@
       mean_health_numerator += it2->health;
     }
     report->mean_health = mean_health_numerator / num_samples;
     reports.push_back(report.Pass());
   }
   callback.Run(reports.Pass());
 }
 
 }  // namespace test
 }  // namespace mojo