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

Unified Diff: media/base/win/mf_helpers.h

Issue 2693923003: Fix IMFSample leak in DXVAVideoDecodeAccelerator (Closed)
Patch Set: fix explicit initialization Created 3 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 | media/base/win/mf_helpers.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/win/mf_helpers.h
diff --git a/media/base/win/mf_helpers.h b/media/base/win/mf_helpers.h
index a99ecedbb3fdbf0dd79b008597dfc9ebaab7c774..4877cbe893f0daa65c9fe2f0b643cf7cb7b38866 100644
--- a/media/base/win/mf_helpers.h
+++ b/media/base/win/mf_helpers.h
@@ -47,9 +47,8 @@ MF_INITIALIZER_EXPORT void LogDXVAError(int line);
// Creates a Media Foundation sample with one buffer of length |buffer_length|
// on a |align|-byte boundary. Alignment must be a perfect power of 2 or 0.
-MF_INITIALIZER_EXPORT IMFSample* CreateEmptySampleWithBuffer(
- uint32_t buffer_length,
- int align);
+MF_INITIALIZER_EXPORT base::win::ScopedComPtr<IMFSample>
+CreateEmptySampleWithBuffer(uint32_t buffer_length, int align);
// Provides scoped access to the underlying buffer in an IMFMediaBuffer
// instance.
@@ -74,4 +73,4 @@ class MF_INITIALIZER_EXPORT MediaBufferScopedPointer {
} // namespace media
-#endif // MEDIA_BASE_WIN_MF_HELPERS_H_
+#endif // MEDIA_BASE_WIN_MF_HELPERS_H_
« no previous file with comments | « no previous file | media/base/win/mf_helpers.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698