Chromium Code Reviews| Index: Source/modules/webaudio/AudioListener.h |
| diff --git a/Source/modules/webaudio/AudioListener.h b/Source/modules/webaudio/AudioListener.h |
| index 214db390b700f960513c7b408f0d190be05c3f2e..fdd43a4d1ebb288dd4e412f3445fc2a5af0453be 100644 |
| --- a/Source/modules/webaudio/AudioListener.h |
| +++ b/Source/modules/webaudio/AudioListener.h |
| @@ -38,6 +38,7 @@ |
| namespace WebCore { |
| +class HRTFDatabaseLoader; |
| class PannerNode; |
| // AudioListener maintains the state of the listener in the audio scene as defined in the OpenAL specification. |
| @@ -79,6 +80,9 @@ public: |
| void addPanner(PannerNode*); |
| void removePanner(PannerNode*); |
| + void setHRTFDatabaseLoader(PassRefPtrWillBeRawPtr<HRTFDatabaseLoader>); |
|
tkent
2014/07/16 23:32:25
PassRefPtrWIllBeRawPtr should be PassRefPtr becaus
KhNo
2014/07/17 16:23:17
Done.
|
| + HRTFDatabaseLoader* hrtfDatabaseLoader() { return m_hrtfDatabaseLoader.get(); } |
| + |
| void trace(Visitor*) { } |
| private: |
| @@ -100,9 +104,11 @@ private: |
| // Synchronize a panner's process() with setting of the state of the listener. |
| mutable Mutex m_listenerLock; |
| - |
| // List for pannerNodes in context. |
| Vector<PannerNode*> m_panners; |
| + // HRTF DB loader for panner node. It should be allocated once on constructor of panner node, |
|
Raymond Toy
2014/07/16 16:35:38
How do you enforce that it is allocated only once
KhNo
2014/07/17 16:23:17
Yes, you right. I missed it. Basically it should b
|
| + // then keep handle for efficiant loading. |
|
Raymond Toy
2014/07/16 16:35:38
Typo: efficiant -> efficient
KhNo
2014/07/17 16:23:17
Done.
|
| + RefPtrWillBeRawPtr<HRTFDatabaseLoader> m_hrtfDatabaseLoader; |
|
tkent
2014/07/16 23:32:25
RefPtrWillBeRawPtr should be RefPtr.
KhNo
2014/07/17 16:23:17
Thanks for comment. :)
Done.
|
| }; |
| } // WebCore |