Commit acbd9fd3 authored by rtoy's avatar rtoy Committed by Commit bot

Minor cosmetic changes

First, use %16p in debugging prints in offline context so that the
context address lines up with other uses of the address.

Second, add numbers to the AudioNode:NodeType enum. I'm tired of
counting every time I need to look up a node number when
DEBUG_AUDIONODE_REFERENCES is enabled.

BUG=
TESTS=

Review-Url: https://codereview.chromium.org/2269543003
Cr-Commit-Position: refs/heads/master@{#413467}
parent f19cf120
...@@ -70,27 +70,27 @@ public: ...@@ -70,27 +70,27 @@ public:
enum { ProcessingSizeInFrames = 128 }; enum { ProcessingSizeInFrames = 128 };
enum NodeType { enum NodeType {
NodeTypeUnknown, NodeTypeUnknown = 0,
NodeTypeDestination, NodeTypeDestination = 1,
NodeTypeOscillator, NodeTypeOscillator = 2,
NodeTypeAudioBufferSource, NodeTypeAudioBufferSource = 3,
NodeTypeMediaElementAudioSource, NodeTypeMediaElementAudioSource = 4,
NodeTypeMediaStreamAudioDestination, NodeTypeMediaStreamAudioDestination = 5,
NodeTypeMediaStreamAudioSource, NodeTypeMediaStreamAudioSource = 6,
NodeTypeJavaScript, NodeTypeJavaScript = 7,
NodeTypeBiquadFilter, NodeTypeBiquadFilter = 8,
NodeTypePanner, NodeTypePanner = 9,
NodeTypeStereoPanner, NodeTypeStereoPanner = 10,
NodeTypeConvolver, NodeTypeConvolver = 11,
NodeTypeDelay, NodeTypeDelay = 12,
NodeTypeGain, NodeTypeGain = 13,
NodeTypeChannelSplitter, NodeTypeChannelSplitter = 14,
NodeTypeChannelMerger, NodeTypeChannelMerger = 15,
NodeTypeAnalyser, NodeTypeAnalyser = 16,
NodeTypeDynamicsCompressor, NodeTypeDynamicsCompressor = 17,
NodeTypeWaveShaper, NodeTypeWaveShaper = 18,
NodeTypeIIRFilter, NodeTypeIIRFilter = 19,
NodeTypeEnd NodeTypeEnd = 20
}; };
AudioHandler(NodeType, AudioNode&, float sampleRate); AudioHandler(NodeType, AudioNode&, float sampleRate);
......
...@@ -92,7 +92,7 @@ OfflineAudioContext* OfflineAudioContext::create(ExecutionContext* context, unsi ...@@ -92,7 +92,7 @@ OfflineAudioContext* OfflineAudioContext::create(ExecutionContext* context, unsi
} }
#if DEBUG_AUDIONODE_REFERENCES #if DEBUG_AUDIONODE_REFERENCES
fprintf(stderr, "[%p]: OfflineAudioContext::OfflineAudioContext()\n", audioContext); fprintf(stderr, "[%16p]: OfflineAudioContext::OfflineAudioContext()\n", audioContext);
#endif #endif
DEFINE_STATIC_LOCAL(SparseHistogram, offlineContextChannelCountHistogram, DEFINE_STATIC_LOCAL(SparseHistogram, offlineContextChannelCountHistogram,
("WebAudio.OfflineAudioContext.ChannelCount")); ("WebAudio.OfflineAudioContext.ChannelCount"));
...@@ -141,7 +141,7 @@ OfflineAudioContext::OfflineAudioContext(Document* document, unsigned numberOfCh ...@@ -141,7 +141,7 @@ OfflineAudioContext::OfflineAudioContext(Document* document, unsigned numberOfCh
OfflineAudioContext::~OfflineAudioContext() OfflineAudioContext::~OfflineAudioContext()
{ {
#if DEBUG_AUDIONODE_REFERENCES #if DEBUG_AUDIONODE_REFERENCES
fprintf(stderr, "[%p]: OfflineAudioContext::~OfflineAudioContext()\n", this); fprintf(stderr, "[%16p]: OfflineAudioContext::~OfflineAudioContext()\n", this);
#endif #endif
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment