Add missing header files to GN files (4 of N)

third_party/WebKit/Source/platform/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//testing/libfuzzer/fuzzer_test.gni")
 import("//testing/test.gni")
 import("//third_party/WebKit/public/public_features.gni")
 import("//third_party/WebKit/Source/build/scripts/scripts.gni")
@@ skipping 1221 matching lines @@
     "scheduler/base/time_converter.h",
     "scheduler/base/time_domain.cc",
     "scheduler/base/time_domain.h",
     "scheduler/base/virtual_time_domain.cc",
     "scheduler/base/virtual_time_domain.h",
     "scheduler/base/work_queue.cc",
     "scheduler/base/work_queue.h",
     "scheduler/base/work_queue_sets.cc",
     "scheduler/base/work_queue_sets.h",
     "scheduler/child/compositor_worker_scheduler.cc",
+    "scheduler/child/compositor_worker_scheduler.h",
     "scheduler/child/idle_canceled_delayed_task_sweeper.cc",
     "scheduler/child/idle_canceled_delayed_task_sweeper.h",
     "scheduler/child/idle_helper.cc",
     "scheduler/child/idle_helper.h",
     "scheduler/child/scheduler_helper.cc",
     "scheduler/child/scheduler_helper.h",
     "scheduler/child/scheduler_tqm_delegate.h",
     "scheduler/child/scheduler_tqm_delegate_impl.cc",
     "scheduler/child/scheduler_tqm_delegate_impl.h",
     "scheduler/child/single_thread_idle_task_runner.cc",
     "scheduler/child/web_scheduler_impl.cc",
     "scheduler/child/web_scheduler_impl.h",
     "scheduler/child/web_task_runner_impl.cc",
     "scheduler/child/web_task_runner_impl.h",
     "scheduler/child/webthread_base.cc",
     "scheduler/child/webthread_impl_for_worker_scheduler.cc",
+    "scheduler/child/webthread_impl_for_worker_scheduler.h",
     "scheduler/child/worker_global_scope_scheduler.cc",
     "scheduler/child/worker_global_scope_scheduler.h",
     "scheduler/child/worker_scheduler.cc",
+    "scheduler/child/worker_scheduler.h",
     "scheduler/child/worker_scheduler_impl.cc",
     "scheduler/child/worker_scheduler_impl.h",
     "scheduler/renderer/auto_advancing_virtual_time_domain.cc",
     "scheduler/renderer/auto_advancing_virtual_time_domain.h",
     "scheduler/renderer/budget_pool.cc",
     "scheduler/renderer/budget_pool.h",
     "scheduler/renderer/deadline_task_runner.cc",
     "scheduler/renderer/deadline_task_runner.h",
     "scheduler/renderer/idle_time_estimator.cc",
     "scheduler/renderer/idle_time_estimator.h",
@@ skipping 13 matching lines @@
     "scheduler/renderer/throttled_time_domain.h",
     "scheduler/renderer/user_model.cc",
     "scheduler/renderer/user_model.h",
     "scheduler/renderer/web_frame_scheduler_impl.cc",
     "scheduler/renderer/web_frame_scheduler_impl.h",
     "scheduler/renderer/web_view_scheduler_impl.cc",
     "scheduler/renderer/web_view_scheduler_impl.h",
     "scheduler/renderer/webthread_impl_for_renderer_scheduler.cc",
     "scheduler/renderer/webthread_impl_for_renderer_scheduler.h",
     "scheduler/utility/webthread_impl_for_utility_thread.cc",
+    "scheduler/utility/webthread_impl_for_utility_thread.h",
     "scroll/MainThreadScrollingReason.h",
     "scroll/ProgrammaticScrollAnimator.cpp",
     "scroll/ProgrammaticScrollAnimator.h",
     "scroll/ScrollAnimator.cpp",
     "scroll/ScrollAnimator.h",
     "scroll/ScrollAnimatorBase.cpp",
     "scroll/ScrollAnimatorBase.h",
     "scroll/ScrollAnimatorCompositorCoordinator.cpp",
     "scroll/ScrollAnimatorCompositorCoordinator.h",
     "scroll/ScrollStateData.h",
@@ skipping 128 matching lines @@
     "weborigin/SecurityOrigin.h",
     "weborigin/SecurityOriginHash.h",
     "weborigin/SecurityPolicy.cpp",
     "weborigin/SecurityPolicy.h",
     "weborigin/SecurityViolationReportingPolicy.h",
     "weborigin/Suborigin.cpp",
     "weborigin/Suborigin.h",
     "weborigin/URLSecurityOriginMap.h",
   ]
 
-  if (is_android) {
-    set_sources_assignment_filter([])
-    sources += [ "exported/linux/WebFontRenderStyle.cpp" ]
-  }
-
   sources -= blink_platform_neon_files
   sources -= blink_platform_msa_files
   sources -= blink_platform_sse_files
 
   # Add in the generated files.
   sources += get_target_outputs(":character_data") +
              get_target_outputs(":color_data") +
              get_target_outputs(":font_family_names") +
              get_target_outputs(":http_names") +
              get_target_outputs(":instrumentation_probes") +
@@ skipping 101 matching lines @@
       "/wd4724",  # Modulo by 0.
     ]
   } else {
     sources -= [ "text/LocaleWin.cpp" ]
   }
 
   if (is_android) {
     # Add in some Linux files also shared with Android.
     set_sources_assignment_filter([])
     sources += [
+      "../../public/platform/linux/WebFontRenderStyle.h",

[Nico, 2017/04/27 14:18:09]

It's weird to me that source/platform would include a header from public/ --
shouldn't this be in WebKit/public/BUILD.gn ? ...and indeed, there it is:
https://cs.chromium.org/chromium/src/third_party/WebKit/public/BUILD.gn?type=...

Why is this needed? If it's needed I feel it should be in a separate CL since
it's kind of weird, while the rest of this CL is pretty clear.

[wychen, 2017/04/27 23:06:29]

Moved to WebKit/public. Thanks for catching this!

+      "exported/linux/WebFontRenderStyle.cpp",
       "fonts/linux/FontPlatformDataLinux.cpp",
       "fonts/linux/FontRenderStyle.cpp",
       "fonts/linux/FontRenderStyle.h",
     ]
     set_sources_assignment_filter(sources_assignment_filter)
   } else {
     sources -= [ "scroll/ScrollbarThemeAndroid.cpp" ]
   }
 
   if (use_minikin_hyphenation) {
@@ skipping 591 matching lines @@
     "//third_party/WebKit/Source:config",
     "//third_party/WebKit/Source:inside_blink",
   ]
 
   deps = [
     ":test_support",
     "//testing/gmock",
     "//testing/gtest",
   ]
 }