summaryrefslogtreecommitdiff
path: root/fpdfsdk/src
diff options
context:
space:
mode:
authorRaymes Khoury <raymes@chromium.org>2015-10-01 15:12:08 +1000
committerRaymes Khoury <raymes@chromium.org>2015-10-01 15:12:08 +1000
commit953f5c5f3858ce20ea5f28e0045c201148fba036 (patch)
treec3711eaa8e32a871b86681ffd8a3b386f2bb554d /fpdfsdk/src
parent158e335717efba9dce3aa6f6d1e31ed884e1f59e (diff)
downloadpdfium-953f5c5f3858ce20ea5f28e0045c201148fba036.tar.xz
Revert "Leak per-isolate data in pdfium"
This reverts commit 3e144b8c23d7c52ed36329e87f0cb01f38ec1ed7. This may be causing the failures seen in crbug.com/537799. BUG=537799 Review URL: https://codereview.chromium.org/1382433003 .
Diffstat (limited to 'fpdfsdk/src')
-rw-r--r--fpdfsdk/src/javascript/JS_Runtime.cpp8
1 files changed, 1 insertions, 7 deletions
diff --git a/fpdfsdk/src/javascript/JS_Runtime.cpp b/fpdfsdk/src/javascript/JS_Runtime.cpp
index 7ce8a5343b..1e1486df5c 100644
--- a/fpdfsdk/src/javascript/JS_Runtime.cpp
+++ b/fpdfsdk/src/javascript/JS_Runtime.cpp
@@ -63,13 +63,7 @@ CJS_Runtime::~CJS_Runtime() {
delete m_ContextArray.GetAt(i);
m_ContextArray.RemoveAll();
-
- // TODO(raymes): Currently we're freeing per-isolate data everytime a
- // document is destroyed even though it may be in use by other documents. For
- // now we leak the per-isolate data (when m_isolateManaged is false) until
- // crbug.com/531339 is fixed.
- if (m_isolateManaged)
- FXJS_ReleaseRuntime(GetIsolate(), m_context);
+ FXJS_ReleaseRuntime(GetIsolate(), m_context);
m_pApp = NULL;
m_pDocument = NULL;