Set global SkFontConfigInterface in the gpu process.

Why: It's a combination of 2 things:
* ChromeOS is beginning to use Skottie for some of its animations.
  The animation may contain text embedded in it, in addition to
  specifying the font for said text.
* ChromeOS uses OOP (out-of-process) raster, meaning all raster
  operations happen in the GPU process. More specifically in this
  case, a skottie animation object is instantiated in the GPU
  process and we call skottie::Animation::render() in it. As such,
  the GPU process must somehow have access to the font cache, or
  Skottie fails to load the appropriate font typeface when rendering.

This follows the exact same pattern as what's done in the render
process (see renderer_blink_platform_impl.cc) and utility process by
connecting to the font service that's hosted in the browser process.

Bug: b:217271404

Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome
Change-Id: I4d6dfd1d96f8e412a21c71c0391401848efd4da3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3464510
Reviewed-by: Zhenyao Mo <[email protected]>
Reviewed-by: Daniel Cheng <[email protected]>
Commit-Queue: Eric Sum <[email protected]>
Cr-Commit-Position: refs/heads/main@{#972745}

Author: esum26

content/browser/gpu/gpu_process_host_receiver_bindings.cc

@@ -7,6 +7,7 @@
 #include "content/browser/gpu/gpu_process_host.h"
 
 #include "build/build_config.h"
+#include "build/chromeos_buildflags.h"
 #include "content/public/browser/browser_task_traits.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/content_browser_client.h"
@@ -19,6 +20,11 @@
 #include "media/mojo/mojom/android_overlay.mojom.h"
 #endif
 
+#if BUILDFLAG(IS_CHROMEOS_ASH)
+#include "components/services/font/public/mojom/font_service.mojom.h"  // nogncheck
+#include "content/browser/font_service.h"  // nogncheck
+#endif
+
 namespace content {
 
 namespace {
@@ -43,6 +49,14 @@ void GpuProcessHost::BindHostReceiver(
   }
 #endif
 
+#if BUILDFLAG(IS_CHROMEOS_ASH)
+  if (auto font_receiver =
+          generic_receiver.As<font_service::mojom::FontService>()) {
+    ConnectToFontService(std::move(font_receiver));
+    return;
+  }
+#endif
+
   GetContentClient()->browser()->BindGpuHostReceiver(
       std::move(generic_receiver));
 }

content/gpu/BUILD.gn

@@ -99,6 +99,13 @@ target(link_target_type, "gpu_sources") {
     ]
   }
 
+  if (is_chromeos_ash) {
+    deps += [
+      "//components/services/font/public/cpp",
+      "//components/services/font/public/mojom",
+    ]
+  }
+
   if (is_android) {
     deps += [
       "//components/tracing:graphics_provider",

content/gpu/gpu_child_thread.cc

@@ -21,6 +21,7 @@
 #include "base/task/sequenced_task_runner.h"
 #include "base/threading/thread_checker.h"
 #include "build/build_config.h"
+#include "build/chromeos_buildflags.h"
 #include "content/child/child_process.h"
 #include "content/common/process_visibility_tracker.h"
 #include "content/gpu/browser_exposed_gpu_interfaces.h"
@@ -50,6 +51,13 @@
 #include "media/mojo/clients/mojo_android_overlay.h"
 #endif
 
+#if BUILDFLAG(IS_CHROMEOS_ASH)
+#include "components/services/font/public/cpp/font_loader.h"  // nogncheck
+#include "components/services/font/public/mojom/font_service.mojom.h"  // nogncheck
+#include "third_party/skia/include/core/SkRefCnt.h"
+#include "third_party/skia/include/ports/SkFontConfigInterface.h"
+#endif
+
 namespace content {
 namespace {
 
@@ -140,6 +148,15 @@ void GpuChildThread::Init(const base::Time& process_start_time) {
   }
 #endif
 
+#if BUILDFLAG(IS_CHROMEOS_ASH)
+  if (!in_process_gpu()) {
+    mojo::PendingRemote<font_service::mojom::FontService> font_service;
+    BindHostReceiver(font_service.InitWithNewPipeAndPassReceiver());
+    SkFontConfigInterface::SetGlobal(
+        sk_make_sp<font_service::FontLoader>(std::move(font_service)));
+  }
+#endif
+
   memory_pressure_listener_ = std::make_unique<base::MemoryPressureListener>(
       FROM_HERE, base::BindRepeating(&GpuChildThread::OnMemoryPressure,
                                      base::Unretained(this)));