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

Side by Side Diff: Source/modules/mediasource/SourceBuffer.h

Issue 1149293003: Oilpan: eagerly sweep some mediasource and mediastream objects. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Updated to use EAGERLY_FINALIZE() Created 5 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 virtual void resume() override; 97 virtual void resume() override;
98 virtual void stop() override; 98 virtual void stop() override;
99 99
100 // EventTarget interface 100 // EventTarget interface
101 virtual ExecutionContext* executionContext() const override; 101 virtual ExecutionContext* executionContext() const override;
102 virtual const AtomicString& interfaceName() const override; 102 virtual const AtomicString& interfaceName() const override;
103 103
104 // WebSourceBufferClient interface 104 // WebSourceBufferClient interface
105 virtual void initializationSegmentReceived() override; 105 virtual void initializationSegmentReceived() override;
106 106
107 // Oilpan: eagerly release owned m_webSourceBuffer
108 EAGERLY_FINALIZE();
107 DECLARE_VIRTUAL_TRACE(); 109 DECLARE_VIRTUAL_TRACE();
108 110
109 private: 111 private:
110 SourceBuffer(PassOwnPtr<WebSourceBuffer>, MediaSource*, GenericEventQueue*); 112 SourceBuffer(PassOwnPtr<WebSourceBuffer>, MediaSource*, GenericEventQueue*);
111 113
112 bool isRemoved() const; 114 bool isRemoved() const;
113 void scheduleEvent(const AtomicString& eventName); 115 void scheduleEvent(const AtomicString& eventName);
114 116
115 void appendBufferInternal(const unsigned char*, unsigned, ExceptionState&); 117 void appendBufferInternal(const unsigned char*, unsigned, ExceptionState&);
116 void appendBufferAsyncPart(); 118 void appendBufferAsyncPart();
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 bool m_streamMaxSizeValid; 153 bool m_streamMaxSizeValid;
152 unsigned long long m_streamMaxSize; 154 unsigned long long m_streamMaxSize;
153 AsyncMethodRunner<SourceBuffer> m_appendStreamAsyncPartRunner; 155 AsyncMethodRunner<SourceBuffer> m_appendStreamAsyncPartRunner;
154 Member<Stream> m_stream; 156 Member<Stream> m_stream;
155 OwnPtr<FileReaderLoader> m_loader; 157 OwnPtr<FileReaderLoader> m_loader;
156 }; 158 };
157 159
158 } // namespace blink 160 } // namespace blink
159 161
160 #endif // SourceBuffer_h 162 #endif // SourceBuffer_h
OLDNEW
« no previous file with comments | « no previous file | Source/modules/mediastream/MediaStream.h » ('j') | Source/modules/mediastream/MediaStreamTrack.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698