media: Replace wtf/Assertions.h macros in favor of base/logging.h macros

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

 /*
  * Copyright (C) 2011, 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 55 matching lines @@
 {
     m_mediaElement->setAudioSourceNode(nullptr);
     AudioHandler::dispose();
 }
 
 void MediaElementAudioSourceHandler::setFormat(size_t numberOfChannels, float sourceSampleRate)
 {
     if (numberOfChannels != m_sourceNumberOfChannels || sourceSampleRate != m_sourceSampleRate) {
         if (!numberOfChannels || numberOfChannels > AbstractAudioContext::maxNumberOfChannels() || !AudioUtilities::isValidAudioBufferSampleRate(sourceSampleRate)) {
             // process() will generate silence for these uninitialized values.
-            WTF_LOG(Media, "MediaElementAudioSourceNode::setFormat(%u, %f) - unhandled format change", static_cast<unsigned>(numberOfChannels), sourceSampleRate);
+            DVLOG(3) << "setFormat(" << numberOfChannels << ", " << sourceSampleRate << ") - unhandled format change";

[Raymond Toy, 2016/05/19 14:48:42]

How did you decide on 3?

Doesn't WTF_LOG always produce an output?  DVLOG(3) will only produce an output
if user has set up debugging logging at level 3, right?  If this is so, don't we
want to preserve WTF_LOG behavior?

[Srirama, 2016/05/19 14:54:44]

We are using 3 in core/html media files, to make it less verbose
9https://codereview.chromium.org/1988753002/). So i just followed same here.
Probably we can use 1 here as there are less logs compared to core/html?

[Raymond Toy, 2016/05/19 15:31:37]

I see.

I think these are fairly rare.  Can we just make it always output a message? 
When the condition fails and there's no message, it's really hard to figure out
why there's no audio anymore.

[Srirama, 2016/05/20 07:33:39]

Even WTF_LOG will print only in debug mode and if user enables it through
-blink-platform-log-channels=Media command line switch. Anyway i found DLOG
which will print always in Debug mode. I think that should solve our purpose. So
is it fine?

             // Synchronize with process().
             Locker<MediaElementAudioSourceHandler> locker(*this);
             m_sourceNumberOfChannels = 0;
             m_sourceSampleRate = 0;
             return;
         }
 
         // Synchronize with process() to protect m_sourceNumberOfChannels,
         // m_sourceSampleRate, and m_multiChannelResampler.
         Locker<MediaElementAudioSourceHandler> locker(*this);
@@ skipping 157 matching lines @@
     mediaElementAudioSourceHandler().lock();
 }
 
 void MediaElementAudioSourceNode::unlock()
 {
     mediaElementAudioSourceHandler().unlock();
 }
 
 } // namespace blink