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

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

Issue 1690493002: Switch to LayoutThemeMobile when emulating mobile device in DevTools. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: more mac compile Created 4 years, 10 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/core/BUILD.gn
diff --git a/third_party/WebKit/Source/core/BUILD.gn b/third_party/WebKit/Source/core/BUILD.gn
index 1f20d25a8bd5b48cc08d16ceb82c7513ae9ee1b8..54960b9de350161aa442264ed75904d47e6d3440 100644
--- a/third_party/WebKit/Source/core/BUILD.gn
+++ b/third_party/WebKit/Source/core/BUILD.gn
@@ -238,6 +238,7 @@ source_set("remaining") {
if (is_win) {
cflags += [ "/wd4334" ]
+ sources -= [ "layout/LayoutThemeFontProviderDefault.cpp" ]
pdr. 2016/02/12 06:36:14 I'm a little out of my depth with these build chan
dgozman 2016/02/19 19:15:16 Latest patch makes a 16KB difference on my linux r
} else { # !is_win
sources -= [
"layout/LayoutThemeFontProviderWin.cpp",
@@ -245,14 +246,12 @@ source_set("remaining") {
"layout/LayoutThemeWin.h",
]
}
+
if (!is_linux) {
sources -= [
"layout/LayoutThemeLinux.cpp",
"layout/LayoutThemeLinux.h",
]
- if (!is_android) {
- sources -= [ "layout/LayoutThemeFontProviderLinux.cpp" ]
- }
}
if (!is_android) {
@@ -263,14 +262,6 @@ source_set("remaining") {
}
if (is_mac) {
- sources -= [
- "layout/LayoutThemeDefault.cpp",
- "layout/LayoutThemeDefault.h",
- "layout/LayoutThemeFontProvider.cpp",
- "layout/LayoutThemeFontProvider.h",
- "paint/ThemePainterDefault.cpp",
- "paint/ThemePainterDefault.h",
- ]
libs += [ "Carbon.framework" ]
} else { # !is_mac
sources -= [ "editing/commands/SmartReplaceCF.cpp" ]
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/core.gyp » ('j') | third_party/WebKit/Source/core/layout/LayoutTheme.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698