Re-land "Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.""

Re-land "Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.""

Fix the destruction order.

This reverts commit 129a60ec2d4d98ed953beaca629c99a52fb7e9b0.
BUG=594120
TBR=jochen@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/9967cc5861fbff894eed8fca40e1e5ed524b04c6

[Tom Sepez, 2016-03-24 17:31:54]

Description was changed from

==========
Re-land "Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.""

This reverts commit 129a60ec2d4d98ed953beaca629c99a52fb7e9b0.
BUG=594120
==========

to

==========
Re-land "Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.""

Fix the destruction order.

This reverts commit 129a60ec2d4d98ed953beaca629c99a52fb7e9b0.
BUG=594120
==========

[Tom Sepez, 2016-03-24 17:31:54]

tsepez@chromium.org changed reviewers:
+ jochen@chromium.org

[Tom Sepez, 2016-03-24 17:33:02]

Jochen, try #2. Please review. Thanks.

[Tom Sepez, 2016-03-24 18:37:47]

On 2016/03/24 17:33:02, Tom Sepez wrote:
> Jochen, try #2. Please review. Thanks.

Moving to TBR, its a small tweak on the last LGTM'd one.

[Tom Sepez, 2016-03-24 18:38:02]

Description was changed from

==========
Re-land "Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.""

Fix the destruction order.

This reverts commit 129a60ec2d4d98ed953beaca629c99a52fb7e9b0.
BUG=594120
==========

to

==========
Re-land "Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.""

Fix the destruction order.

This reverts commit 129a60ec2d4d98ed953beaca629c99a52fb7e9b0.
BUG=594120
TBR=jochen@chromium.org
==========

[Tom Sepez, 2016-03-24 18:38:37]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2016-03-24 18:38:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1831723004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1831723004/20001

[commit-bot: I haz the power, 2016-03-24 18:39:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-24 18:39:05]

Failed to commit the patch.

[Tom Sepez, 2016-03-24 18:45:40]

Description was changed from

==========
Re-land "Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.""

Fix the destruction order.

This reverts commit 129a60ec2d4d98ed953beaca629c99a52fb7e9b0.
BUG=594120
TBR=jochen@chromium.org
==========

to

==========
Re-land "Preserve m_pDynamicObjsMap until FXJS_PerIsolateData is destroyed.""

Fix the destruction order.

This reverts commit 129a60ec2d4d98ed953beaca629c99a52fb7e9b0.
BUG=594120
TBR=jochen@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/9967cc5861fbff894eed8fca40e1e5ed524b...
==========

[Tom Sepez, 2016-03-24 18:45:42]

Committed patchset #2 (id:20001) manually as
9967cc5861fbff894eed8fca40e1e5ed524b04c6 (presubmit successful).

[Tom Sepez, 2016-04-05 18:37:03]

Seems reasonable. LGTM.