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

Unified Diff: webkit/glue/media/buffered_resource_loader_unittest.cc

Issue 6246017: Fix memory leaks in BufferedDataSource, BufferedResourceLoader and SimpleDataSource unit tests. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src
Patch Set: Created 9 years, 11 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
Index: webkit/glue/media/buffered_resource_loader_unittest.cc
diff --git a/webkit/glue/media/buffered_resource_loader_unittest.cc b/webkit/glue/media/buffered_resource_loader_unittest.cc
index fc8851ca66ca4f09515f14007ad715da894f55e0..4ba8a27583842d96a86876c5716b332cc3cfe930 100644
--- a/webkit/glue/media/buffered_resource_loader_unittest.cc
+++ b/webkit/glue/media/buffered_resource_loader_unittest.cc
@@ -74,7 +74,6 @@ class BufferedResourceLoaderTest : public testing::Test {
}
virtual ~BufferedResourceLoaderTest() {
- ignore_result(frame_.release());
}
void Initialize(const char* url, int first_position, int last_position) {
@@ -483,4 +482,3 @@ TEST_F(BufferedResourceLoaderTest, AllowDefer_DeferredReadPastWindow) {
// TODO(hclam): add unit test for defer loading.
} // namespace webkit_glue
-
« no previous file with comments | « webkit/glue/media/buffered_data_source_unittest.cc ('k') | webkit/glue/media/simple_data_source_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698