Loading: move core/fetch to platform/loader/fetch

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 1441 matching lines @@
   ]
   deps = [
     ":blink_common",
     "//cc/surfaces:surfaces",
     "//components/link_header_util:link_header_util",
     "//components/mime_util:mime_util",
     "//device/base/synchronization",
     "//mojo/public/cpp/bindings",
     "//mojo/public/cpp/bindings:wtf_support",
     "//third_party/WebKit/Source/platform/heap",
+    "//third_party/WebKit/Source/platform/loader",

[kinuko, 2017/01/16 05:25:09]

Would you be able to first just move all the files without making structural
changes in BUILD.gn (i.e. just add all files in this file) to make this CL
dedicated for simple file moving, and then create a separate CL only for
BUILD.gn changes? 

[Takashi Toyoshima, 2017/01/17 13:51:48]

Thanks. Since now brettw@ says this is straightforward change, let me keep this
as is.

brettw@
Thank you for replying. My concerns (you asked in chat) were
 - Is it reasonable to have a split BUILD.gn file under platform
 - Do I pick up necessary and sufficient import gnis in loader/BUILD.gn
correctly

     "//third_party/WebKit/public:mojo_bindings_blink",
     "//third_party/WebKit/public:offscreen_canvas_mojo_bindings_blink",
     "//third_party/ced",
     "//third_party/harfbuzz-ng",
     "//third_party/icu",
     "//ui/gfx",
     "//ui/gfx/geometry",
   ]
 
   if (is_mac) {
@@ skipping 107 matching lines @@
     configs -= [ "//build/config/compiler:default_symbols" ]
     configs += [ "//build/config/compiler:no_symbols" ]
   }
 }
 
 static_library("test_support") {
   visibility += [ "//third_party/WebKit/*" ]
   testonly = true
 
   sources = [
+    "loader/fetch/MockResource.cpp",
+    "loader/fetch/MockResource.h",
+    "loader/fetch/MockResourceClient.cpp",
+    "loader/fetch/MockResourceClient.h",
     "network/mime/MockMimeRegistry.h",
     "scheduler/base/task_queue_manager_delegate_for_test.cc",
     "scheduler/base/test_time_source.cc",
     "scheduler/child/scheduler_tqm_delegate_for_test.cc",
     "scheduler/test/fake_renderer_scheduler.cc",
     "scheduler/test/fake_web_task_runner.cc",
     "scheduler/test/fake_web_task_runner.h",
     "scheduler/test/lazy_scheduler_message_loop_delegate_for_tests.cc",
     "scheduler/test/lazy_scheduler_message_loop_delegate_for_tests.h",
     "scheduler/test/renderer_scheduler_test_support.cc",
@@ skipping 159 matching lines @@
     "image-decoders/gif/GIFImageDecoderTest.cpp",
     "image-decoders/ico/ICOImageDecoderTest.cpp",
     "image-decoders/jpeg/JPEGImageDecoderTest.cpp",
     "image-decoders/png/PNGImageDecoderTest.cpp",
     "image-decoders/webp/WEBPImageDecoderTest.cpp",
     "image-encoders/JPEGImageEncoderTest.cpp",
     "instrumentation/tracing/TracedValueTest.cpp",
     "instrumentation/tracing/web_process_memory_dump_test.cc",
     "json/JSONParserTest.cpp",
     "json/JSONValuesTest.cpp",
+    "loader/fetch/ClientHintsPreferencesTest.cpp",
+    "loader/fetch/CrossOriginAccessControlTest.cpp",
+    "loader/fetch/FetchUtilsTest.cpp",
+    "loader/fetch/MemoryCacheCorrectnessTest.cpp",
+    "loader/fetch/MemoryCacheCorrectnessTestHelper.cpp",
+    "loader/fetch/MemoryCacheCorrectnessTestHelper.h",
+    "loader/fetch/MemoryCacheTest.cpp",
+    "loader/fetch/MockFetchContext.h",
+    "loader/fetch/RawResourceTest.cpp",
+    "loader/fetch/ResourceFetcherTest.cpp",
+    "loader/fetch/ResourceLoaderOptionsTest.cpp",
+    "loader/fetch/ResourceTest.cpp",
     "mac/VersionUtilMacTest.mm",
     "mojo/CommonCustomTypesStructTraitsTest.cpp",
     "mojo/GeometryStructTraitsTest.cpp",
     "mojo/KURLSecurityOriginTest.cpp",
     "network/EncodedFormDataTest.cpp",
     "network/HTTPParsersTest.cpp",
     "network/LinkHeaderTest.cpp",
     "network/NetworkUtilsTest.cpp",
     "network/ResourceRequestTest.cpp",
     "network/ResourceResponseTest.cpp",
@@ skipping 316 matching lines @@
     "//third_party/WebKit/Source:config",
     "//third_party/WebKit/Source:inside_blink",
   ]
 
   deps = [
     ":test_support",
     "//testing/gmock",
     "//testing/gtest",
   ]
 }