Privatize StrongBinding lifetime management

content/browser/battery_status/battery_monitor_integration_browsertest.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 <utility>
 
 #include "base/callback_list.h"
 #include "base/lazy_instance.h"
 #include "base/macros.h"
 #include "base/threading/thread_task_runner_handle.h"
@@ skipping 31 matching lines @@
     LAZY_INSTANCE_INITIALIZER;
 
 // Updates the global battery state and notifies existing test monitors.
 void UpdateBattery(const device::BatteryStatus& battery_status) {
   g_battery_status = battery_status;
   g_callback_list.Get().Notify(battery_status);
 }
 
 class FakeBatteryMonitor : public device::BatteryMonitor {
  public:
+  FakeBatteryMonitor() {}
+  ~FakeBatteryMonitor() override {}
+
   static void Create(mojo::InterfaceRequest<BatteryMonitor> request) {
-    new FakeBatteryMonitor(std::move(request));
+    mojo::MakeStrongBinding(base::MakeUnique<FakeBatteryMonitor>(),
+                            std::move(request));
   }
 
  private:
-  FakeBatteryMonitor(mojo::InterfaceRequest<BatteryMonitor> request)
-      : binding_(this, std::move(request)) {}
-  ~FakeBatteryMonitor() override {}
-
   void QueryNextStatus(const QueryNextStatusCallback& callback) override {
     // We don't expect overlapped calls to QueryNextStatus.
     DCHECK(callback_.is_null());
 
     callback_ = callback;
 
     if (!subscription_) {
       subscription_ =
           g_callback_list.Get().Add(base::Bind(&FakeBatteryMonitor::DidChange,
                                                base::Unretained(this)));
       // Report initial value.
       DidChange(g_battery_status);
     }
   }
 
   void DidChange(const device::BatteryStatus& battery_status) {
     if (!callback_.is_null()) {
       callback_.Run(battery_status.Clone());
       callback_.Reset();
     }
   }
 
   std::unique_ptr<BatteryUpdateSubscription> subscription_;
-  mojo::StrongBinding<BatteryMonitor> binding_;
   QueryNextStatusCallback callback_;
 };
 
 // Overrides the default service implementation with the test implementation
 // declared above.
 class TestContentBrowserClient : public ContentBrowserClient {
  public:
   void ExposeInterfacesToRenderer(
       shell::InterfaceRegistry* registry,
       RenderProcessHost* render_process_host) override {
@@ skipping 87 matching lines @@
   TestNavigationObserver same_tab_observer2(shell()->web_contents(), 1);
   status.level = 0.6;
   UpdateBattery(status);
   same_tab_observer2.Wait();
   EXPECT_EQ("pass", shell()->web_contents()->GetLastCommittedURL().ref());
 }
 
 }  //  namespace
 
 }  //  namespace content