Fix CPDFSDK_PageView cleanup.

Fix CPDFSDK_PageView cleanup.

In https://pdfium.googlesource.com/pdfium/+/461eeafe191068ac8c32f2717907fc6a22a667d2
we moved the map cleanup for the page view to happen before the page view was
destroyed and before we killed the annotation focus.

The map removal must happen before the pageview is destroyed, but must happen
after we've killed the annotation focus in order to not create duplicate page
views which point to the same page.

BUG=chromium:645122
Committed: https://pdfium.googlesource.com/pdfium/+/c2d0e29cd8fa24c9af0cc4f2a15f90096a5ca8e2

[dsinclair, 2016-09-08 20:12:56]

The CQ bit was checked by dsinclair@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-08 20:13:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dsinclair, 2016-09-08 20:14:02]

dsinclair@chromium.org changed reviewers:
+ thestig@chromium.org, tsepez@chromium.org, weili@chromium.org

[dsinclair, 2016-09-08 20:14:03]

PTAL.

The ownership between the page view's and the pages is .... hard.

[Tom Sepez, 2016-09-08 20:15:35]

Good luck. LGTM.

[commit-bot: I haz the power, 2016-09-08 20:20:30]

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

[commit-bot: I haz the power, 2016-09-08 20:20:31]

Dry run: Try jobs failed on following builders:
  mac on master.tryserver.client.pdfium (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.pdfium/builders/mac/builds/1984)

[dsinclair, 2016-09-08 20:22:22]

The CQ bit was checked by dsinclair@chromium.org

[commit-bot: I haz the power, 2016-09-08 20:22:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-08 20:25:40]

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

[commit-bot: I haz the power, 2016-09-08 20:25:41]

Try jobs failed on following builders:
  linux on master.tryserver.client.pdfium (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.pdfium/builders/linux/builds/1994)
  mac_no_v8 on master.tryserver.client.pdfium (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.pdfium/builders/mac_no_v8/build...)

[dsinclair, 2016-09-08 20:41:23]

The CQ bit was checked by dsinclair@chromium.org

[commit-bot: I haz the power, 2016-09-08 20:41:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-08 21:03:54]

Description was changed from

==========
Fix CPDFSDK_PageView cleanup.

In
https://pdfium.googlesource.com/pdfium/+/461eeafe191068ac8c32f2717907fc6a22a6...
we moved the map cleanup for the page view to happen before the page view was
destroyed and before we killed the annotation focus.

The map removal must happen before the pageview is destroyed, but must happen
after we've killed the annotation focus in order to not create duplicate page
views which point to the same page.

BUG=chromium:645122
==========

to

==========
Fix CPDFSDK_PageView cleanup.

In
https://pdfium.googlesource.com/pdfium/+/461eeafe191068ac8c32f2717907fc6a22a6...
we moved the map cleanup for the page view to happen before the page view was
destroyed and before we killed the annotation focus.

The map removal must happen before the pageview is destroyed, but must happen
after we've killed the annotation focus in order to not create duplicate page
views which point to the same page.

BUG=chromium:645122

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

[commit-bot: I haz the power, 2016-09-08 21:03:54]

Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/c2d0e29cd8fa24c9af0cc4f2a15f90096a5c...