From 4793f674f29be8bf06a60672032e2e2df5164c8c Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Fri, 30 Oct 2015 10:31:03 -0700 Subject: Add standard "Must be after no_chromium_code" comment in GN files. R=thakis@chromium.org Review URL: https://codereview.chromium.org/1411563004 . --- BUILD.gn | 3 +-- third_party/BUILD.gn | 3 +++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/BUILD.gn b/BUILD.gn index abd189b8eb..6accde5b3f 100644 --- a/BUILD.gn +++ b/BUILD.gn @@ -554,8 +554,7 @@ static_library("fxge") { configs += [ ":pdfium_config", "//build/config/compiler:no_chromium_code", - # As of this writing, this needs to be last to suppress a -Wswitch warning. - # https://crbug.com/547906 + # Must be after no_chromium_code for warning flags to be ordered correctly. ":fxge_warnings", ] diff --git a/third_party/BUILD.gn b/third_party/BUILD.gn index f6358147be..e9e50cd0b9 100644 --- a/third_party/BUILD.gn +++ b/third_party/BUILD.gn @@ -46,6 +46,7 @@ static_library("fx_freetype") { configs += [ "//build/config/compiler:no_chromium_code", "//third_party/pdfium:pdfium_config", + # Must be after no_chromium_code for warning flags to be ordered correctly. ":fx_freetype_warnings", ] defines = [ "FT2_BUILD_LIBRARY" ] @@ -95,6 +96,7 @@ source_set("fx_agg") { configs += [ "//build/config/compiler:no_chromium_code", "//third_party/pdfium:pdfium_config", + # Must be after no_chromium_code for warning flags to be ordered correctly. ":fx_agg_warnings", ] sources = [ @@ -134,6 +136,7 @@ source_set("fx_lcms2") { configs += [ "//build/config/compiler:no_chromium_code", "//third_party/pdfium:pdfium_config", + # Must be after no_chromium_code for warning flags to be ordered correctly. ":fx_lcms2_warnings", ] sources = [ -- cgit v1.2.3