From c161b2d2f9c702fb8dec490da82c86ea67d7ea30 Mon Sep 17 00:00:00 2001
From: Nico Weber <thakis@chromium.org>
Date: Fri, 20 Oct 2023 08:17:14 -0400
Subject: [PATCH] LibPDF: Extract ICCBasedColorSpace::sRGB() helper

---
 Userland/Libraries/LibPDF/ColorSpace.cpp | 12 ++++++++----
 Userland/Libraries/LibPDF/ColorSpace.h   |  2 ++
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/Userland/Libraries/LibPDF/ColorSpace.cpp b/Userland/Libraries/LibPDF/ColorSpace.cpp
index b5a14ede35..74d4a91d5e 100644
--- a/Userland/Libraries/LibPDF/ColorSpace.cpp
+++ b/Userland/Libraries/LibPDF/ColorSpace.cpp
@@ -499,9 +499,6 @@ ICCBasedColorSpace::ICCBasedColorSpace(NonnullRefPtr<Gfx::ICC::Profile> profile)
 
 PDFErrorOr<ColorOrStyle> ICCBasedColorSpace::style(ReadonlySpan<Value> arguments) const
 {
-    if (!s_srgb_profile)
-        s_srgb_profile = TRY(Gfx::ICC::sRGB());
-
     Vector<u8> bytes;
     for (size_t i = 0; i < arguments.size(); ++i) {
         auto const& arg = arguments[i];
@@ -522,7 +519,7 @@ PDFErrorOr<ColorOrStyle> ICCBasedColorSpace::style(ReadonlySpan<Value> arguments
 
     auto pcs = TRY(m_profile->to_pcs(bytes));
     Array<u8, 3> output;
-    TRY(s_srgb_profile->from_pcs(m_profile, pcs, output.span()));
+    TRY(sRGB()->from_pcs(m_profile, pcs, output.span()));
 
     return Color(output[0], output[1], output[2]);
 }
@@ -553,6 +550,13 @@ Vector<float> ICCBasedColorSpace::default_decode() const
     }
 }
 
+NonnullRefPtr<Gfx::ICC::Profile> ICCBasedColorSpace::sRGB()
+{
+    if (!s_srgb_profile)
+        s_srgb_profile = MUST(Gfx::ICC::sRGB());
+    return *s_srgb_profile;
+}
+
 PDFErrorOr<NonnullRefPtr<LabColorSpace>> LabColorSpace::create(Document* document, Vector<Value>&& parameters)
 {
     if (parameters.size() != 1)
diff --git a/Userland/Libraries/LibPDF/ColorSpace.h b/Userland/Libraries/LibPDF/ColorSpace.h
index 81296e40b3..70beac13a8 100644
--- a/Userland/Libraries/LibPDF/ColorSpace.h
+++ b/Userland/Libraries/LibPDF/ColorSpace.h
@@ -188,6 +188,8 @@ public:
     Vector<float> default_decode() const override;
     ColorSpaceFamily const& family() const override { return ColorSpaceFamily::ICCBased; }
 
+    static NonnullRefPtr<Gfx::ICC::Profile> sRGB();
+
 private:
     ICCBasedColorSpace(NonnullRefPtr<Gfx::ICC::Profile>);