Note creation of AudioContext when DEBUG_AUDIONODE_REFERENCES is set.

Note creation of AudioContext when DEBUG_AUDIONODE_REFERENCES is set.

When DEBUG_AUDIONODE_REFERENCES is set, a message for the destruction
of an AudioContext is printed, but there's no matching message for the
creation of an AudioContext. Add one to make it symmetric and so we
can see how many contexts are created.

BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=178958

[Raymond Toy, 2014-07-25 17:04:28]

PTAL at this trivial change that has no effect on normal usage. It's only for
debugging references.

[Ken Russell (switch to Gerrit), 2014-07-25 18:28:56]

LGTM, but:

https://codereview.chromium.org/421603002/diff/1/Source/modules/webaudio/Audi...
File Source/modules/webaudio/AudioContext.cpp (right):

https://codereview.chromium.org/421603002/diff/1/Source/modules/webaudio/Audi...
Source/modules/webaudio/AudioContext.cpp:129: fprintf(stderr, "%p:
AudioContext::AudioContext() #%u\n", this,
AudioContext::s_hardwareContextCount);
Why not just use WTF_LOG from wtf/Assertions.h?

[Raymond Toy, 2014-07-25 18:41:28]

On 2014/07/25 18:28:56, Ken Russell wrote:
> LGTM, but:
> 
>
https://codereview.chromium.org/421603002/diff/1/Source/modules/webaudio/Audi...
> File Source/modules/webaudio/AudioContext.cpp (right):
> 
>
https://codereview.chromium.org/421603002/diff/1/Source/modules/webaudio/Audi...
> Source/modules/webaudio/AudioContext.cpp:129: fprintf(stderr, "%p:
> AudioContext::AudioContext() #%u\n", this,
> AudioContext::s_hardwareContextCount);
> Why not just use WTF_LOG from wtf/Assertions.h?

Because that's what is used everywhere else. Perhaps it's time to change to use
WTF_LOG everywhere.

[Ken Russell (switch to Gerrit), 2014-07-25 19:41:56]

On 2014/07/25 18:41:28, Raymond Toy wrote:
> On 2014/07/25 18:28:56, Ken Russell wrote:
> > LGTM, but:
> > 
> >
>
https://codereview.chromium.org/421603002/diff/1/Source/modules/webaudio/Audi...
> > File Source/modules/webaudio/AudioContext.cpp (right):
> > 
> >
>
https://codereview.chromium.org/421603002/diff/1/Source/modules/webaudio/Audi...
> > Source/modules/webaudio/AudioContext.cpp:129: fprintf(stderr, "%p:
> > AudioContext::AudioContext() #%u\n", this,
> > AudioContext::s_hardwareContextCount);
> > Why not just use WTF_LOG from wtf/Assertions.h?
> 
> Because that's what is used everywhere else. Perhaps it's time to change to
use
> WTF_LOG everywhere.

OK. If you decide to do this then let's do it in a separate CL.

[Raymond Toy, 2014-07-25 20:14:20]

On 2014/07/25 19:41:56, Ken Russell wrote:
> On 2014/07/25 18:41:28, Raymond Toy wrote:
> > On 2014/07/25 18:28:56, Ken Russell wrote:
> > > LGTM, but:
> > > 
> > >
> >
>
https://codereview.chromium.org/421603002/diff/1/Source/modules/webaudio/Audi...
> > > File Source/modules/webaudio/AudioContext.cpp (right):
> > > 
> > >
> >
>
https://codereview.chromium.org/421603002/diff/1/Source/modules/webaudio/Audi...
> > > Source/modules/webaudio/AudioContext.cpp:129: fprintf(stderr, "%p:
> > > AudioContext::AudioContext() #%u\n", this,
> > > AudioContext::s_hardwareContextCount);
> > > Why not just use WTF_LOG from wtf/Assertions.h?
> > 
> > Because that's what is used everywhere else. Perhaps it's time to change to
> use
> > WTF_LOG everywhere.
> 
> OK. If you decide to do this then let's do it in a separate CL.

Will do.  Thanks.

[Raymond Toy, 2014-07-25 20:14:45]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2014-07-25 20:15:06]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rtoy@chromium.org/421603002/1

[commit-bot: I haz the power, 2014-07-25 21:18:09]

Change committed as 178958