Move handle of HRTFDatabaseLoader to AudioListener.

Source/modules/webaudio/AudioListener.h

 /*
  * 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
@@ skipping 20 matching lines @@
 
 #include "bindings/v8/ScriptWrappable.h"
 #include "platform/geometry/FloatPoint3D.h"
 #include "platform/heap/Handle.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/RefCounted.h"
 #include "wtf/Vector.h"
 
 namespace WebCore {
 
+class HRTFDatabaseLoader;
 class PannerNode;
 
 // AudioListener maintains the state of the listener in the audio scene as defined in the OpenAL specification.
 
 class AudioListener : public RefCountedWillBeGarbageCollectedFinalized<AudioListener>, public ScriptWrappable {
 public:
     static PassRefPtrWillBeRawPtr<AudioListener> create()
     {
         return adoptRefWillBeNoop(new AudioListener());
     }
@@ skipping 21 matching lines @@
     double dopplerFactor() const { return m_dopplerFactor; }
 
     // Speed of sound
     void setSpeedOfSound(double);
     double speedOfSound() const { return m_speedOfSound; }
 
     Mutex& listenerLock() { return m_listenerLock; }
     void addPanner(PannerNode*);
     void removePanner(PannerNode*);
 
+    void setHRTFDatabaseLoader(PassRefPtrWillBeRawPtr<HRTFDatabaseLoader>);

[tkent, 2014/07/16 23:32:25]

PassRefPtrWIllBeRawPtr should be PassRefPtr because HRTFDatabaseLoader is not a
garbage-collected class.

[KhNo, 2014/07/17 16:23:17]

Done.

+    HRTFDatabaseLoader* hrtfDatabaseLoader() { return m_hrtfDatabaseLoader.get(); }
+
     void trace(Visitor*) { }
 
 private:
     AudioListener();
 
     void setPosition(const FloatPoint3D&);
     void setOrientation(const FloatPoint3D&);
     void setUpVector(const FloatPoint3D&);
     void setVelocity(const FloatPoint3D&);
 
     void markPannersAsDirty(unsigned);
 
     FloatPoint3D m_position;
     FloatPoint3D m_orientation;
     FloatPoint3D m_upVector;
     FloatPoint3D m_velocity;
     double m_dopplerFactor;
     double m_speedOfSound;
 
     // 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 when the PannerNode is
constructed?  Based on the constructor for the panner node below, this handle is
set for every new panner that is created.

[KhNo, 2014/07/17 16:23:17]

Yes, you right. I missed it. Basically it should be set only once.
I will add condition skip it if it is already existed.
Thanks for review.

+    // 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
 
 #endif // AudioListener_h