Remove DCHECK_IS_ON() condition check around *.isGraphOwner() method

third_party/WebKit/Source/modules/webaudio/DeferredTaskHandler.cpp

 /*
  * Copyright (C) 2010, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 47 matching lines @@
 void DeferredTaskHandler::offlineLock() {
   // CHECK is here to make sure to explicitly crash if this is called from
   // other than the offline render thread, which is considered as the audio
   // thread in OfflineAudioContext.
   CHECK(isAudioThread()) << "DeferredTaskHandler::offlineLock() must be called "
                             "within the offline audio thread.";
 
   m_contextGraphMutex.lock();
 }
 
+bool DeferredTaskHandler::isGraphOwner() {
 #if DCHECK_IS_ON()
-bool DeferredTaskHandler::isGraphOwner() {
   return m_contextGraphMutex.locked();
+#else
+  return false;

[Raymond Toy, 2017/04/07 17:11:01]

Add comment why false and not true.

+#endif
 }
-#endif
 
 void DeferredTaskHandler::addDeferredBreakConnection(AudioHandler& node) {
   DCHECK(isAudioThread());
   m_deferredBreakConnectionList.push_back(&node);
 }
 
 void DeferredTaskHandler::breakConnections() {
   DCHECK(isAudioThread());
   ASSERT(isGraphOwner());
 
@@ skipping 198 matching lines @@
   m_deletableOrphanHandlers.clear();
 }
 
 void DeferredTaskHandler::setAudioThreadToCurrentThread() {
   DCHECK(!isMainThread());
   ThreadIdentifier thread = currentThread();
   releaseStore(&m_audioThread, thread);
 }
 
 }  // namespace blink