Chromium Code Reviews| Index: Source/modules/encryptedmedia/MediaKeySession.h |
| diff --git a/Source/modules/encryptedmedia/MediaKeySession.h b/Source/modules/encryptedmedia/MediaKeySession.h |
| index 45072199301fd42012c494d5aba33a931f055356..f09cf864e638f2a40d165c36c5b1d41e8b51efbc 100644 |
| --- a/Source/modules/encryptedmedia/MediaKeySession.h |
| +++ b/Source/modules/encryptedmedia/MediaKeySession.h |
| @@ -64,7 +64,7 @@ public: |
| void setError(MediaKeyError*); |
| MediaKeyError* error() { return m_error.get(); } |
| - void initializeNewSession(const String& mimeType, Uint8Array* initData); |
| + void initializeNewSession(const String& mimeType, Uint8Array& initData); |
|
ddorwin
2014/01/13 20:20:57
const &?
xhwang
2014/01/14 04:04:11
Done.
|
| void update(Uint8Array* response, ExceptionState&); |
| void release(ExceptionState&); |
| @@ -79,7 +79,6 @@ public: |
| private: |
| MediaKeySession(ExecutionContext*, ContentDecryptionModule*, MediaKeys*); |
| - void initializeNewSessionTimerFired(Timer<MediaKeySession>*); |
| void updateTimerFired(Timer<MediaKeySession>*); |
| // ContentDecryptionModuleSessionClient |
| @@ -94,16 +93,6 @@ private: |
| // Used to remove the reference from the parent MediaKeys when close()'d. |
| MediaKeys* m_keys; |
| - // FIXME: Check whether |initData| can be changed by JS. Maybe we should not pass it as a pointer. |
| - // FIXME: Move the queue and timer to MediaKeys. |
| - struct InitializeNewSessionData { |
| - InitializeNewSessionData(const String& mimeType, Uint8Array* initData) : mimeType(mimeType), initData(initData) { } |
| - String mimeType; |
| - RefPtr<Uint8Array> initData; |
| - }; |
| - Deque<InitializeNewSessionData> m_pendingInitializeNewSessionData; |
| - Timer<MediaKeySession> m_initializeNewSessionTimer; |
| - |
| Deque<RefPtr<Uint8Array> > m_pendingKeys; |
| Timer<MediaKeySession> m_updateTimer; |
| }; |