Improve linearized pdf load/show time.

pdf/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/features.gni")
 import("//third_party/pdfium/pdfium.gni")
+import("//testing/test.gni")

[Lei Zhang, 2016/10/21 09:33:08]

nit: alphabetical order

[snake, 2016/10/21 15:13:14]

Done.

 
 assert(enable_pdf)
 
 pdf_engine = 0  # 0 PDFium
 
 static_library("pdf") {
   deps = [
     "//base",
     "//gin",
     "//net",
     "//ppapi/cpp:objects",
     "//ppapi/cpp/private:internal_module",
     "//ui/base",
+    "//ui/gfx/range",
   ]
 
   sources = [
-    "chunk_stream.cc",
     "chunk_stream.h",
     "document_loader.cc",
     "document_loader.h",
     "draw_utils.cc",
     "draw_utils.h",
     "out_of_process_instance.cc",
     "out_of_process_instance.h",
     "paint_aggregator.cc",
     "paint_aggregator.h",
     "paint_manager.cc",
     "paint_manager.h",
     "pdf.cc",
     "pdf.h",
     "pdf_engine.h",
     "preview_mode_client.cc",
     "preview_mode_client.h",
+    "range_set.cc",
+    "range_set.h",
+    "timer.cc",
+    "timer.h",
+    "url_loader_wrapper.h",
+    "url_loader_wrapper_impl.cc",
+    "url_loader_wrapper_impl.h",
   ]
 
   # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
   configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
 
   if (pdf_engine == 0) {
     deps += [
       "//pdf/pdfium/fuzzers",
       "//third_party/pdfium",
     ]
@@ skipping 13 matching lines @@
       "pdfium/pdfium_range.cc",
       "pdfium/pdfium_range.h",
     ]
   }
 
   defines = [ "PDFIUM_PRINT_TEXT_WITH_GDI" ]
   if (pdf_enable_xfa) {
     defines += [ "PDF_ENABLE_XFA" ]
   }
 }
+
+test("pdf_unittests") {
+  sources = [
+    "chunk_stream_unittest.cc",
+    "document_loader_unittest.cc",
+    "range_set_unittest.cc",
+    "run_all_unittests.cc",
+  ]
+
+  deps = [
+    ":pdf",
+    "//base",
+    "//base/test:test_support",
+    "//net",
+    "//ppapi/c",
+    "//ppapi/cpp",
+    "//printing",
+    "//testing/gmock",
+    "//testing/gtest",
+    "//third_party/pdfium",
+    "//ui/gfx",

[Lei Zhang, 2016/10/21 09:33:08]

only "//ui/gfx/range" is needed?

[snake, 2016/10/21 15:13:14]

Done.

+    "//ui/gfx/geometry",
+  ]
+}