reflow comments in modules/{webaudio,vr}

third_party/WebKit/Source/modules/webaudio/AudioNode.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 140 matching lines @@
       return "WaveShaperNode";
     case NodeTypeUnknown:
     case NodeTypeEnd:
     default:
       ASSERT_NOT_REACHED();
       return "UnknownNode";
   }
 }
 
 void AudioHandler::setNodeType(NodeType type) {
-  // Don't allow the node type to be changed to a different node type, after it's already been
-  // set!  And the new type can't be unknown or end!
+  // Don't allow the node type to be changed to a different node type, after
+  // it's already been set.  And the new type can't be unknown or end.
   DCHECK_EQ(m_nodeType, NodeTypeUnknown);
   DCHECK_NE(type, NodeTypeUnknown);
   DCHECK_NE(type, NodeTypeEnd);
 
   m_nodeType = type;
 
 #if DEBUG_AUDIONODE_REFERENCES
   ++s_nodeCount[type];
   fprintf(stderr, "[%16p]: %16p: %2d: AudioHandler::AudioHandler [%3d]\n",
           context(), this, getNodeType(), s_nodeCount[getNodeType()]);
@@ skipping 130 matching lines @@
     input->changedOutputs();
 }
 
 void AudioHandler::processIfNecessary(size_t framesToProcess) {
   DCHECK(context()->isAudioThread());
 
   if (!isInitialized())
     return;
 
   // Ensure that we only process once per rendering quantum.
-  // This handles the "fanout" problem where an output is connected to multiple inputs.
-  // The first time we're called during this time slice we process, but after that we don't want to re-process,
-  // instead our output(s) will already have the results cached in their bus;
+  // This handles the "fanout" problem where an output is connected to multiple
+  // inputs.  The first time we're called during this time slice we process, but
+  // after that we don't want to re-process, instead our output(s) will already
+  // have the results cached in their bus;
   double currentTime = context()->currentTime();
   if (m_lastProcessingTime != currentTime) {
-    m_lastProcessingTime =
-        currentTime;  // important to first update this time because of feedback loops in the rendering graph
+    // important to first update this time because of feedback loops in the
+    // rendering graph.
+    m_lastProcessingTime = currentTime;
 
     pullInputs(framesToProcess);
 
     bool silentInputs = inputsAreSilent();
     if (!silentInputs)
       m_lastNonSilentTime =
           (context()->currentSampleFrame() + framesToProcess) /
           static_cast<double>(m_sampleRate);
 
     if (silentInputs && propagatesSilence()) {
       silenceOutputs();
     } else {
-      // Unsilence the outputs first because the processing of the node may cause the outputs
-      // to go silent and we want to propagate that hint to the downstream nodes!  (For
-      // example, a Gain node with a gain of 0 will want to silence its output.)
+      // Unsilence the outputs first because the processing of the node may
+      // cause the outputs to go silent and we want to propagate that hint to
+      // the downstream nodes.  (For example, a Gain node with a gain of 0 will
+      // want to silence its output.)
       unsilenceOutputs();
       process(framesToProcess);
     }
   }
 }
 
 void AudioHandler::checkNumberOfChannelsForInput(AudioNodeInput* input) {
   DCHECK(context()->isAudioThread());
   ASSERT(context()->isGraphOwner());
 
@@ skipping 48 matching lines @@
     DCHECK(isMainThread());
     BaseAudioContext::AutoLocker locker(context());
 
     m_isDisabled = false;
     for (auto& output : m_outputs)
       output->enable();
   }
 }
 
 void AudioHandler::disableOutputsIfNecessary() {
-  // Disable outputs if appropriate. We do this if the number of connections is 0 or 1. The case
-  // of 0 is from deref() where there are no connections left. The case of 1 is from
-  // AudioNodeInput::disable() where we want to disable outputs when there's only one connection
-  // left because we're ready to go away, but can't quite yet.
+  // Disable outputs if appropriate. We do this if the number of connections is
+  // 0 or 1. The case of 0 is from deref() where there are no connections left.
+  // The case of 1 is from AudioNodeInput::disable() where we want to disable
+  // outputs when there's only one connection left because we're ready to go
+  // away, but can't quite yet.
   if (m_connectionRefCount <= 1 && !m_isDisabled) {
-    // Still may have JavaScript references, but no more "active" connection references, so put all of our outputs in a "dormant" disabled state.
-    // Garbage collection may take a very long time after this time, so the "dormant" disabled nodes should not bog down the rendering...
+    // Still may have JavaScript references, but no more "active" connection
+    // references, so put all of our outputs in a "dormant" disabled state.
+    // Garbage collection may take a very long time after this time, so the
+    // "dormant" disabled nodes should not bog down the rendering...
 
-    // As far as JavaScript is concerned, our outputs must still appear to be connected.
-    // But internally our outputs should be disabled from the inputs they're connected to.
-    // disable() can recursively deref connections (and call disable()) down a whole chain of connected nodes.
+    // As far as JavaScript is concerned, our outputs must still appear to be
+    // connected.  But internally our outputs should be disabled from the inputs
+    // they're connected to.  disable() can recursively deref connections (and
+    // call disable()) down a whole chain of connected nodes.
 
-    // TODO(rtoy,hongchan): we need special cases the convolver, delay, biquad, and IIR since
-    // they have a significant tail-time and shouldn't be disconnected simply because they no
-    // longer have any input connections. This needs to be handled more generally where
-    // AudioNodes have a tailTime attribute. Then the AudioNode only needs to remain "active"
-    // for tailTime seconds after there are no longer any active connections.
+    // TODO(rtoy,hongchan): we need special cases the convolver, delay, biquad,
+    // and IIR since they have a significant tail-time and shouldn't be
+    // disconnected simply because they no longer have any input connections.
+    // This needs to be handled more generally where AudioNodes have a tailTime
+    // attribute. Then the AudioNode only needs to remain "active" for tailTime
+    // seconds after there are no longer any active connections.
     if (getNodeType() != NodeTypeConvolver && getNodeType() != NodeTypeDelay &&
         getNodeType() != NodeTypeBiquadFilter &&
         getNodeType() != NodeTypeIIRFilter) {
       m_isDisabled = true;
       clearInternalStateWhenDisabled();
       for (auto& output : m_outputs)
         output->disable();
     }
   }
 }
@@ skipping 71 matching lines @@
 void AudioHandler::updateChannelCountMode() {
   m_channelCountMode = m_newChannelCountMode;
   updateChannelsForInputs();
 }
 
 void AudioHandler::updateChannelInterpretation() {
   m_channelInterpretation = m_newChannelInterpretation;
 }
 
 unsigned AudioHandler::numberOfOutputChannels() const {
-  // This should only be called for ScriptProcessorNodes which are the only nodes where you can
-  // have an output with 0 channels.  All other nodes have have at least one output channel, so
-  // there's no reason other nodes should ever call this function.
+  // This should only be called for ScriptProcessorNodes which are the only
+  // nodes where you can have an output with 0 channels.  All other nodes have
+  // have at least one output channel, so there's no reason other nodes should
+  // ever call this function.
   DCHECK(0) << "numberOfOutputChannels() not valid for node type "
             << getNodeType();
   return 1;
 }
 // ----------------------------------------------------------------
 
 AudioNode::AudioNode(BaseAudioContext& context)
     : m_context(context), m_handler(nullptr) {
   ThreadState::current()->registerPreFinalizer(this);
 }
@@ skipping 84 matching lines @@
     return nullptr;
   }
 
   if (context() != destination->context()) {
     exceptionState.throwDOMException(SyntaxError,
                                      "cannot connect to a destination "
                                      "belonging to a different audio context.");
     return nullptr;
   }
 
-  // ScriptProcessorNodes with 0 output channels can't be connected to any destination.  If there
-  // are no output channels, what would the destination receive?  Just disallow this.
+  // ScriptProcessorNodes with 0 output channels can't be connected to any
+  // destination.  If there are no output channels, what would the destination
+  // receive?  Just disallow this.
   if (handler().getNodeType() == AudioHandler::NodeTypeJavaScript &&
       handler().numberOfOutputChannels() == 0) {
     exceptionState.throwDOMException(InvalidAccessError,
                                      "cannot connect a ScriptProcessorNode "
                                      "with 0 output channels to any "
                                      "destination node.");
     return nullptr;
   }
 
   destination->handler()
@@ skipping 304 matching lines @@
 }
 
 void AudioNode::didAddOutput(unsigned numberOfOutputs) {
   m_connectedNodes.append(nullptr);
   DCHECK_EQ(numberOfOutputs, m_connectedNodes.size());
   m_connectedParams.append(nullptr);
   DCHECK_EQ(numberOfOutputs, m_connectedParams.size());
 }
 
 }  // namespace blink