Privatize StrongBinding lifetime management

components/dom_distiller/content/browser/distillability_driver.cc

 // Copyright 2015 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/dom_distiller/content/browser/distillability_driver.h"
 
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_contents_user_data.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
 #include "services/shell/public/cpp/interface_registry.h"
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(
     dom_distiller::DistillabilityDriver);
 
 namespace dom_distiller {
 
 // Implementation of the Mojo DistillabilityService. This is called by the
 // renderer to notify the browser that a page is distillable.
 class DistillabilityServiceImpl : public mojom::DistillabilityService {
  public:
-  DistillabilityServiceImpl(
-      mojo::InterfaceRequest<mojom::DistillabilityService> request,
+  explicit DistillabilityServiceImpl(
       base::WeakPtr<DistillabilityDriver> distillability_driver)
-      : binding_(this, std::move(request)),
-        distillability_driver_(distillability_driver) {}
+      : distillability_driver_(distillability_driver) {}
 
   ~DistillabilityServiceImpl() override {
     if (!distillability_driver_) return;
     distillability_driver_->SetNeedsMojoSetup();
   }
 
   void NotifyIsDistillable(bool is_distillable, bool is_last_update) override {
     if (!distillability_driver_) return;
     distillability_driver_->OnDistillability(is_distillable, is_last_update);
   }
 
  private:
-  mojo::StrongBinding<mojom::DistillabilityService> binding_;
   base::WeakPtr<DistillabilityDriver> distillability_driver_;
 };
 
 DistillabilityDriver::DistillabilityDriver(
     content::WebContents* web_contents)
     : content::WebContentsObserver(web_contents),
       mojo_needs_setup_(true),
       weak_factory_(this) {
   if (!web_contents) return;
   SetupMojoService(web_contents->GetMainFrame());
 }
 
 DistillabilityDriver::~DistillabilityDriver() {
   content::WebContentsObserver::Observe(nullptr);
 }
 
 void DistillabilityDriver::CreateDistillabilityService(
     mojo::InterfaceRequest<mojom::DistillabilityService> request) {
-  new DistillabilityServiceImpl(std::move(request), weak_factory_.GetWeakPtr());
+  mojo::MakeStrongBinding(
+      base::MakeUnique<DistillabilityServiceImpl>(weak_factory_.GetWeakPtr()),
+      std::move(request));
 }
 
 void DistillabilityDriver::SetDelegate(
     const base::Callback<void(bool, bool)>& delegate) {
   m_delegate_ = delegate;
 }
 
 void DistillabilityDriver::OnDistillability(
     bool distillable, bool is_last) {
   if (m_delegate_.is_null()) return;
@@ skipping 31 matching lines @@
     return;
   }
 
   frame_host->GetInterfaceRegistry()->AddInterface(
       base::Bind(&DistillabilityDriver::CreateDistillabilityService,
           weak_factory_.GetWeakPtr()));
   mojo_needs_setup_ = false;
 }
 
 }  // namespace dom_distiller