0

[ios] Enable use of ui/color for use_blink

In crrev.com/c/5051135 ios-specific guards were added to address
failures on ios-simulator, but these do not apply to use_blink.

Also some refactoring to switch -= for += and to merge use_blink
sections.

Bug: 1502915
Change-Id: Ie375aefc73a5ebe981670980f78ea60376f09a99
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5075190
Commit-Queue: Ian Vollick <vollick@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1231946}
This commit is contained in:
Ian Vollick
2023-12-01 15:55:15 +00:00
committed by Chromium LUCI CQ
parent aef7aef480
commit f35eb2fd87
3 changed files with 17 additions and 21 deletions
ui

@ -514,8 +514,6 @@ component("base") {
"//ui/base/dragdrop/mojom",
"//ui/display",
"//ui/display/util:gpu_info_util",
"//ui/events",
"//ui/events/devices",
"//ui/resources:webui_resources_grd",
"//ui/strings",
"//url",
@ -544,15 +542,15 @@ component("base") {
":l10n_util_ios",
"//base",
]
if (!use_blink) {
deps -= [
"//ui/events",
"//ui/events/devices",
]
}
} else { # !is_ios
}
if (use_blink) {
deps += [
"//ui/color",
"//ui/events",
"//ui/events/devices",
]
public_deps += [ "//ui/color:color_headers" ]
deps += [ "//ui/color" ]
}
if (is_win) {
@ -1065,8 +1063,6 @@ test("ui_base_unittests") {
"//ui/base/metadata:metadata_tests",
"//ui/base/prediction:prediction",
"//ui/display",
"//ui/events:events_base",
"//ui/events:test_support",
"//ui/gfx:test_support",
"//ui/resources",
"//ui/resources:ui_test_pak",
@ -1108,12 +1104,6 @@ test("ui_base_unittests") {
"test/ios/ui_image_test_utils_unittest.mm",
]
deps += [ ":l10n_util_ios" ]
if (!use_blink) {
deps -= [
"//ui/events:events_base",
"//ui/events:test_support",
]
}
} else { # !is_ios
sources += [
"accelerators/accelerator_manager_unittest.cc",
@ -1129,8 +1119,14 @@ test("ui_base_unittests") {
"themed_vector_icon_unittest.cc",
"webui/web_ui_util_unittest.cc",
]
}
deps += [ "//ui/color" ]
if (use_blink) {
deps += [
"//ui/color",
"//ui/events:events_base",
"//ui/events:test_support",
]
}
if (is_linux || is_chromeos) {

@ -8,7 +8,7 @@ import("//mojo/public/tools/bindings/mojom.gni")
import("//testing/test.gni")
import("//ui/base/ui_features.gni")
assert(!is_ios, "Not supported on iOS")
assert(use_blink, "Not supported without use_blink")
source_set("color_headers") {
sources = [

@ -17,7 +17,7 @@ test("ui_unittests") {
deps += [ "//ui/actions:unit_tests" ]
}
if (!is_ios) {
if (use_blink) {
deps += [ "//ui/color:unit_tests" ]
}
}