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

Unified Diff: content/browser/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
« no previous file with comments | « components/tracing/common/tracing_messages.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 217acff233a9cbd6b4442ef975c0c0cde57e1527..37be7e1fbdf037e630f4f97f14886defbf8e9b2d 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 = [
@@ -1587,6 +1588,27 @@ 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",
+ ]
+
+ 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" ]
}
@@ -1715,6 +1737,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 | « components/tracing/common/tracing_messages.cc ('k') | content/browser/accessibility/accessibility_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698