Remove #if ENABLE(ASSERT) in bool MutexBase::locked().

Remove #if ENABLE(ASSERT) in bool MutexBase::locked().

It is dificult to replace ASSERT with DCHECK if we are using MutexBase::locked().
Hence we need to remove #if ENABLE(ASSERT) in bool MutexBase::locked().

BUG=626446
R=mikhail.pozdnyakov@intel.com

[HyungwookLee, 2016-07-28 11:41:15]

Description was changed from

==========
Removing #if ENABLE(ASSERT) in bool MutexBase::locked().

It is dificult to replace ASSERT with DCHECK if we are using
MutexBase::locked().
Hence we need to remove #if ENABLE(ASSERT) in bool MutexBase::locked().

BUG=626446
==========

to

==========
Removing #if ENABLE(ASSERT) in bool MutexBase::locked().

It is dificult to replace ASSERT with DCHECK if we are using
MutexBase::locked().
Hence we need to remove #if ENABLE(ASSERT) in bool MutexBase::locked().

BUG=626446
R=mikhail.pozdnyakov@intel.com
==========

[HyungwookLee, 2016-07-28 11:43:32]

hyungwook.lee@navercorp.com changed reviewers:
+ mikhail.pozdnyakov@intel.com

[HyungwookLee, 2016-08-01 00:59:47]

hyungwook.lee@navercorp.com changed reviewers:
+ esprehn@chromium.org

[HyungwookLee, 2016-08-01 01:04:29]

Description was changed from

==========
Removing #if ENABLE(ASSERT) in bool MutexBase::locked().

It is dificult to replace ASSERT with DCHECK if we are using
MutexBase::locked().
Hence we need to remove #if ENABLE(ASSERT) in bool MutexBase::locked().

BUG=626446
R=mikhail.pozdnyakov@intel.com
==========

to

==========
Remove #if ENABLE(ASSERT) in bool MutexBase::locked().

It is dificult to replace ASSERT with DCHECK if we are using
MutexBase::locked().
Hence we need to remove #if ENABLE(ASSERT) in bool MutexBase::locked().

BUG=626446
R=mikhail.pozdnyakov@intel.com
==========

[esprehn, 2016-08-05 23:18:56]

I think you want to instead do:

#if ENABLE(ASSERT)
  bool locked() { return ...; }
#else
  bool locked() { RELEASE_NOTREACHED(); return false; }
#endif

I don't think it's worth switching to DCHECK if it makes all the locks bigger
with members.

https://codereview.chromium.org/2191853002/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/wtf/ThreadingPrimitives.h (right):

https://codereview.chromium.org/2191853002/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/wtf/ThreadingPrimitives.h:53: size_t m_recursionCount;
I don't think we want to do this, it makes all of the mutexes bigger.