See https://codereview.chromium.org/1864583006/ which went through CQ, but then broke the debug bots.
The problem was that DLOG(FATAL) was being hit in one test. It was compiled out in release-with-asserts builds.
However, just like DCHECK is hit in release-with-asserts, so should DLOG(FATAL).
Cc: brettw@chromium.org thakis@chromium.org Owner: ---- Status: Available (was: Started)
hmm this turns out to be tricky.
The problem is that if I just enable DLOG in release-with-asserts, then there are a bunch of compile errors because variables/functions which are used in DLOGs are behind "ifndef NDEBUG". I could change them to also have || DCHECK_ALWAYS_ON. However some of this code is in outside repos that don't understand DCHECK_ALWAYS_ON, i.e. leveldb, and they're getting the DLOG indirectly because logging.h redefines assert as DLOG_ASSERT
Comment 1 by jam@chromium.org
, Apr 13 2016Owner: ----
Status: Available (was: Started)