Fix memory leaking on ClosePage.

Fix memory leaking on ClosePage.

BUG=79367, 48791
The fonts was not cleared after unloading pages.

Test pdf: http://www.nasa.gov/pdf/750614main_NASA_FY_2014_Budget_Estimates-508.pdf
 For this file, we have ~5 fonts per page, which equal ~1 Mb per page.
 In this PDF we have 670 pages, as result after slow scrolling(reading) full document we have ~600 Mb fonts data in memory.

Memory usage of PDF Plugin:
 before this CL: ~660 Mb
 after this CL: ~100 Mb

This is last part of reverted CL:
 Original CL: https://codereview.chromium.org/2158023002
 Revert reason: BUG=647612
Fix bug CL: https://codereview.chromium.org/2350193003
 Previous CL: https://codereview.chromium.org/2350243002

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

[snake, 2016-09-20 15:17:45]

Description was changed from

==========
Fix memory leaking on ClosePage.

BUG=79367, 48791

The fonts was not cleared after unloading pages.

Test pdf:
http://www.nasa.gov/pdf/750614main_NASA_FY_2014_Budget_Estimates-508.pdf
 For this file, we have ~5 fonts per page, which equal ~1 Mb per page.
 In this PDF we have 670 pages, as result after slow scrolling(reading) full
document we have ~600 Mb fonts data in memory.

Memory usage of PDF Plugin:
 before this CL: ~660 Mb
 after this CL: ~100 Mb

This is last part of reverted CL:
 Original CL: https://codereview.chromium.org/2158023002
 Revert reason: BUG=647612
 Fix bug CL: https://codereview.chromium.org/2350193003
 Previous CL: https://codereview.chromium.org/2350243002
==========

to

==========
Fix memory leaking on ClosePage.

BUG=79367, 48791

The fonts was not cleared after unloading pages.

Test pdf:
http://www.nasa.gov/pdf/750614main_NASA_FY_2014_Budget_Estimates-508.pdf
 For this file, we have ~5 fonts per page, which equal ~1 Mb per page.
 In this PDF we have 670 pages, as result after slow scrolling(reading) full
document we have ~600 Mb fonts data in memory.

Memory usage of PDF Plugin:
 before this CL: ~660 Mb
 after this CL: ~100 Mb

This is last part of reverted CL:
 Original CL: https://codereview.chromium.org/2158023002
 Revert reason: BUG=647612
 Fix bug CL: https://codereview.chromium.org/2350193003
 Previous CL: https://codereview.chromium.org/2350243002
==========

[snake, 2016-09-20 15:17:45]

art-snake@yandex-team.ru changed reviewers:
+ dsinclair@chromium.org, npm@chromium.org

[dsinclair, 2016-09-20 18:28:18]

https://codereview.chromium.org/2351193002/diff/1/core/fpdfapi/fpdf_page/fpdf...
File core/fpdfapi/fpdf_page/fpdf_page_doc.cpp (right):

https://codereview.chromium.org/2351193002/diff/1/core/fpdfapi/fpdf_page/fpdf...
core/fpdfapi/fpdf_page/fpdf_page_doc.cpp:481: if (profile->use_count() == 1) {
Can we make this 

if (profile->use_count() > 1)
  continue;

so it's similar to the rest of them?

[snake, 2016-09-20 18:45:44]

https://codereview.chromium.org/2351193002/diff/1/core/fpdfapi/fpdf_page/fpdf...
File core/fpdfapi/fpdf_page/fpdf_page_doc.cpp (right):

https://codereview.chromium.org/2351193002/diff/1/core/fpdfapi/fpdf_page/fpdf...
core/fpdfapi/fpdf_page/fpdf_page_doc.cpp:481: if (profile->use_count() == 1) {
On 2016/09/20 18:28:18, dsinclair wrote:
> Can we make this 
> 
> if (profile->use_count() > 1)
>   continue;
> 
> so it's similar to the rest of them?

Done.

[dsinclair, 2016-09-20 18:46:18]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2016-09-20 18:46:18]

lgtm

[commit-bot: I haz the power, 2016-09-20 18:46:28]

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-20 18:59:04]

Description was changed from

==========
Fix memory leaking on ClosePage.

BUG=79367, 48791

The fonts was not cleared after unloading pages.

Test pdf:
http://www.nasa.gov/pdf/750614main_NASA_FY_2014_Budget_Estimates-508.pdf
 For this file, we have ~5 fonts per page, which equal ~1 Mb per page.
 In this PDF we have 670 pages, as result after slow scrolling(reading) full
document we have ~600 Mb fonts data in memory.

Memory usage of PDF Plugin:
 before this CL: ~660 Mb
 after this CL: ~100 Mb

This is last part of reverted CL:
 Original CL: https://codereview.chromium.org/2158023002
 Revert reason: BUG=647612
 Fix bug CL: https://codereview.chromium.org/2350193003
 Previous CL: https://codereview.chromium.org/2350243002
==========

to

==========
Fix memory leaking on ClosePage.

BUG=79367, 48791

The fonts was not cleared after unloading pages.

Test pdf:
http://www.nasa.gov/pdf/750614main_NASA_FY_2014_Budget_Estimates-508.pdf
 For this file, we have ~5 fonts per page, which equal ~1 Mb per page.
 In this PDF we have 670 pages, as result after slow scrolling(reading) full
document we have ~600 Mb fonts data in memory.

Memory usage of PDF Plugin:
 before this CL: ~660 Mb
 after this CL: ~100 Mb

This is last part of reverted CL:
 Original CL: https://codereview.chromium.org/2158023002
 Revert reason: BUG=647612
 Fix bug CL: https://codereview.chromium.org/2350193003
 Previous CL: https://codereview.chromium.org/2350243002

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

[commit-bot: I haz the power, 2016-09-20 18:59:05]

Committed patchset #2 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/a6df44456956914e86b9d9af722b30628c45...