extensions: Clean up ENABLE_EXTENSIONS in //extensions
TestRuntimeApiDelegate can be safely added to the build now.
Also, replace ENABLE_EXTENSIONS || ENABLE_DESKTOP_ANDROID_EXTENSIONS
with ENABLE_EXTENSIONS_CORE, to be consistent with the rest of the
codebase.
Bug: 356905053
Change-Id: I616391c2843c6c5e20188803f2a51c6f83165927
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6576511
Reviewed-by: Devlin Cronin <[email protected]>
Commit-Queue: James Cook <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1464202}
diff --git a/extensions/BUILD.gn b/extensions/BUILD.gn
index 0ea291f1..586fd74 100644
--- a/extensions/BUILD.gn
+++ b/extensions/BUILD.gn
@@ -111,6 +111,8 @@
"browser/test_image_loader.h",
"browser/test_management_policy.cc",
"browser/test_management_policy.h",
+ "browser/test_runtime_api_delegate.cc",
+ "browser/test_runtime_api_delegate.h",
"common/extension_builder.cc",
"common/extension_builder.h",
"common/manifest_test.cc",
@@ -198,16 +200,6 @@
deps += [ "//components/guest_view/browser:test_support" ]
}
- # TODO(https://crbug.com/356905053): The following files don't compile cleanly
- # with desktop-android. Either make them compile, or determine they should
- # not be included and place them under a more appropriate if-block.
- if (enable_extensions) {
- sources += [
- "browser/test_runtime_api_delegate.cc",
- "browser/test_runtime_api_delegate.h",
- ]
- }
-
# Generally, //extensions should not depend on //chromeos. However, a number
# of the APIs and the extensions shell already do. We should try to avoid
# expanding these dependencies.
