Move PDF thumbnails files to //pdf/ui/

These files mostly contain utilities for supporting the PDF Viewer's thumbnail bar, thus qualifying them for inclusion in the new directory. Bug: 1184342 Change-Id: Ifcb36dc9e27c0fc24c0abdac4e2d19d780e0d7fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2753341 Auto-Submit: Daniel Hosseinian <[email protected]> Commit-Queue: Daniel Hosseinian <[email protected]> Commit-Queue: K. Moon <[email protected]> Reviewed-by: K. Moon <[email protected]> Cr-Commit-Position: refs/heads/master@{#862222}
AryanGitHub · Mar 12, 2021 · ccff942 · ccff942
1 parent b7b3b0d
commit ccff942
Show file tree

Hide file tree

Showing 9 changed files with 14 additions and 14 deletions.
diff --git a/pdf/BUILD.gn b/pdf/BUILD.gn
@@ -138,12 +138,12 @@ if (enable_pdf) {
       "preview_mode_client.h",
       "range_set.cc",
       "range_set.h",
-      "thumbnail.cc",
-      "thumbnail.h",
       "ui/file_name.cc",
       "ui/file_name.h",
       "ui/format_page_size.cc",
       "ui/format_page_size.h",
+      "ui/thumbnail.cc",
+      "ui/thumbnail.h",
       "url_loader_wrapper.h",
       "url_loader_wrapper_impl.cc",
       "url_loader_wrapper_impl.h",
@@ -364,9 +364,9 @@ if (enable_pdf) {
       "ppapi_migration/url_loader_unittest.cc",
       "range_set_unittest.cc",
       "test/run_all_unittests.cc",
-      "thumbnail_unittest.cc",
       "ui/file_name_unittest.cc",
       "ui/format_page_size_unittest.cc",
+      "ui/thumbnail_unittest.cc",
     ]
 
     configs += [

diff --git a/pdf/out_of_process_instance.cc b/pdf/out_of_process_instance.cc
@@ -43,8 +43,8 @@
 #include "pdf/ppapi_migration/input_event_conversions.h"
 #include "pdf/ppapi_migration/url_loader.h"
 #include "pdf/ppapi_migration/value_conversions.h"
-#include "pdf/thumbnail.h"
 #include "pdf/ui/format_page_size.h"
+#include "pdf/ui/thumbnail.h"
 #include "ppapi/c/dev/ppb_cursor_control_dev.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/c/private/ppb_pdf.h"

diff --git a/pdf/pdfium/pdfium_engine_unittest.cc b/pdf/pdfium/pdfium_engine_unittest.cc
@@ -22,7 +22,7 @@
 #include "pdf/ppapi_migration/input_event_conversions.h"
 #include "pdf/test/test_client.h"
 #include "pdf/test/test_document_loader.h"
-#include "pdf/thumbnail.h"
+#include "pdf/ui/thumbnail.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/geometry/point.h"

diff --git a/pdf/pdfium/pdfium_page.cc b/pdf/pdfium/pdfium_page.cc
@@ -27,7 +27,7 @@
 #include "pdf/pdfium/pdfium_engine.h"
 #include "pdf/pdfium/pdfium_unsupported_features.h"
 #include "pdf/ppapi_migration/geometry_conversions.h"
-#include "pdf/thumbnail.h"
+#include "pdf/ui/thumbnail.h"
 #include "ppapi/c/private/ppb_pdf.h"
 #include "printing/units.h"
 #include "third_party/pdfium/public/cpp/fpdf_scopers.h"

diff --git a/pdf/pdfium/pdfium_page_unittest.cc b/pdf/pdfium/pdfium_page_unittest.cc
@@ -21,7 +21,7 @@
 #include "pdf/pdfium/pdfium_test_base.h"
 #include "pdf/ppapi_migration/geometry_conversions.h"
 #include "pdf/test/test_client.h"
-#include "pdf/thumbnail.h"
+#include "pdf/ui/thumbnail.h"
 #include "ppapi/c/private/ppb_pdf.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/pdfium/public/fpdf_formfill.h"

diff --git a/pdf/ui/README.md b/pdf/ui/README.md
@@ -4,5 +4,5 @@ This directory contains utilities for supporting PDF Viewer features other than
 the main content area.
 
 TODO(crbug.com/1184342): Accordingly move more code into this directory. For
-instance, code that supports the document properties dialog and thumbnails
-belong in this directory.
+instance, code that supports the document properties dialog belong in this
+directory.
diff --git a/pdf/thumbnail.cc → pdf/ui/thumbnail.cc b/pdf/thumbnail.cc → pdf/ui/thumbnail.cc
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "pdf/thumbnail.h"
+#include "pdf/ui/thumbnail.h"
 
 #include <algorithm>
 #include <cmath>

diff --git a/pdf/thumbnail.h → pdf/ui/thumbnail.h b/pdf/thumbnail.h → pdf/ui/thumbnail.h
@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef PDF_THUMBNAIL_H_
-#define PDF_THUMBNAIL_H_
+#ifndef PDF_UI_THUMBNAIL_H_
+#define PDF_UI_THUMBNAIL_H_
 
 #include "third_party/skia/include/core/SkBitmap.h"
 
@@ -38,4 +38,4 @@ class Thumbnail final {
 
 }  // namespace chrome_pdf
 
-#endif  // PDF_THUMBNAIL_H_
+#endif  // PDF_UI_THUMBNAIL_H_
diff --git a/pdf/thumbnail_unittest.cc → pdf/ui/thumbnail_unittest.cc b/pdf/thumbnail_unittest.cc → pdf/ui/thumbnail_unittest.cc
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "pdf/thumbnail.h"
+#include "pdf/ui/thumbnail.h"
 
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/geometry/size.h"