Use StrongBinding in NativeViewportImpl instead of manually deleting-on-connection-error.

components/view_manager/native_viewport/native_viewport_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 "components/view_manager/native_viewport/native_viewport_impl.h"
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 11 matching lines @@
     const scoped_refptr<gles2::GpuState>& gpu_state,
     mojo::InterfaceRequest<mojo::NativeViewport> request,
     scoped_ptr<mojo::AppRefCount> app_refcount)
     : is_headless_(is_headless),
       app_refcount_(app_refcount.Pass()),
       context_provider_(new OnscreenContextProvider(gpu_state)),
       sent_metrics_(false),
       metrics_(mojo::ViewportMetrics::New()),
       binding_(this, request.Pass()),
       weak_factory_(this) {
-  binding_.set_error_handler(this);
 }
 
 NativeViewportImpl::~NativeViewportImpl() {
   // Destroy before |platform_viewport_| because this will destroy
   // CommandBufferDriver objects that contain child windows. Otherwise if this
   // class destroys its window first, X errors will occur.
   context_provider_.reset();
 
   // Destroy the NativeViewport early on as it may call us back during
   // destruction and we want to be in a known state.
@@ skipping 110 matching lines @@
   }
 
   event_dispatcher_->OnEvent(event.Pass(), callback);
   return false;
 }
 
 void NativeViewportImpl::OnDestroyed() {
   delete this;
 }
 
-void NativeViewportImpl::OnConnectionError() {
-  binding_.set_error_handler(nullptr);
-  delete this;
-}
-
 void NativeViewportImpl::AckEvent(int32 pointer_id) {
   pointers_waiting_on_ack_.erase(pointer_id);
 }
 
 }  // namespace native_viewport