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

Side by Side Diff: BUILD.gn

Issue 1411563004: Add standard "Must be after no_chromium_code" comment in GN files. (Closed) Base URL: https://pdfium.googlesource.com/pdfium@master
Patch Set: rebase Created 5 years, 1 month 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 | third_party/BUILD.gn » ('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 import("pdfium.gni") 6 import("pdfium.gni")
7 7
8 config("pdfium_config") { 8 config("pdfium_config") {
9 cflags = [] 9 cflags = []
10 include_dirs = [ "third_party/freetype/include" ] 10 include_dirs = [ "third_party/freetype/include" ]
(...skipping 536 matching lines...) Expand 10 before | Expand all | Expand 10 after
547 "core/src/fxge/ge/fx_ge_path.cpp", 547 "core/src/fxge/ge/fx_ge_path.cpp",
548 "core/src/fxge/ge/fx_ge_ps.cpp", 548 "core/src/fxge/ge/fx_ge_ps.cpp",
549 "core/src/fxge/ge/fx_ge_text.cpp", 549 "core/src/fxge/ge/fx_ge_text.cpp",
550 "core/src/fxge/ge/text_int.h", 550 "core/src/fxge/ge/text_int.h",
551 ] 551 ]
552 552
553 configs -= [ "//build/config/compiler:chromium_code" ] 553 configs -= [ "//build/config/compiler:chromium_code" ]
554 configs += [ 554 configs += [
555 ":pdfium_config", 555 ":pdfium_config",
556 "//build/config/compiler:no_chromium_code", 556 "//build/config/compiler:no_chromium_code",
557 # As of this writing, this needs to be last to suppress a -Wswitch warning. 557 # Must be after no_chromium_code for warning flags to be ordered correctly.
558 # https://crbug.com/547906
559 ":fxge_warnings", 558 ":fxge_warnings",
560 ] 559 ]
561 560
562 if (pdf_use_skia) { 561 if (pdf_use_skia) {
563 sources = [ 562 sources = [
564 "core/src/fxge/skia/fx_skia_blitter_new.cpp", 563 "core/src/fxge/skia/fx_skia_blitter_new.cpp",
565 "core/src/fxge/skia/fx_skia_device.cpp", 564 "core/src/fxge/skia/fx_skia_device.cpp",
566 ] 565 ]
567 include_dirs = [ 566 include_dirs = [
568 "//third_party/skia/include/config", 567 "//third_party/skia/include/config",
(...skipping 243 matching lines...) Expand 10 before | Expand all | Expand 10 after
812 "//v8", 811 "//v8",
813 "//v8/include", 812 "//v8/include",
814 ] 813 ]
815 } 814 }
816 configs -= [ "//build/config/compiler:chromium_code" ] 815 configs -= [ "//build/config/compiler:chromium_code" ]
817 configs += [ 816 configs += [
818 ":pdfium_config", 817 ":pdfium_config",
819 "//build/config/compiler:no_chromium_code", 818 "//build/config/compiler:no_chromium_code",
820 ] 819 ]
821 } 820 }
OLDNEW
« no previous file with comments | « no previous file | third_party/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698