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

Unified Diff: Source/core/html/imports/HTMLImportLoader.h

Issue 1032293002: Oilpan: dispose an HTMLImportsController on document detach/removal. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: No need to call dispose() when destructing HTMLImportsController Created 5 years, 9 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 | « Source/core/html/imports/HTMLImportChild.cpp ('k') | Source/core/html/imports/HTMLImportLoader.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/imports/HTMLImportLoader.h
diff --git a/Source/core/html/imports/HTMLImportLoader.h b/Source/core/html/imports/HTMLImportLoader.h
index bd342b361298aa49f98934242121b3b14fc6267c..b0ac3700cf5cbbe43b499886d9ed9d0ea4d34d18 100644
--- a/Source/core/html/imports/HTMLImportLoader.h
+++ b/Source/core/html/imports/HTMLImportLoader.h
@@ -71,12 +71,12 @@ public:
}
virtual ~HTMLImportLoader();
+ void dispose();
Document* document() const { return m_document.get(); }
void addImport(HTMLImportChild*);
-#if !ENABLE(OILPAN)
void removeImport(HTMLImportChild*);
-#endif
+
void moveToFirst(HTMLImportChild*);
HTMLImportChild* firstImport() const { return m_imports[0]; }
bool isFirstImport(const HTMLImportChild* child) const { return m_imports.size() ? firstImport() == child : false; }
@@ -85,9 +85,6 @@ public:
bool hasError() const { return m_state == StateError; }
bool shouldBlockScriptExecution() const;
-#if !ENABLE(OILPAN)
- void importDestroyed();
-#endif
void startLoading(const ResourcePtr<RawResource>&);
// Tells the loader that all of the import's stylesheets finished
« no previous file with comments | « Source/core/html/imports/HTMLImportChild.cpp ('k') | Source/core/html/imports/HTMLImportLoader.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698