From 4a7f4e3fab99845b7efb34dd9fb7a8cfe4c57350 Mon Sep 17 00:00:00 2001 From: Kevin Lubick Date: Tue, 26 Sep 2023 23:32:53 +0000 Subject: [PATCH] Update to use GrDirectContexts::MakeGL --- lib/web_ui/skwasm/surface.cpp | 3 ++- shell/common/shell_io_manager.cc | 3 ++- shell/gpu/gpu_surface_gl_skia.cc | 3 ++- testing/test_gl_surface.cc | 3 ++- 4 files changed, 8 insertions(+), 4 deletions(-) diff --git a/lib/web_ui/skwasm/surface.cpp b/lib/web_ui/skwasm/surface.cpp index f1601bbef976c..cebb4889faf2b 100644 --- a/lib/web_ui/skwasm/surface.cpp +++ b/lib/web_ui/skwasm/surface.cpp @@ -8,6 +8,7 @@ #include "third_party/skia/include/gpu/GrBackendSurface.h" #include "third_party/skia/include/gpu/GrDirectContext.h" #include "third_party/skia/include/gpu/ganesh/gl/GrGLBackendSurface.h" +#include "third_party/skia/include/gpu/ganesh/gl/GrGLDirectContext.h" using namespace Skwasm; @@ -91,7 +92,7 @@ void Surface::_init() { makeCurrent(_glContext); emscripten_webgl_enable_extension(_glContext, "WEBGL_debug_renderer_info"); - _grContext = GrDirectContext::MakeGL(GrGLMakeNativeInterface()); + _grContext = GrDirectContexts::MakeGL(GrGLMakeNativeInterface()); // WebGL should already be clearing the color and stencil buffers, but do it // again here to ensure Skia receives them in the expected state. diff --git a/shell/common/shell_io_manager.cc b/shell/common/shell_io_manager.cc index 4a51c2f8d96d9..ec22661b3763d 100644 --- a/shell/common/shell_io_manager.cc +++ b/shell/common/shell_io_manager.cc @@ -8,6 +8,7 @@ #include "flutter/fml/message_loop.h" #include "flutter/shell/common/context_options.h" +#include "third_party/skia/include/gpu/ganesh/gl/GrGLDirectContext.h" #include "third_party/skia/include/gpu/gl/GrGLInterface.h" namespace flutter { @@ -22,7 +23,7 @@ sk_sp ShellIOManager::CreateCompatibleResourceLoadingContext( const auto options = MakeDefaultContextOptions(ContextType::kResource); - if (auto context = GrDirectContext::MakeGL(gl_interface, options)) { + if (auto context = GrDirectContexts::MakeGL(gl_interface, options)) { // Do not cache textures created by the image decoder. These textures // should be deleted when they are no longer referenced by an SkImage. context->setResourceCacheLimit(0); diff --git a/shell/gpu/gpu_surface_gl_skia.cc b/shell/gpu/gpu_surface_gl_skia.cc index 3118bee005134..04c708808e0a9 100644 --- a/shell/gpu/gpu_surface_gl_skia.cc +++ b/shell/gpu/gpu_surface_gl_skia.cc @@ -20,6 +20,7 @@ #include "third_party/skia/include/gpu/GrContextOptions.h" #include "third_party/skia/include/gpu/ganesh/SkSurfaceGanesh.h" #include "third_party/skia/include/gpu/ganesh/gl/GrGLBackendSurface.h" +#include "third_party/skia/include/gpu/ganesh/gl/GrGLDirectContext.h" #include "third_party/skia/include/gpu/gl/GrGLTypes.h" // These are common defines present on all OpenGL headers. However, we don't @@ -51,7 +52,7 @@ sk_sp GPUSurfaceGLSkia::MakeGLContext( const auto options = MakeDefaultContextOptions(ContextType::kRender, GrBackendApi::kOpenGL); - auto context = GrDirectContext::MakeGL(delegate->GetGLInterface(), options); + auto context = GrDirectContexts::MakeGL(delegate->GetGLInterface(), options); if (!context) { FML_LOG(ERROR) << "Failed to set up Skia Gr context."; diff --git a/testing/test_gl_surface.cc b/testing/test_gl_surface.cc index 8c98bacde9fdc..c70364696277a 100644 --- a/testing/test_gl_surface.cc +++ b/testing/test_gl_surface.cc @@ -20,6 +20,7 @@ #include "third_party/skia/include/gpu/GrBackendSurface.h" #include "third_party/skia/include/gpu/ganesh/SkSurfaceGanesh.h" #include "third_party/skia/include/gpu/ganesh/gl/GrGLBackendSurface.h" +#include "third_party/skia/include/gpu/ganesh/gl/GrGLDirectContext.h" #include "third_party/skia/include/gpu/gl/GrGLAssembleInterface.h" #include "third_party/skia/include/gpu/gl/GrGLTypes.h" @@ -336,7 +337,7 @@ sk_sp TestGLSurface::CreateGrContext() { return nullptr; } - context_ = GrDirectContext::MakeGL(interface); + context_ = GrDirectContexts::MakeGL(interface); return context_; }