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

Side by Side Diff: third_party/harfbuzz-ng/BUILD.gn

Issue 1627533002: [Chromecast] Depend on Chromium's freetype-android (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Add clarification of using freetype-android 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 unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/chrome_build.gni") 5 import("//build/config/chrome_build.gni")
6 import("//build/config/linux/pkg_config.gni") 6 import("//build/config/linux/pkg_config.gni")
7 import("//build/config/ui.gni") 7 import("//build/config/ui.gni")
8 8
9 # The GYP build supports system harfbuzz for non-official builds when using 9 # The GYP build supports system harfbuzz for non-official builds when using
10 # pangoft2 1.31.0 or greater (which pulls it in). 10 # pangoft2 1.31.0 or greater (which pulls it in).
(...skipping 183 matching lines...) Expand 10 before | Expand all | Expand 10 after
194 ] 194 ]
195 defines += [ "HAVE_CORETEXT" ] 195 defines += [ "HAVE_CORETEXT" ]
196 } 196 }
197 197
198 # When without -fvisibility=hidden for pango to use the harfbuzz 198 # When without -fvisibility=hidden for pango to use the harfbuzz
199 # in the tree, all symbols pango needs must be included, or 199 # in the tree, all symbols pango needs must be included, or
200 # pango uses mixed versions of harfbuzz and leads to crash. 200 # pango uses mixed versions of harfbuzz and leads to crash.
201 # See crbug.com/462689. 201 # See crbug.com/462689.
202 if (is_linux && use_pango && !is_chromeos && !is_official_build && 202 if (is_linux && use_pango && !is_chromeos && !is_official_build &&
203 current_cpu != "arm" && current_cpu != "mipsel") { 203 current_cpu != "arm" && current_cpu != "mipsel") {
204 deps += [ "//build/linux:freetype2" ]
204 configs -= [ "//build/config/gcc:symbol_visibility_hidden" ] 205 configs -= [ "//build/config/gcc:symbol_visibility_hidden" ]
205 configs += [ 206 configs += [ "//build/config/linux:glib" ]
206 "//build/config/linux:freetype2",
207 "//build/config/linux:glib",
208 ]
209 sources += [ 207 sources += [
210 "src/hb-ft.cc", 208 "src/hb-ft.cc",
211 "src/hb-ft.h", 209 "src/hb-ft.h",
212 "src/hb-glib.cc", 210 "src/hb-glib.cc",
213 "src/hb-glib.h", 211 "src/hb-glib.h",
214 ] 212 ]
215 } 213 }
216 } 214 }
217 } 215 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698