Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(979)

Unified Diff: third_party/WebKit/Source/platform/BUILD.gn

Issue 2867693004: Snapshot of all changes to get jumbo in blink and content.
Patch Set: Rebased again Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/BUILD.gn
diff --git a/third_party/WebKit/Source/platform/BUILD.gn b/third_party/WebKit/Source/platform/BUILD.gn
index 76e8f75e4ee5f6007c4e07de14e61097cf756699..81723ce50f2f17c02916eec617d51bbbccd7b0ea 100644
--- a/third_party/WebKit/Source/platform/BUILD.gn
+++ b/third_party/WebKit/Source/platform/BUILD.gn
@@ -3,6 +3,7 @@
# found in the LICENSE file.
import("//build/config/features.gni")
+import("//build/config/jumbo.gni")
import("//build/config/ui.gni")
import("//testing/libfuzzer/fuzzer_test.gni")
import("//testing/test.gni")
@@ -214,7 +215,8 @@ config("blink_platform_pch") {
}
}
-component("platform") {
+jumbo_component("platform") {
+ target_type = "component"
visibility = [] # Allow re-assignment of list.
visibility = [
"//components/pdf/common:interfaces_blink",
@@ -1555,6 +1557,34 @@ component("platform") {
"INSIDE_BLINK",
]
+ jumbo_excluded_sources = [
+ # Collides with fonts/opentype/FontSetting (AddToHash^2)
+ # Needed??
+ "fonts/FontDescription.cpp",
+
+ # Too much using WTF::Unicode
+ "text/Character.cpp",
+ "text/CharacterEmoji.cpp",
+ "text/TextBoundaries.cpp",
+ "text/UnicodeUtilities.cpp",
+
+ # Too much using icu
+ "text/LocaleICU.cpp",
+
+ # Duplicate code wth JSONParser.cpp
+ "json/JSONValues.cpp",
+
+ # operator<<(...FloatPoint3D)
+ "graphics/filters/PointLightSource.cpp",
+
+ # SizeAsJSONArray
+ "graphics/compositing/PaintArtifactCompositor.cpp",
+ ]
+
+ if (is_mac || is_win) {
+ jumbo_excluded_sources -= [ "text/LocaleICU.cpp" ]
+ }
+
include_dirs = [
#"$angle_path/include",
"$root_gen_dir/blink",
@@ -1707,7 +1737,8 @@ component("platform") {
}
}
-static_library("test_support") {
+jumbo_target("test_support") {
+ target_type = "static_library"
visibility += [ "//third_party/WebKit/*" ]
testonly = true
@@ -1802,7 +1833,8 @@ static_library("test_support") {
}
}
-test("blink_platform_unittests") {
+jumbo_target("blink_platform_unittests") {
+ target_type="test"
visibility = [] # Allow re-assignment of list.
visibility = [ "*" ]
@@ -2009,6 +2041,70 @@ test("blink_platform_unittests") {
sources += [ "testing/RunAllTests.cpp" ]
+ jumbo_excluded_sources = [
+ # Same CreateDecoder as GIFImageDecoderTest.cpp
+ "image-decoders/bmp/BMPImageDecoderTest.cpp",
+ "image-decoders/ico/ICOImageDecoderTest.cpp",
+ "image-decoders/png/PNGImageDecoderTest.cpp",
+ "image-decoders/webp/WEBPImageDecoderTest.cpp",
+ "image-decoders/jpeg/JPEGImageDecoderTest.cpp",
+
+ # Too many TestClass1 and TestClass2
+ "PODFreeListArenaTest.cpp",
+
+ # Too many TestDisplayItem
+ "graphics/paint/PaintControllerTest.cpp",
+
+ # Too many InsertionAndDeletionTest
+ "PODIntervalTreeTest.cpp",
+
+ # Too many ExpectedRun
+ "fonts/OrientationIteratorTest.cpp",
+
+ # Too many IsValid
+ "network/ParsedContentTypeTest.cpp",
+
+ # Too many TestRun and ExpectedRun
+ "fonts/SmallCapsIteratorTest.cpp",
+ "fonts/SymbolsIteratorTest.cpp",
+
+ # Something with base::Bind types.
+ "scheduler/child/idle_helper_unittest.cc",
+ "scheduler/renderer/auto_advancing_virtual_time_domain_unittest.cc",
+ "scheduler/renderer/task_queue_throttler_unittest.cc",
+ "scheduler/renderer/renderer_scheduler_impl_unittest.cc",
+
+ # Too many DECLARE_RUNSVECTOR
+ "fonts/shaping/RunSegmenterTest.cpp",
+
+ # Too many To16Bit
+ "fonts/shaping/ShapingLineBreakerTest.cpp",
+
+ # Too many EXPECT_RECT_EQ
+ "graphics/paint/GeometryMapperTest.cpp",
+
+ # Too many MockTaskObserver
+ "scheduler/child/scheduler_helper_unittest.cc",
+
+ # Too many AppendToVectorTestTask
+ "scheduler/child/worker_scheduler_impl_unittest.cc",
+ "scheduler/child/worker_global_scope_scheduler_unittest.cc",
+
+ # Too many MockObserver
+ "scheduler/renderer/render_widget_signals_unittest.cc",
+
+ # Too many MakeRepeatingTask and RunRepeatingTask
+ "scheduler/renderer/web_view_scheduler_impl_unittest.cc",
+
+ # Too many MockTask
+ "scheduler/renderer/webthread_impl_for_renderer_scheduler_unittest.cc",
+
+ ]
+ if (!is_mac) {
+ # Too many MockScrollableArea
+ jumbo_excluded_sources += [ "scroll/ScrollAnimatorTest.cpp" ]
+ }
+
configs += [
":blink_platform_pch",
"//third_party/WebKit/Source/platform/wtf:wtf_config",
@@ -2107,7 +2203,8 @@ group("blink_platform_unittests_data") {
}
if (current_cpu == "arm") {
- source_set("blink_arm_neon") {
+ jumbo_target("blink_arm_neon") {
+ target_type = "source_set"
sources = blink_platform_neon_files
# The *NEON.cpp files fail to compile when -mthumb is passed. Force
# them to build in ARM mode.
@@ -2123,20 +2220,23 @@ if (current_cpu == "arm") {
}
if (current_cpu == "mipsel" || current_cpu == "mips64el") {
- source_set("blink_mips_msa") {
+ jumbo_target("blink_mips_msa") {
+ target_type = "source_set"
sources = blink_platform_msa_files
}
}
if (current_cpu == "x86" || current_cpu == "x64") {
- source_set("blink_x86_sse") {
+ jumbo_target("blink_x86_sse") {
+ target_type = "source_set"
sources = blink_platform_sse_files
}
}
# This source set is used for fuzzers that need an environment similar to unit
# tests.
-source_set("blink_fuzzer_test_support") {
+jumbo_target("blink_fuzzer_test_support") {
+ target_type = "source_set"
testonly = true
visibility = [] # Allow re-assignment of list.
visibility = [ "*" ]

Powered by Google App Engine
This is Rietveld 408576698