Update webrtc/base/common.h after recent _DEBUG->!NDEBUG change.

Update webrtc/base/common.h after recent _DEBUG->!NDEBUG change.

R=tfarina@chromium.org
TBR=tfarina@chromium.org
BUG=

Committed: https://crrev.com/e502bbe138599a10fe530cb789e3f715a56fd461
Cr-Commit-Position: refs/heads/master@{#10470}

[tommi, 2015-10-31 21:29:20]

The CQ bit was checked by tommi@webrtc.org

[commit-bot: I haz the power, 2015-10-31 21:29:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1410113007/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1410113007/1

[tfarina, 2015-10-31 21:33:38]

https://codereview.webrtc.org/1410113007/diff/1/webrtc/base/common.h
File webrtc/base/common.h (right):

https://codereview.webrtc.org/1410113007/diff/1/webrtc/base/common.h#newcode64
webrtc/base/common.h:64: #if !defined(NDEBUG)
LGTM

[tfarina, 2015-10-31 21:35:32]

Unrelated question below.

https://codereview.webrtc.org/1410113007/diff/1/webrtc/base/common.h
File webrtc/base/common.h (right):

https://codereview.webrtc.org/1410113007/diff/1/webrtc/base/common.h#newcode57
webrtc/base/common.h:57: #define ARRAY_SIZE(x) (static_cast<int>(sizeof(x) /
sizeof(x[0])))
Can we put this inline with the version in Chromium base/macros.h:arraysize?

https://chromium.googlesource.com/chromium/src/+/master/base/macros.h#47

[tommi, 2015-10-31 21:38:28]

https://codereview.webrtc.org/1410113007/diff/1/webrtc/base/common.h
File webrtc/base/common.h (right):

https://codereview.webrtc.org/1410113007/diff/1/webrtc/base/common.h#newcode57
webrtc/base/common.h:57: #define ARRAY_SIZE(x) (static_cast<int>(sizeof(x) /
sizeof(x[0])))
On 2015/10/31 21:35:32, tfarina wrote:
> Can we put this inline with the version in Chromium base/macros.h:arraysize?
> 
> https://chromium.googlesource.com/chromium/src/+/master/base/macros.h#47

There is a webrtc/base/arraysize.h already actually and it's what's being used
in new code. This macro will go away.

[tommi, 2015-10-31 21:41:50]

Committed patchset #1 (id:1) manually as
e502bbe138599a10fe530cb789e3f715a56fd461 (presubmit successful).

[commit-bot: I haz the power, 2015-10-31 21:41:56]

Patchset 1 (id:??) landed as
https://crrev.com/e502bbe138599a10fe530cb789e3f715a56fd461
Cr-Commit-Position: refs/heads/master@{#10470}