• Peter Kasting's avatar
    Fix problems with gtest_enable_absl_printers and enable by default. · b5edda77
    Peter Kasting authored
    * With this turned on, gtest unconditionally asks absl to initialize
      the symbol handler.  This causes base's attempt to do the same thing
      to fail.  Make base handle this by trying a force-reinit if this
      failure occurs.
    * Allow gtest to see the full absl sources, since it unconditionally
      #includes banned headers.
    * Fix include dirs not being set correctly, leading to compile failures.
    * Narrow gtest absl dependency from public_deps to deps, since we don't
      want "all targets depending on gtest" to silently be able to use absl.
    * Leave this flag disabled by default for NaCl, since the toolchain
      there can't compile some of the absl code.
    
    Bug: none
    Change-Id: I24930d1c2086bec1522922665708abee5ce8fc08
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332038
    Commit-Queue: Peter Kasting <pkasting@chromium.org>
    Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
    Auto-Submit: Peter Kasting <pkasting@chromium.org>
    Reviewed-by: default avatarBruce Dawson <brucedawson@chromium.org>
    Reviewed-by: default avatarDirk Pranke <dpranke@google.com>
    Cr-Commit-Position: refs/heads/master@{#793786}
    b5edda77
absl.gni 2.98 KB