content: Fix Context creation logic in ContextProviderFactoryImpl.

content/browser/renderer_host/compositor_impl_android.cc

 // Copyright (c) 2012 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 "content/browser/renderer_host/compositor_impl_android.h"
 
 #include <android/bitmap.h>
 #include <android/native_window_jni.h>
 #include <stdint.h>
 #include <unordered_set>
@@ skipping 375 matching lines @@
                                          ->AllocateSurfaceClientId())),
       resource_manager_(root_window),
       has_transparent_background_(false),
       device_scale_factor_(1),
       window_(NULL),
       surface_handle_(gpu::kNullSurfaceHandle),
       client_(client),
       root_window_(root_window),
       needs_animate_(false),
       pending_swapbuffers_(0U),
+      num_successive_gpu_process_initialization_failures_(0),
       num_successive_context_creation_failures_(0),
       output_surface_request_pending_(false),
       needs_begin_frames_(false),
       weak_factory_(this) {
   ui::ContextProviderFactory::GetInstance()
       ->GetSurfaceManager()
       ->RegisterSurfaceClientId(surface_id_allocator_->client_id());
   DCHECK(client);
   DCHECK(root_window);
   DCHECK(root_window->GetLayer() == nullptr);
@@ skipping 235 matching lines @@
 
   if (!display_output_surface)
     return;
 
   InitializeDisplay(std::move(display_output_surface),
                     std::move(vulkan_context_provider), nullptr);
 }
 #endif
 
 void CompositorImpl::CreateCompositorOutputSurface(
-    const scoped_refptr<cc::ContextProvider>& context_provider) {
-  // This callback should run only if we have a pending output surface request,
-  // since that is when we should have queued a context request.
-  // In case the surface was invalidated after the context request was queued,
-  // the request should have been dropped by the ContextProviderFactory.
+    const scoped_refptr<cc::ContextProvider>& context_provider,
+    ui::ContextProviderFactory::ContextCreationResult result) {
   DCHECK(output_surface_request_pending_);
-  DCHECK(host_->visible());
-  DCHECK(window_);
-  DCHECK_NE(surface_handle_, gpu::kNullSurfaceHandle);
-  DCHECK(context_provider);
 
-  scoped_refptr<ContextProviderCommandBuffer> context_provider_command_buffer =
-      static_cast<ContextProviderCommandBuffer*>(context_provider.get());
-  std::unique_ptr<cc::OutputSurface> display_output_surface(
-      new OutputSurfaceWithoutParent(
-          context_provider_command_buffer,
-          base::Bind(&CompositorImpl::PopulateGpuCapabilities,
-                     base::Unretained(this))));
-  InitializeDisplay(std::move(display_output_surface), nullptr,
-                    std::move(context_provider));
+  switch (result) {
+    // Don't retry if we are shutting down or if the Gpu Surface handle was
+    // lost. The Gpu Surface handle loss should happen only if we are invisible
+    // or this was from a previous request and the current surface has changed,
+    // in which case we would have made another request with the factory.
+    case ui::ContextProviderFactory::ContextCreationResult::
+        FAILURE_BROWSER_SHUTDOWN:
+    case ui::ContextProviderFactory::ContextCreationResult::
+        FAILURE_GPU_SURFACE_HANDLE_LOST:
+      break;
+    case ui::ContextProviderFactory::ContextCreationResult::
+        FAILURE_GPU_PROCESS_INITIALIZATION_FAILED:
+      // Retry only if we are visible.
+      if (host_->visible()) {
+        LOG_IF(FATAL,
+               ++num_successive_gpu_process_initialization_failures_ >= 2)
+            << "Too many GPU Process initialization failures. Giving up... ";
+        HandlePendingOutputSurfaceRequest();
+      }
+      break;
+    case ui::ContextProviderFactory::ContextCreationResult::SUCCESS:
+      DCHECK(host_->visible());
+      DCHECK(window_);
+      DCHECK_NE(surface_handle_, gpu::kNullSurfaceHandle);
+      DCHECK(context_provider);
+
+      num_successive_gpu_process_initialization_failures_ = 0;
+      scoped_refptr<ContextProviderCommandBuffer>
+          context_provider_command_buffer =
+              static_cast<ContextProviderCommandBuffer*>(
+                  context_provider.get());
+      std::unique_ptr<cc::OutputSurface> display_output_surface(
+          new OutputSurfaceWithoutParent(
+              context_provider_command_buffer,
+              base::Bind(&CompositorImpl::PopulateGpuCapabilities,
+                         base::Unretained(this))));
+      InitializeDisplay(std::move(display_output_surface), nullptr,
+                        std::move(context_provider));
+      break;
+  }
 }
 
 void CompositorImpl::InitializeDisplay(
     std::unique_ptr<cc::OutputSurface> display_output_surface,
     scoped_refptr<cc::VulkanContextProvider> vulkan_context_provider,
     scoped_refptr<cc::ContextProvider> context_provider) {
   DCHECK(output_surface_request_pending_);
 
   pending_swapbuffers_ = 0;
 
@@ skipping 112 matching lines @@
 
   TRACE_EVENT0("compositor", "Compositor::SetNeedsAnimate");
   host_->SetNeedsAnimate();
 }
 
 bool CompositorImpl::HavePendingReadbacks() {
   return !readback_layer_tree_->children().empty();
 }
 
 }  // namespace content