From 5615a2691adbcee9d992135b2f7931b6daf3abdb Mon Sep 17 00:00:00 2001
From: Nico Weber <thakis@chromium.org>
Date: Tue, 16 Jan 2024 20:38:13 -0500
Subject: [PATCH] LibPDF: Extract activate_clip() / deactivate_clip() functions

No behavior change.
---
 Userland/Libraries/LibPDF/Renderer.cpp | 24 +++++++++++++++++-------
 Userland/Libraries/LibPDF/Renderer.h   |  3 +++
 2 files changed, 20 insertions(+), 7 deletions(-)

diff --git a/Userland/Libraries/LibPDF/Renderer.cpp b/Userland/Libraries/LibPDF/Renderer.cpp
index 04cd6f9a70..59ccd4a92e 100644
--- a/Userland/Libraries/LibPDF/Renderer.cpp
+++ b/Userland/Libraries/LibPDF/Renderer.cpp
@@ -290,11 +290,7 @@ RENDERER_HANDLER(path_append_rect)
     return {};
 }
 
-///
-// Path painting operations
-///
-
-void Renderer::begin_path_paint()
+void Renderer::activate_clip()
 {
     auto bounding_box = state().clipping_paths.current.bounding_box();
     m_painter.clear_clip_rect();
@@ -304,11 +300,25 @@ void Renderer::begin_path_paint()
     m_painter.add_clip_rect(bounding_box.to_type<int>());
 }
 
+void Renderer::deactivate_clip()
+{
+    m_painter.clear_clip_rect();
+    state().clipping_paths.current = state().clipping_paths.next;
+}
+
+///
+// Path painting operations
+///
+
+void Renderer::begin_path_paint()
+{
+    activate_clip();
+}
+
 void Renderer::end_path_paint()
 {
     m_current_path.clear();
-    m_painter.clear_clip_rect();
-    state().clipping_paths.current = state().clipping_paths.next;
+    deactivate_clip();
 }
 
 RENDERER_HANDLER(path_stroke)
diff --git a/Userland/Libraries/LibPDF/Renderer.h b/Userland/Libraries/LibPDF/Renderer.h
index 177679dd4e..32a3467d29 100644
--- a/Userland/Libraries/LibPDF/Renderer.h
+++ b/Userland/Libraries/LibPDF/Renderer.h
@@ -127,6 +127,9 @@ private:
     PDFErrorOr<void> handle_text_next_line_show_string(ReadonlySpan<Value> args, Optional<NonnullRefPtr<DictObject>> = {});
     PDFErrorOr<void> handle_text_next_line_show_string_set_spacing(ReadonlySpan<Value> args, Optional<NonnullRefPtr<DictObject>> = {});
 
+    void activate_clip();
+    void deactivate_clip();
+
     void begin_path_paint();
     void end_path_paint();
     PDFErrorOr<void> set_graphics_state_from_dict(NonnullRefPtr<DictObject>);