Fix destruction order between SurfaceManger and OffscreenCanvasSurfaceImpl

cc/surfaces/surface_factory.cc

Index: cc/surfaces/surface_factory.cc
diff --git a/cc/surfaces/surface_factory.cc b/cc/surfaces/surface_factory.cc
index d56b3174a8cc41d399468e8def7dc3991b3cdd17..435f30c674dd84a4bb08ede5dd484de822958550 100644
--- a/cc/surfaces/surface_factory.cc
+++ b/cc/surfaces/surface_factory.cc
@@ -32,8 +32,10 @@ SurfaceFactory::~SurfaceFactory() {
 }
 
 void SurfaceFactory::DestroyAll() {
-  for (auto& pair : surface_map_)
-    manager_->Destroy(std::move(pair.second));
+  if (manager_) {
+    for (auto& pair : surface_map_)
+      manager_->Destroy(std::move(pair.second));
+  }
   surface_map_.clear();
 }
 
@@ -50,7 +52,8 @@ void SurfaceFactory::Destroy(SurfaceId surface_id) {
   DCHECK(it->second->factory().get() == this);
   std::unique_ptr<Surface> surface(std::move(it->second));
   surface_map_.erase(it);
-  manager_->Destroy(std::move(surface));
+  if (manager_)
+    manager_->Destroy(std::move(surface));
 }
 
 void SurfaceFactory::SetPreviousFrameSurface(SurfaceId new_id,