WebAudio: Use references instead of pointers in AudioNodeInput and AudioNodeOutput.

Source/modules/webaudio/AudioNodeOutput.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 130 matching lines @@
 {
     ASSERT(context()->isGraphOwner());
     return m_params.size();
 }
 
 unsigned AudioNodeOutput::renderingFanOutCount() const
 {
     return m_renderingFanOutCount;
 }
 
-void AudioNodeOutput::addInput(AudioNodeInput* input)
+void AudioNodeOutput::addInput(AudioNodeInput& input)
 {
     ASSERT(context()->isGraphOwner());
-
-    ASSERT(input);
-    if (!input)
-        return;
-
-    m_inputs.add(input, input->node());
-    input->node()->makeConnection();
+    m_inputs.add(&input, &input.node());
+    input.node().makeConnection();
 }
 
-void AudioNodeOutput::removeInput(AudioNodeInput* input)
+void AudioNodeOutput::removeInput(AudioNodeInput& input)
 {
     ASSERT(context()->isGraphOwner());
-
-    ASSERT(input);
-    if (!input)
-        return;
-
-    input->node()->breakConnection();
-    m_inputs.remove(input);
+    input.node().breakConnection();
+    m_inputs.remove(&input);
 }
 
 void AudioNodeOutput::disconnectAllInputs()
 {
     ASSERT(context()->isGraphOwner());
 
     // AudioNodeInput::disconnect() changes m_inputs by calling removeInput().
     while (!m_inputs.isEmpty())
-        m_inputs.begin()->key->disconnect(this);
+        m_inputs.begin()->key->disconnect(*this);
 }
 
-void AudioNodeOutput::addParam(AudioParam* param)
+void AudioNodeOutput::addParam(AudioParam& param)
 {
     ASSERT(context()->isGraphOwner());
-
-    ASSERT(param);
-    if (!param)
-        return;
-
-    m_params.add(param);
+    m_params.add(&param);
 }
 
-void AudioNodeOutput::removeParam(AudioParam* param)
+void AudioNodeOutput::removeParam(AudioParam& param)
 {
     ASSERT(context()->isGraphOwner());
-
-    ASSERT(param);
-    if (!param)
-        return;
-
-    m_params.remove(param);
+    m_params.remove(&param);
 }
 
 void AudioNodeOutput::disconnectAllParams()
 {
     ASSERT(context()->isGraphOwner());
 
     // AudioParam::disconnect() changes m_params by calling removeParam().
     while (!m_params.isEmpty()) {
         AudioParam* param = m_params.begin()->get();
         param->disconnect(this);
     }
 }
 
 void AudioNodeOutput::disconnectAll()
 {
     disconnectAllInputs();
     disconnectAllParams();
 }
 
 void AudioNodeOutput::disable()
 {
     ASSERT(context()->isGraphOwner());
 
     if (m_isEnabled) {
         for (InputsIterator i = m_inputs.begin(); i != m_inputs.end(); ++i)
-            i->key->disable(this);
+            i->key->disable(*this);
         m_isEnabled = false;
     }
 }
 
 void AudioNodeOutput::enable()
 {
     ASSERT(context()->isGraphOwner());
 
     if (!m_isEnabled) {
         for (InputsIterator i = m_inputs.begin(); i != m_inputs.end(); ++i)
-            i->key->enable(this);
+            i->key->enable(*this);
         m_isEnabled = true;
     }
 }
 
 } // namespace WebCore
 
 #endif // ENABLE(WEB_AUDIO)