AudioBufferSourceNode.buffer can only be set once.

Source/modules/webaudio/AudioBufferSourceNode.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 322 matching lines @@
     m_virtualReadIndex = virtualReadIndex;
 
     return true;
 }
 
 
 void AudioBufferSourceNode::setBuffer(AudioBuffer* buffer, ExceptionState& exceptionState)
 {
     ASSERT(isMainThread());
 
+    if (m_buffer) {
+        exceptionState.throwDOMException(
+            InvalidStateError,
+            "buffer can only be set once.");

[hongchan, 2015/01/16 23:42:09]

Perhaps the first word should be capitalized? Other places in chromium code base
are throwing properly capitalized message when it is a complete sentence.

WDYT?

[Raymond Toy, 2015/01/17 00:03:16]

Will do. And get rid of the trailing period.

[Raymond Toy, 2015/01/20 18:06:00]

Done.

+        return;
+    }
+
     // The context must be locked since changing the buffer can re-configure the number of channels that are output.
     AudioContext::AutoLocker contextLocker(context());
 
     // This synchronizes with process().
     MutexLocker processLocker(m_processLock);
 
     if (buffer) {
         // Do any necesssary re-configuration to the buffer's number of channels.
         unsigned numberOfChannels = buffer->numberOfChannels();
 
@@ skipping 158 matching lines @@
 {
     visitor->trace(m_buffer);
     visitor->trace(m_playbackRate);
     visitor->trace(m_pannerNode);
     AudioScheduledSourceNode::trace(visitor);
 }
 
 } // namespace blink
 
 #endif // ENABLE(WEB_AUDIO)