Make Mandoline shut down cleanly.

components/surfaces/display_impl.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/surfaces/display_impl.h"
 
 #include "cc/output/compositor_frame.h"
 #include "cc/surfaces/display.h"
 #include "components/surfaces/surfaces_context_provider.h"
 #include "components/surfaces/surfaces_output_surface.h"
@@ skipping 41 matching lines @@
   factory_.Create(cc_id_);
   display_->SetSurfaceId(cc_id_, 1.f);
   if (pending_frame_)
     Draw();
 }
 
 DisplayImpl::~DisplayImpl() {
   if (display_) {
     factory_.Destroy(cc_id_);
     scheduler_->RemoveDisplay(display_.get());
+    // By deleting the object after display_ is reset, OutputSurfaceLost can
+    // know not to do anything (which would result in double delete).
+    delete display_.release();
   }
 }
 
 void DisplayImpl::SubmitFrame(mojo::FramePtr frame,
                               const SubmitFrameCallback& callback) {
   DCHECK(pending_callback_.is_null());
   pending_frame_ = frame.Pass();
   pending_callback_ = callback;
   if (display_)
     Draw();
@@ skipping 17 matching lines @@
 }
 
 void DisplayImpl::DidSwapBuffersComplete() {
 }
 
 void DisplayImpl::CommitVSyncParameters(base::TimeTicks timebase,
                                         base::TimeDelta interval) {
 }
 
 void DisplayImpl::OutputSurfaceLost() {
+  if (!display_)  // Shutdown case
+    return;
+
   // If our OutputSurface is lost we can't draw until we get a new one. For now,
   // destroy the display and create a new one when our ContextProvider provides
   // a new one.
   // TODO: This is more violent than necessary - we could simply remove this
   // display from the scheduler's set and pass a new context in to the
   // OutputSurface. It should be able to reinitialize properly.
   scheduler_->RemoveDisplay(display_.get());
   display_.reset();
   factory_.Destroy(cc_id_);
   viewport_param_binding_.Close();
@@ skipping 19 matching lines @@
   DCHECK(returner_);
 
   mojo::Array<mojo::ReturnedResourcePtr> ret(resources.size());
   for (size_t i = 0; i < resources.size(); ++i) {
     ret[i] = mojo::ReturnedResource::From(resources[i]);
   }
   returner_->ReturnResources(ret.Pass());
 }
 
 }  // namespace surfaces