Update tgfx to the latest version. (#2742)

Co-authored-by: domrjchen <dom@idom.me>

Author: Hparty

CMakeLists.txt

@@ -21,7 +21,8 @@ set(CMAKE_CXX_VISIBILITY_PRESET hidden)
 
 # Options for building libpag
 option(PAG_USE_OPENGL "Allow use of OpenGL as GPU backend" ON)
-option(PAG_USE_SWIFTSHADER "Allow build with SwiftShader library" OFF)
+option(PAG_USE_SWIFTSHADER "Allow build with the SwiftShader library" OFF)
+option(PAG_USE_ANGLE "Allow build with the ANGLE library" OFF)
 option(PAG_USE_QT "Allow build with QT frameworks" OFF)
 option(PAG_USE_RTTR "Enable RTTR support" OFF)
 option(PAG_USE_HARFBUZZ "Enable HarfBuzz support" OFF)
@@ -76,12 +77,13 @@ endif ()
 
 if (PAG_USE_QT)
     set(PAG_USE_SWIFTSHADER OFF)
+    set(PAG_USE_ANGLE OFF)
+elseif (PAG_USE_SWIFTSHADER)
+    set(PAG_USE_ANGLE OFF)
 endif ()
 
-if (PAG_USE_QT OR PAG_USE_SWIFTSHADER)
+if (PAG_USE_QT OR PAG_USE_SWIFTSHADER OR PAG_USE_ANGLE)
     set(PAG_USE_OPENGL ON)
-else ()
-    set(USE_NATIVE_PLATFORM ON)
 endif ()
 
 if (PAG_USE_FFAVC)
@@ -201,6 +203,8 @@ elseif (PAG_USE_SWIFTSHADER)
     list(APPEND PAG_INCLUDES ${TGFX_DIR}/vendor/swiftshader/include)
     file(GLOB_RECURSE PLATFORM_FILES src/platform/swiftshader/*.*)
     list(APPEND PAG_FILES ${PLATFORM_FILES})
+else ()
+    set(USE_NATIVE_PLATFORM ON)
 endif ()
 
 if (PAG_USE_RTTR)
@@ -350,9 +354,16 @@ elseif (WIN32)
     list(APPEND PAG_DEFINES NOMINMAX _USE_MATH_DEFINES)
 
     if (USE_NATIVE_PLATFORM)
-        list(APPEND PAG_INCLUDES ${TGFX_DIR}/vendor/angle/include)
-        file(GLOB ANGLE_LIBS ${TGFX_DIR}/vendor/angle/${PLATFORM}/${ARCH}/*${CMAKE_STATIC_LIBRARY_SUFFIX})
-        list(APPEND PAG_STATIC_LIBS ${ANGLE_LIBS})
+        if (PAG_USE_ANGLE)
+            list(APPEND PAG_DEFINES PAG_USE_ANGLE)
+            list(APPEND PAG_INCLUDES ${TGFX_DIR}/vendor/angle/include)
+            file(GLOB ANGLE_LIBS ${TGFX_DIR}/vendor/angle/${PLATFORM}/${ARCH}/*${CMAKE_STATIC_LIBRARY_SUFFIX})
+            list(APPEND PAG_STATIC_LIBS ${ANGLE_LIBS})
+        else ()
+            find_library(OPENGL_LIB opengl32)
+            list(APPEND PAG_STATIC_LIBS ${OPENGL_LIB})
+        endif ()
+
         file(GLOB_RECURSE PLATFORM_FILES src/platform/win/*.*)
         list(APPEND PAG_FILES ${PLATFORM_FILES})
     endif ()
@@ -431,6 +442,7 @@ if (NOT HAS_CUSTOM_TGFX_DIR AND EXISTS ${TGFX_CACHE_DIR})
     list(APPEND TGFX_OPTIONS "-DTGFX_USE_OPENGL=${PAG_USE_OPENGL}")
     list(APPEND TGFX_OPTIONS "-DTGFX_USE_QT=${PAG_USE_QT}")
     list(APPEND TGFX_OPTIONS "-DTGFX_USE_SWIFTSHADER=${PAG_USE_SWIFTSHADER}")
+    list(APPEND TGFX_OPTIONS "-DTGFX_USE_ANGLE=${PAG_USE_ANGLE}")
     list(APPEND TGFX_OPTIONS "-DTGFX_USE_FREETYPE=${PAG_USE_FREETYPE}")
     list(APPEND TGFX_OPTIONS "-DTGFX_USE_PNG_DECODE=${PAG_USE_PNG_DECODE}")
     list(APPEND TGFX_OPTIONS "-DTGFX_USE_PNG_ENCODE=${PAG_USE_PNG_ENCODE}")
@@ -462,6 +474,7 @@ else ()
     set(TGFX_USE_OPENGL ${PAG_USE_OPENGL})
     set(TGFX_USE_QT ${PAG_USE_QT})
     set(TGFX_USE_SWIFTSHADER ${PAG_USE_SWIFTSHADER})
+    set(TGFX_USE_ANGLE ${PAG_USE_ANGLE})
     set(TGFX_USE_FREETYPE ${PAG_USE_FREETYPE})
     set(TGFX_USE_PNG_DECODE ${PAG_USE_PNG_DECODE})
     set(TGFX_USE_PNG_ENCODE ${PAG_USE_PNG_ENCODE})

DEPS

@@ -12,7 +12,7 @@
       },
       {
         "url": "${PAG_GROUP}/tgfx.git",
-        "commit": "235ce5e99ddb33cb8679702ea6b17a7098e10023",
+        "commit": "2d4b07c7388b5a5825fbbc032a30bb7a05e6452b",
         "dir": "third_party/tgfx"
       },
       {

src/platform/win/GPUDrawable.cpp

@@ -19,7 +19,11 @@
 #include "GPUDrawable.h"
 #include <windows.h>
 #include "base/utils/Log.h"
+#ifdef PAG_USE_ANGLE
 #include "tgfx/gpu/opengl/egl/EGLWindow.h"
+#else
+#include "tgfx/gpu/opengl/wgl/WGLWindow.h"
+#endif
 
 namespace pag {
 std::shared_ptr<GPUDrawable> GPUDrawable::FromWindow(void* nativeWindow, void* sharedContext) {
@@ -56,8 +60,13 @@ std::shared_ptr<tgfx::Device> GPUDrawable::getDevice() {
     return nullptr;
   }
   if (window == nullptr) {
+#ifdef PAG_USE_ANGLE
     window = tgfx::EGLWindow::MakeFrom(reinterpret_cast<EGLNativeWindowType>(nativeWindow),
                                        sharedContext);
+#else
+    window = tgfx::WGLWindow::MakeFrom(reinterpret_cast<HWND>(nativeWindow),
+                                       reinterpret_cast<HGLRC>(sharedContext));
+#endif
   }
   return window ? window->getDevice() : nullptr;
 }
@@ -75,4 +84,4 @@ void GPUDrawable::onFreeSurface() {
 void GPUDrawable::present(tgfx::Context* context) {
   window->present(context);
 }
-}  // namespace pag
+}  // namespace pag

src/platform/win/GPUDrawable.h

@@ -22,7 +22,8 @@
 
 namespace tgfx {
 class EGLWindow;
-}
+class WGLWindow;
+}  // namespace tgfx
 
 namespace pag {
 class GPUDrawable : public Drawable {
@@ -51,10 +52,14 @@ class GPUDrawable : public Drawable {
  private:
   int _width = 0;
   int _height = 0;
+#ifdef PAG_USE_ANGLE
   std::shared_ptr<tgfx::EGLWindow> window = nullptr;
+#else
+  std::shared_ptr<tgfx::WGLWindow> window = nullptr;
+#endif
   void* nativeWindow = nullptr;
   void* sharedContext = nullptr;
 
   GPUDrawable(void* nativeWindow, void* sharedContext);
 };
-}  // namespace pag
+}  // namespace pag

src/rendering/PAGAnimator.cpp

@@ -275,7 +275,7 @@ std::vector<int> PAGAnimator::doAdvance() {
 
 void PAGAnimator::doUpdate(bool setStartTime) {
   locker.lock();
-  if (task != nullptr && task->executing()) {
+  if (task != nullptr && task->status() == tgfx::TaskStatus::Executing) {
     locker.unlock();
     return;
   }

src/rendering/graphics/Canvas.h

@@ -105,10 +105,6 @@ class Canvas {
     canvas->clear();
   }
 
-  void clearRect(const tgfx::Rect& rect, const tgfx::Color& color) {
-    canvas->clearRect(rect, color);
-  }
-
   void drawLine(float x0, float y0, float x1, float y1, const tgfx::Paint& paint) {
     canvas->drawLine(x0, y0, x1, y1, createPaint(paint));
   }

src/rendering/renderers/FilterRenderer.cpp

@@ -152,8 +152,12 @@ void FilterRenderer::MeasureFilterBounds(tgfx::Rect* bounds, const FilterModifie
   }
 }
 
-tgfx::Rect GetClipBounds(Canvas* canvas, const FilterList* filterList) {
+tgfx::Rect GetClipBounds(Canvas* canvas, const FilterList* filterList,
+                         const tgfx::Rect& contentBounds) {
   auto clip = canvas->getTotalClip();
+  if (clip.isInverseFillType()) {
+    return contentBounds;
+  }
   auto matrix = canvas->getMatrix();
   if (filterList->useParentSizeInput) {
     tgfx::Matrix inverted = tgfx::Matrix::I();
@@ -359,7 +363,7 @@ void FilterRenderer::DrawWithFilter(Canvas* parentCanvas, const FilterModifier*
   auto filterList = MakeFilterList(modifier);
   auto contentBounds = GetContentBounds(filterList.get(), content);
   // 相对于content Bounds的clip Bounds
-  auto clipBounds = GetClipBounds(parentCanvas, filterList.get());
+  auto clipBounds = GetClipBounds(parentCanvas, filterList.get(), contentBounds);
 
   auto contentScale = GetContentScale(parentCanvas, filterList->scaleFactorLimit,
                                       GetScaleFactor(filterList.get(), contentBounds));