1 files changed, 23 insertions, 19 deletions
diff --git a/BUILD.gn b/BUILD.gn
index fc61dce6a8..fc06c82a74 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -251,6 +251,19 @@ test("pdfium_unittests") {
     "//testing/gtest",
   ]
   configs += [ ":pdfium_core_config" ]
+
+  if (is_android) {
+    use_raw_android_executable = true
+  }
+
+  if (pdf_enable_v8) {
+    configs += [ "//v8:external_startup_data" ]
+    deps += [
+      "fxjs:unittests",
+      "//v8",
+    ]
+  }
+
   if (pdf_enable_xfa) {
     deps += [
       "fxbarcode:unittests",
@@ -263,22 +276,6 @@ test("pdfium_unittests") {
       "xfa/fxfa/parser:unittests",
     ]
   }
-
-  # TODO(thestig): Should this be an embeddertest?
-  if (pdf_use_skia || pdf_use_skia_paths) {
-    sources += [ "core/fxge/skia/fx_skia_device_unittest.cpp" ]
-    deps += [ "//skia" ]
-  }
-  if (pdf_enable_v8) {
-    configs += [ "//v8:external_startup_data" ]
-    deps += [
-      "fxjs:unittests",
-      "//v8",
-    ]
-  }
-  if (is_android) {
-    use_raw_android_executable = true
-  }
 }
 
 test("pdfium_embeddertests") {
@@ -335,6 +332,11 @@ test("pdfium_embeddertests") {
   include_dirs = [ "testing/gmock/include" ]
   configs += [ ":pdfium_core_config" ]
 
+  if (is_android) {
+    ignore_all_data_deps = true
+    use_raw_android_executable = true
+  }
+
   if (pdf_enable_v8) {
     sources += [
       "fxjs/cfxjs_engine_embeddertest.cpp",
@@ -345,6 +347,7 @@ test("pdfium_embeddertests") {
     deps += [ "fxjs" ]
     configs += [ "//v8:external_startup_data" ]
   }
+
   if (pdf_enable_xfa) {
     sources += [
       "fxjs/cfxjse_formcalc_context_embeddertest.cpp",
@@ -354,9 +357,10 @@ test("pdfium_embeddertests") {
       "xfa/fxfa/parser/cxfa_document_parser_embeddertest.cpp",
     ]
   }
-  if (is_android) {
-    ignore_all_data_deps = true
-    use_raw_android_executable = true
+
+  if (pdf_use_skia || pdf_use_skia_paths) {
+    sources += [ "core/fxge/skia/fx_skia_device_embeddertest.cpp" ]
+    deps += [ "//skia" ]
   }
 }