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

Unified Diff: content/renderer/mhtml_generator.cc

Issue 8530003: Delete the temporary file when generating MHTML with the extension API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: One more sync Created 9 years, 1 month 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: content/renderer/mhtml_generator.cc
diff --git a/content/renderer/mhtml_generator.cc b/content/renderer/mhtml_generator.cc
index 42aac729c34842eae480275b0a2eb833e8fd2e1d..0102b8fe3986a0fda6069bcb3b060fa5c696822a 100644
--- a/content/renderer/mhtml_generator.cc
+++ b/content/renderer/mhtml_generator.cc
@@ -34,6 +34,7 @@ void MHTMLGenerator::OnSavePageAsMHTML(
IPC::PlatformFileForTransitToPlatformFile(file_for_transit);
file_ = file;
int64 size = GenerateMHTML();
+ base::ClosePlatformFile(file);
NotifyBrowser(job_id, size);
}

Powered by Google App Engine
This is Rietveld 408576698