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

Side by Side Diff: BUILD.gn

Issue 1301473003: Suppress -Wswitch for pdfium's fxge target for now. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 5 years, 4 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
« no previous file with comments | « no previous file | pdfium.gyp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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("//testing/test.gni") 5 import("//testing/test.gni")
6 6
7 pdf_use_skia = false 7 pdf_use_skia = false
8 8
9 declare_args() { 9 declare_args() {
10 # On Android there's no system FreeType. On Windows and Mac, only a few 10 # On Android there's no system FreeType. On Windows and Mac, only a few
(...skipping 487 matching lines...) Expand 10 before | Expand all | Expand 10 after
498 "core/src/fxge/ge/fx_ge_device.cpp", 498 "core/src/fxge/ge/fx_ge_device.cpp",
499 "core/src/fxge/ge/fx_ge_font.cpp", 499 "core/src/fxge/ge/fx_ge_font.cpp",
500 "core/src/fxge/ge/fx_ge_fontmap.cpp", 500 "core/src/fxge/ge/fx_ge_fontmap.cpp",
501 "core/src/fxge/ge/fx_ge_linux.cpp", 501 "core/src/fxge/ge/fx_ge_linux.cpp",
502 "core/src/fxge/ge/fx_ge_path.cpp", 502 "core/src/fxge/ge/fx_ge_path.cpp",
503 "core/src/fxge/ge/fx_ge_ps.cpp", 503 "core/src/fxge/ge/fx_ge_ps.cpp",
504 "core/src/fxge/ge/fx_ge_text.cpp", 504 "core/src/fxge/ge/fx_ge_text.cpp",
505 "core/src/fxge/ge/text_int.h", 505 "core/src/fxge/ge/text_int.h",
506 ] 506 ]
507 507
508 config("fxge_warnings") {
509 if (is_clang) {
510 cflags = [
511 # http://code.google.com/p/pdfium/issues/detail?id=188
512 "-Wno-switch",
513 ]
514 }
515 }
508 configs -= [ "//build/config/compiler:chromium_code" ] 516 configs -= [ "//build/config/compiler:chromium_code" ]
509 configs += [ 517 configs += [
510 ":pdfium_config", 518 ":pdfium_config",
511 "//build/config/compiler:no_chromium_code", 519 "//build/config/compiler:no_chromium_code",
520 ":fxge_warnings",
512 ] 521 ]
513 522
514 if (pdf_use_skia) { 523 if (pdf_use_skia) {
515 sources = [ 524 sources = [
516 "core/src/fxge/skia/fx_skia_blitter_new.cpp", 525 "core/src/fxge/skia/fx_skia_blitter_new.cpp",
517 "core/src/fxge/skia/fx_skia_device.cpp", 526 "core/src/fxge/skia/fx_skia_device.cpp",
518 ] 527 ]
519 include_dirs = [ 528 include_dirs = [
520 "//third_party/skia/include/config", 529 "//third_party/skia/include/config",
521 "//third_party/skia/include/core", 530 "//third_party/skia/include/core",
(...skipping 253 matching lines...) Expand 10 before | Expand all | Expand 10 after
775 "//v8", 784 "//v8",
776 "//v8/include", 785 "//v8/include",
777 "." 786 "."
778 ] 787 ]
779 configs -= [ "//build/config/compiler:chromium_code" ] 788 configs -= [ "//build/config/compiler:chromium_code" ]
780 configs += [ 789 configs += [
781 ":pdfium_config", 790 ":pdfium_config",
782 "//build/config/compiler:no_chromium_code", 791 "//build/config/compiler:no_chromium_code",
783 ] 792 ]
784 } 793 }
OLDNEW
« no previous file with comments | « no previous file | pdfium.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698