Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed

Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.

BUG=594120
R=jochen@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/e432675850161570a8562f8c617da039f51f706d

[Tom Sepez, 2016-03-23 20:31:49]

Description was changed from

==========
Preserve m_pDynamicObjsMap until perIsolateData is destroyed

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

to

==========
Preserve m_pDynamicObjsMap until FXJS_perIsolateData is destroyed.

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

[Tom Sepez, 2016-03-23 20:32:00]

Jochen, for review.

[Tom Sepez, 2016-03-23 20:32:23]

Description was changed from

==========
Preserve m_pDynamicObjsMap until FXJS_perIsolateData is destroyed.

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

to

==========
Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.

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

[Tom Sepez, 2016-03-23 21:34:46]

https://codereview.chromium.org/1822423002/diff/40001/fpdfsdk/jsapi/fxjs_v8.cpp
File fpdfsdk/jsapi/fxjs_v8.cpp (right):

https://codereview.chromium.org/1822423002/diff/40001/fpdfsdk/jsapi/fxjs_v8.c...
fpdfsdk/jsapi/fxjs_v8.cpp:133: v8::Local<v8::Object> obj = value.Get(isolate);
Jochen: does this routine need to remove the disposed object from the map?

[jochen (gone - plz use gerrit), 2016-03-24 14:45:43]

lgtm

[Tom Sepez, 2016-03-24 16:13:22]

Description was changed from

==========
Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.

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

to

==========
Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.

BUG=594120
R=jochen@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/e432675850161570a8562f8c617da039f51f...
==========

[Tom Sepez, 2016-03-24 16:13:25]

Committed patchset #3 (id:40001) manually as
e432675850161570a8562f8c617da039f51f706d (presubmit successful).