build.gn fix.

1 files changed, 45 insertions, 11 deletions

diff --git a/www-client/chromium/files/chromium-142-compiler.patch b/www-client/chromium/files/chromium-142-compiler.patch
index 0970c7c..8b9b331 100644
--- a/www-client/chromium/files/chromium-142-compiler.patch
+++ b/www-client/chromium/files/chromium-142-compiler.patch
@@ -1,5 +1,5 @@
 diff --git a/build/config/compiler/BUILD.gn b/build/config/compiler/BUILD.gn
-index 9cf9079bc5..a7db932900 100644
+index 9cf9079bc5..fe08327b93 100644
 --- a/build/config/compiler/BUILD.gn
 +++ b/build/config/compiler/BUILD.gn
 @@ -261,15 +261,6 @@ declare_args() {
@@ -326,7 +326,41 @@ index 9cf9079bc5..a7db932900 100644
    if (use_clang_modules) {
      cflags_cc = [
        "-fmodules",
-@@ -2286,11 +2050,7 @@ config("chromium_code") {
+@@ -2163,18 +1927,6 @@ config("default_warnings") {
+       ]
+     }
+ 
+-    if (is_component_build && use_clang_warning_suppression_file && !is_win) {
+-      # Warn for globally unique objects which might get duplicated if built
+-      # into different components, due to hidden visibility. Only applicable
+-      # to component builds.
+-
+-      # For details about the warning and now to fix it, see the official docs at
+-      # https://clang.llvm.org/docs/DiagnosticsReference.html#wunique-object-duplication
+-      # If you still have trouble resolving it, file a bug asking for advice in the
+-      # Chromium > Tools > LLVM component, and cc dloehr and thakis.
+-      cflags_cc += [ "-Wunique-object-duplication" ]
+-    }
+-
+     if (is_wasm) {
+       # Abseil's `symbolize_emscripten.inc` file currently has macros that are
+       # not compatible with C++98, so turn off the warnings.
+@@ -2206,14 +1958,6 @@ config("default_warnings") {
+     if (cc_wrapper == "ccache" || cc_wrapper == "icecc") {
+       cflags += [ "-Wno-gnu-line-marker" ]
+     }
+-
+-    if (is_wexit_time_destructors_default &&
+-        use_clang_warning_suppression_file) {
+-      # TODO(crbug.com/40031409): We are migrating towards making this warning
+-      # enabled by default. Once that is completed, the warning
+-      # should always be added for clang builds.
+-      cflags += [ "-Wexit-time-destructors" ]
+-    }
+   }
+ 
+   # TODO(crbug.com/354162568): Clean up and enable.
+@@ -2286,11 +2030,7 @@ config("chromium_code") {
        defines = [ "_HAS_NODISCARD" ]
      }
    } else {
@@ -339,7 +373,7 @@ index 9cf9079bc5..a7db932900 100644
  
      # In Chromium code, we define __STDC_foo_MACROS in order to get the
      # C99 macros on Mac and Linux.
-@@ -2299,24 +2059,6 @@ config("chromium_code") {
+@@ -2299,24 +2039,6 @@ config("chromium_code") {
        "__STDC_FORMAT_MACROS",
      ]
  
@@ -364,7 +398,7 @@ index 9cf9079bc5..a7db932900 100644
      if (is_apple) {
        cflags_objc = [ "-Wimplicit-retain-self" ]
        cflags_objcc = [ "-Wimplicit-retain-self" ]
-@@ -2732,7 +2474,8 @@ config("default_stack_frames") {
+@@ -2732,7 +2454,8 @@ config("default_stack_frames") {
  # [0]: https://pinpoint-dot-chromeperf.appspot.com/job/147634a8be0000
  # [1]: https://pinpoint-dot-chromeperf.appspot.com/job/132bc772be0000
  # [2]: https://crrev.com/c/5447532
@@ -374,7 +408,7 @@ index 9cf9079bc5..a7db932900 100644
    if (is_win) {
      # clang-cl's /O2 corresponds to clang's -O3, and really want -O2 for
      # consistency with the other platforms.
-@@ -2781,7 +2524,8 @@ config("optimize") {
+@@ -2781,7 +2504,8 @@ config("optimize") {
  }
  
  # Turn off optimizations.
@@ -384,7 +418,7 @@ index 9cf9079bc5..a7db932900 100644
    if (is_win) {
      cflags = [
        "/Od",  # Disable optimization.
-@@ -2816,7 +2560,8 @@ config("no_optimize") {
+@@ -2816,7 +2540,8 @@ config("no_optimize") {
  # Turns up the optimization level. Used to explicitly enable -O2 instead of
  # -Os for select targets on platforms that use optimize_for_size. No-op
  # elsewhere.
@@ -394,7 +428,7 @@ index 9cf9079bc5..a7db932900 100644
    ldflags = common_optimize_on_ldflags
    if (is_win) {
      # Favor speed over size, /O2 must be before the common flags.
-@@ -2837,7 +2582,8 @@ config("optimize_max") {
+@@ -2837,7 +2562,8 @@ config("optimize_max") {
  #
  # TODO(crbug.com/41259697) - rework how all of these configs are related
  # so that we don't need this disclaimer.
@@ -404,7 +438,7 @@ index 9cf9079bc5..a7db932900 100644
    ldflags = common_optimize_on_ldflags
    if (is_win) {
      # Favor speed over size, /O2 must be before the common flags.
-@@ -2854,7 +2600,8 @@ config("optimize_speed") {
+@@ -2854,7 +2580,8 @@ config("optimize_speed") {
    rustflags = [ "-Copt-level=3" ]
  }
  
@@ -414,7 +448,7 @@ index 9cf9079bc5..a7db932900 100644
    cflags = [ "-O1" ] + common_optimize_on_cflags
    rustflags = [ "-Copt-level=1" ]
    ldflags = common_optimize_on_ldflags
-@@ -2981,7 +2728,8 @@ config("win_pdbaltpath") {
+@@ -2981,7 +2708,8 @@ config("win_pdbaltpath") {
  }
  
  # Full symbols.
@@ -424,7 +458,7 @@ index 9cf9079bc5..a7db932900 100644
    rustflags = []
    configs = []
    if (is_win) {
-@@ -3137,7 +2885,8 @@ config("symbols") {
+@@ -3137,7 +2865,8 @@ config("symbols") {
  # Minimal symbols.
  # This config guarantees to hold symbol for stack trace which are shown to user
  # when crash happens in unittests running on buildbot.
@@ -434,7 +468,7 @@ index 9cf9079bc5..a7db932900 100644
    rustflags = []
    if (is_win) {
      # Functions, files, and line tables only.
-@@ -3213,7 +2962,8 @@ config("minimal_symbols") {
+@@ -3213,7 +2942,8 @@ config("minimal_symbols") {
  # This configuration contains function names only. That is, the compiler is
  # told to not generate debug information and the linker then just puts function
  # names in the final debug information.