CFXJS_ObjDefinition's lifetime must match isolate.

CFXJS_ObjDefinition's lifetime must match isolate.

Introduced at https://codereview.chromium.org/1424933013/

The v8 objects are 1:1 with the context being destroyed in
FXJS_ReleaseRuntime, but the CFXJS_ObjDefinitions are still
1:1 with the isolate, so they can't be destroyed until the
refcounts hit zero.

BUG=561184
R=jochen@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/628f7053cc646c58c61add247c75bc65ac135327

[Tom Sepez, 2015-11-25 18:26:36]

Description was changed from

==========
CFXJS_ObjDefinition's lifetime must match isolate.

Introduced at https://codereview.chromium.org/1424933013/

The v8 objects are 1:1 with the context being destroyed in
FXJS_ReleaseRuntime, but the CFXJS_ObjDefinitions are still
1:1 with the isolate, so they can't be destroyed until the
refcounts hit zero.

R=jochen@chromium.org
==========

to

==========
CFXJS_ObjDefinition's lifetime must match isolate.

Introduced at https://codereview.chromium.org/1424933013/

The v8 objects are 1:1 with the context being destroyed in
FXJS_ReleaseRuntime, but the CFXJS_ObjDefinitions are still
1:1 with the isolate, so they can't be destroyed until the
refcounts hit zero.

BUG=561184
R=jochen@chromium.org
==========

[jochen (gone - plz use gerrit), 2015-11-25 18:27:47]

lgtm

[Tom Sepez, 2015-11-25 18:28:37]

Jochen, please review.

[Tom Sepez, 2015-11-25 18:30:03]

Description was changed from

==========
CFXJS_ObjDefinition's lifetime must match isolate.

Introduced at https://codereview.chromium.org/1424933013/

The v8 objects are 1:1 with the context being destroyed in
FXJS_ReleaseRuntime, but the CFXJS_ObjDefinitions are still
1:1 with the isolate, so they can't be destroyed until the
refcounts hit zero.

BUG=561184
R=jochen@chromium.org
==========

to

==========
CFXJS_ObjDefinition's lifetime must match isolate.

Introduced at https://codereview.chromium.org/1424933013/

The v8 objects are 1:1 with the context being destroyed in
FXJS_ReleaseRuntime, but the CFXJS_ObjDefinitions are still
1:1 with the isolate, so they can't be destroyed until the
refcounts hit zero.

BUG=561184
R=jochen@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/628f7053cc646c58c61add247c75bc65ac13...
==========

[Tom Sepez, 2015-11-25 18:30:04]

Committed patchset #1 (id:1) manually as
628f7053cc646c58c61add247c75bc65ac135327 (presubmit successful).