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

Unified Diff: chrome/browser/extensions/extension_save_page_api.h

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: Clean-up 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: chrome/browser/extensions/extension_save_page_api.h
diff --git a/chrome/browser/extensions/extension_save_page_api.h b/chrome/browser/extensions/extension_save_page_api.h
index 4afdf62f2fcbb8c5c0e1655ed80634d6d134ee41..a37dd714f85d8229d0e8b24a8a29dc8d5762b697 100644
--- a/chrome/browser/extensions/extension_save_page_api.h
+++ b/chrome/browser/extensions/extension_save_page_api.h
@@ -7,10 +7,12 @@
#include <string>
+#include "base/memory/ref_counted.h"
#include "chrome/browser/extensions/extension_function.h"
#include "content/browser/tab_contents/tab_contents_observer.h"
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
+#include "webkit/blob/deletable_file_reference.h"
class SavePageAsMHTMLFunction : public AsyncExtensionFunction,
public content::NotificationObserver {
@@ -24,6 +26,8 @@ class SavePageAsMHTMLFunction : public AsyncExtensionFunction,
virtual void Observe(int type,
const content::NotificationSource& source,
const content::NotificationDetails& details) OVERRIDE;
+ virtual bool OnMessageReceivedFromRenderView(
+ const IPC::Message& message) OVERRIDE;
// Called on the file thread.
void CreateTemporaryFile();
@@ -33,12 +37,19 @@ class SavePageAsMHTMLFunction : public AsyncExtensionFunction,
void ReturnFailure(const std::string& error);
void ReturnSuccess(int64 file_size);
+ // Returns the TabContents we are associated with, NULL if it's been closed.
+ TabContents* GetTabContents();
+
int tab_id_;
// The path to the temporary file containing the MHTML data.
FilePath mhtml_path_;
+ // The file containing the MHTML.
+ scoped_refptr<webkit_blob::DeletableFileReference> mhtml_file_;
+
content::NotificationRegistrar registrar_;
+
DECLARE_EXTENSION_FUNCTION_NAME("experimental.savePage.saveAsMHTML")
};

Powered by Google App Engine
This is Rietveld 408576698