base: Change DCHECK_IS_ON to a macro DCHECK_IS_ON().
commite649f573a38b00bb20fe0925098251a4ff184566
authordanakj <danakj@chromium.org>
Thu, 8 Jan 2015 23:35:58 +0000 (8 15:35 -0800)
committerCommit bot <commit-bot@chromium.org>
Thu, 8 Jan 2015 23:37:17 +0000 (8 23:37 +0000)
tree6f400822726e85def4cad5a1ee704e232a6364ad
parent95bc5b1779dcf8e4a9e37ef600c0ea76293307e3
base: Change DCHECK_IS_ON to a macro DCHECK_IS_ON().

This ensures that if the header is not included, and a DCHECK is guarded
by this check, that the file will fail to compile instead of silently
compiling the DCHECK out.

For example:

#if DCHECK_IS_ON
DCHECK(SomeThing());
#endif

This example would be compiled out if DCHECK_IS_ON was not defined due
to not including the logging.h header.

Instead, this will fail to compile:

#if DCHECK_IS_ON()
DCHECK(SomeThing());
#endif

R=thakis@chromium.org

Review URL: https://codereview.chromium.org/842523002

Cr-Commit-Position: refs/heads/master@{#310626}
53 files changed:
android_webview/browser/hardware_renderer.cc
base/i18n/build_utf8_validator_tables.cc
base/logging.h
base/logging_unittest.cc
base/mac/mac_logging.h
base/mac/mach_logging.h
base/memory/discardable_shared_memory.cc
base/memory/discardable_shared_memory.h
base/metrics/histogram_macros.h
base/tracked_objects.cc
base/tracked_objects.h
cc/base/completion_event.h
cc/layers/delegated_frame_provider.cc
cc/layers/picture_layer_impl.cc
cc/resources/picture_layer_tiling.cc
cc/resources/picture_layer_tiling_set.cc
cc/resources/prioritized_resource.cc
cc/resources/prioritized_resource.h
cc/resources/prioritized_resource_manager.cc
cc/resources/resource_provider.cc
cc/resources/scoped_resource.cc
cc/resources/scoped_resource.h
cc/trees/layer_tree_host_impl.cc
cc/trees/proxy.cc
cc/trees/proxy.h
cc/trees/single_thread_proxy.cc
cc/trees/single_thread_proxy.h
cc/trees/tree_synchronizer.cc
chrome/browser/extensions/activity_log/hashed_ad_network_database.cc
chrome/browser/extensions/extension_reenabler.cc
chrome/browser/ui/toolbar/toolbar_actions_bar.cc
chrome/installer/util/shell_util.cc
chromecast/base/metrics/cast_histograms.h
components/history/core/browser/top_sites_cache.cc
content/browser/android/java/gin_java_bridge_dispatcher_host.cc
content/browser/service_worker/service_worker_database.cc
content/common/gpu/media/gpu_video_decode_accelerator.cc
content/renderer/gpu/queue_message_swap_promise.cc
content/renderer/gpu/queue_message_swap_promise.h
content/renderer/media/remote_media_stream_impl.cc
extensions/renderer/logging_native_handler.cc
extensions/renderer/logging_native_handler.h
extensions/renderer/script_context_set.cc
gpu/command_buffer/service/gles2_cmd_clear_framebuffer.cc
gpu/command_buffer/service/gles2_cmd_decoder_unittest.cc
gpu/command_buffer/service/test_helper.cc
mojo/edk/system/transport_data.cc
net/spdy/spdy_session.cc
net/spdy/spdy_write_queue.cc
sync/internal_api/http_bridge.cc
ui/compositor/dip_util.cc
ui/gfx/font_list_impl.cc
ui/gl/gl_gl_api_implementation.cc