Do not include :unittests unless enable_unittests (flutter#173729)

matanlurey · web-flow · commit 446adb1acc5f · 2025-08-13T20:19:16.000Z
Closes flutter#173728. This will need to be cherrypicked into both 3.35 and 3.36.
diff --git a/engine/src/flutter/BUILD.gn b/engine/src/flutter/BUILD.gn
@@ -78,10 +78,12 @@ group("flutter") {
 
   if (!is_qnx) {
     public_deps = [
-      ":unittests",
       "//flutter/shell/platform/embedder:flutter_engine",
       "//flutter/sky",
     ]
+    if (enable_unittests) {
+      public_deps += [ ":unittests" ]
+    }
   }
 
   # Ensure the example for a sample embedder compiles.

Original file line number	Diff line number	Diff line change
`@@ -78,10 +78,12 @@ group("flutter") {`
`78`	`78`
`79`	`79`	`if (!is_qnx) {`
`80`	`80`	`public_deps = [`
`81`		`- ":unittests",`
`82`	`81`	`"//flutter/shell/platform/embedder:flutter_engine",`
`83`	`82`	`"//flutter/sky",`
`84`	`83`	`]`
	`84`	`+ if (enable_unittests) {`
	`85`	`+ public_deps += [ ":unittests" ]`
	`86`	`+ }`
`85`	`87`	`}`
`86`	`88`
`87`	`89`	`# Ensure the example for a sample embedder compiles.`