Rename ScopedLoadCounter --> ScopedUnloadPreventer.

Rename PdfiumPage::ScopedLoadCounter to more appropriate
name PdfiumPage::ScopedUnloadPreventer.

Change-Id: Idd8b7d2b57dbbeac0826ade010b29420956c9062
Reviewed-on: https://chromium-review.googlesource.com/1100834
Commit-Queue: Art Snake <art-snake@yandex-team.ru>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#574894}

pdf/pdfium/pdfium_page.cc

@@ -101,12 +101,12 @@ PDFiumPage::PDFiumPage(PDFiumEngine* engine,
 PDFiumPage::PDFiumPage(PDFiumPage&& that) = default;
 
 PDFiumPage::~PDFiumPage() {
-  DCHECK_EQ(0, loading_count_);
+  DCHECK_EQ(0, preventing_unload_count_);
 }
 
 void PDFiumPage::Unload() {
   // Do not unload while in the middle of a load.
-  if (loading_count_)
+  if (preventing_unload_count_)
     return;
 
   text_page_.reset();
@@ -125,7 +125,7 @@ FPDF_PAGE PDFiumPage::GetPage() {
   if (!available_)
     return nullptr;
   if (!page_) {
-    ScopedLoadCounter scoped_load(this);
+    ScopedUnloadPreventer scoped_unload_preventer(this);
     page_.reset(FPDF_LoadPage(engine_->doc(), index_));
     if (page_ && engine_->form()) {
       FORM_OnAfterLoadPage(page(), engine_->form());
@@ -140,7 +140,7 @@ FPDF_PAGE PDFiumPage::GetPrintPage() {
   if (!available_)
     return nullptr;
   if (!page_) {
-    ScopedLoadCounter scoped_load(this);
+    ScopedUnloadPreventer scoped_unload_preventer(this);
     page_.reset(FPDF_LoadPage(engine_->doc(), index_));
   }
   return page();
@@ -148,7 +148,7 @@ FPDF_PAGE PDFiumPage::GetPrintPage() {
 
 void PDFiumPage::ClosePrintPage() {
   // Do not close |page_| while in the middle of a load.
-  if (loading_count_)
+  if (preventing_unload_count_)
     return;
 
   page_.reset();
@@ -158,7 +158,7 @@ FPDF_TEXTPAGE PDFiumPage::GetTextPage() {
   if (!available_)
     return nullptr;
   if (!text_page_) {
-    ScopedLoadCounter scoped_load(this);
+    ScopedUnloadPreventer scoped_unload_preventer(this);
     text_page_.reset(FPDFText_LoadPage(GetPage()));
   }
   return text_page();
@@ -607,13 +607,13 @@ const PDFEngine::PageFeatures* PDFiumPage::GetPageFeatures() {
   return &page_features_;
 }
 
-PDFiumPage::ScopedLoadCounter::ScopedLoadCounter(PDFiumPage* page)
+PDFiumPage::ScopedUnloadPreventer::ScopedUnloadPreventer(PDFiumPage* page)
     : page_(page) {
-  page_->loading_count_++;
+  page_->preventing_unload_count_++;
 }
 
-PDFiumPage::ScopedLoadCounter::~ScopedLoadCounter() {
-  page_->loading_count_--;
+PDFiumPage::ScopedUnloadPreventer::~ScopedUnloadPreventer() {
+  page_->preventing_unload_count_--;
 }
 
 PDFiumPage::Link::Link() = default;

pdf/pdfium/pdfium_page.h

@@ -146,10 +146,10 @@ class PDFiumPage {
   // NONSELECTABLE_AREA if detection failed.
   Area GetURITarget(FPDF_ACTION uri_action, LinkTarget* target) const;
 
-  class ScopedLoadCounter {
+  class ScopedUnloadPreventer {
    public:
-    explicit ScopedLoadCounter(PDFiumPage* page);
-    ~ScopedLoadCounter();
+    explicit ScopedUnloadPreventer(PDFiumPage* page);
+    ~ScopedUnloadPreventer();
 
    private:
     PDFiumPage* const page_;
@@ -169,7 +169,7 @@ class PDFiumPage {
   ScopedFPDFPage page_;
   ScopedFPDFTextPage text_page_;
   int index_;
-  int loading_count_ = 0;
+  int preventing_unload_count_ = 0;
   pp::Rect rect_;
   bool calculated_links_ = false;
   std::vector<Link> links_;