Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(22)

Unified Diff: Source/modules/webaudio/AudioContext.h

Issue 940963005: Revert of WebAudio: Fix AudioNode leak in a case that AudioNode is not disconnected from the ... (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/modules/webaudio/AudioContext.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/webaudio/AudioContext.h
diff --git a/Source/modules/webaudio/AudioContext.h b/Source/modules/webaudio/AudioContext.h
index 9102268fb8eb529bc3b9781c6cf11a1f01e15f95..52462a91b678158cadb5e841a9679d590ef182d6 100644
--- a/Source/modules/webaudio/AudioContext.h
+++ b/Source/modules/webaudio/AudioContext.h
@@ -79,10 +79,7 @@
// AudioContext is the cornerstone of the web audio API and all AudioNodes are created from it.
// For thread safety between the audio thread and the main thread, it has a rendering graph locking mechanism.
-class AudioContext
- : public RefCountedGarbageCollectedEventTargetWithInlineData<AudioContext>
- , public ActiveDOMObject
- , private ThreadState::MarkingTask {
+class AudioContext : public RefCountedGarbageCollectedEventTargetWithInlineData<AudioContext>, public ActiveDOMObject {
DEFINE_EVENT_TARGET_REFCOUNTING_WILL_BE_REMOVED(RefCountedGarbageCollected<AudioContext>);
WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(AudioContext);
DEFINE_WRAPPERTYPEINFO();
@@ -106,8 +103,6 @@
bool isInitialized() const { return m_isInitialized; }
bool isOfflineContext() { return m_isOfflineContext; }
- AudioContextState contextState() const { return m_contextState; }
- void setLastZombie(void*);
// Document notification
virtual void stop() override final;
@@ -295,10 +290,6 @@
void initialize();
void uninitialize();
- // ThreadState::MarkingTask functions.
- void willStartMarking(ThreadState&) override;
- void didFinishMarking(ThreadState&) override;
-
// ExecutionContext calls stop twice.
// We'd like to schedule only one stop action for them.
bool m_isStopScheduled;
@@ -408,10 +399,6 @@
bool m_didInitializeContextGraphMutex;
RecursiveMutex m_contextGraphMutex;
volatile ThreadIdentifier m_audioThread;
- // Accessing m_last*Zombie should be protected by
- // m_didInitializeContextGraphMutex.
- void* m_lastZombie;
- void* m_lastRemovableZombie;
// Only accessed in the audio thread.
// Oilpan: Since items are added to these vectors by the audio thread (not registered to Oilpan),
« no previous file with comments | « no previous file | Source/modules/webaudio/AudioContext.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698