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

Unified Diff: content/browser/BUILD.gn

Issue 2867693004: Snapshot of all changes to get jumbo in blink and content.
Patch Set: Exclude certain files from jumbo because of a Windows problem Created 3 years, 3 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
« no previous file with comments | « cc/tiles/software_image_decode_cache.cc ('k') | content/browser/accessibility/accessibility_ui.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/BUILD.gn
diff --git a/content/browser/BUILD.gn b/content/browser/BUILD.gn
index dd8fc91450e2fccef45df2315933cc8b43138f56..072ba8cc98c6d6228f69b7252bd1c15f15a1c549 100644
--- a/content/browser/BUILD.gn
+++ b/content/browser/BUILD.gn
@@ -11,8 +11,9 @@ import("//ppapi/features/features.gni")
import("//printing/features/features.gni")
import("//third_party/WebKit/public/public_features.gni")
import("//tools/ipc_fuzzer/ipc_fuzzer.gni")
+import("//content/content.gni")
-source_set("browser") {
+content_source_set("browser") {
# Only the public target should depend on this. All other targets (even
# internal content ones) should depend on the public one.
visibility = [
@@ -1650,6 +1651,37 @@ source_set("browser") {
"zygote_host/zygote_host_impl_linux.h",
]
+ jumbo_excluded_sources = [
+ # Shares constant names with dom_storage/local_storage_context_mojo.cpp
+ "dom_storage/dom_storage_context_impl.cc",
+
+ # If someone includes net/url_request/url_request_context_getter.h
+ # in this file, then Bind generation fails when binding
+ # NavigationURLLoaderImplCore::Start.
+ "loader/navigation_url_loader_impl.cc",
+
+ # Too many RecordCancelled.
+ "web_contents/aura/overscroll_navigation_overlay.cc",
+
+ # Too many GetEventAckName.
+ "renderer_host/input/legacy_input_router_impl.cc",
+
+ # Too many kNoDocumentURLErrorMessage, kShutdownErrorMessage,
+ # GetWebContents()
+ "service_worker/service_worker_provider_host.cc",
+ ]
+
+ if (is_win) {
+ jumbo_excluded_sources += [
+ # Mojo header node.h undefs SendMessage so things that include
+ # Windows headers can't be grouped with anything that might
+ # include node.h from mojo. Until we have figured out a way
+ # for mojo headers and Windows headers to live side by side.
+ "renderer_host/legacy_render_widget_host_win.cc",
+ "renderer_host/render_widget_host_view_aura.cc",
+ ]
+ }
+
if (toolkit_views) {
deps += [ "//ui/events" ]
}
@@ -1785,6 +1817,9 @@ source_set("browser") {
# This prevents the inclusion of atlhost.h which paired
# with the windows 8 sdk it does the wrong thing.
"__ATLHOST_H__",
+
+ # Must be defined to get access to M_PI and M_PI_2 in VS.
+ "_USE_MATH_DEFINES",
]
deps += [
"//third_party/iaccessible2",
« no previous file with comments | « cc/tiles/software_image_decode_cache.cc ('k') | content/browser/accessibility/accessibility_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698