Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(649)

Side by Side Diff: chrome/browser/pdf/pdf_extension_test.cc

Issue 1521913002: PDF: Delete non-material viewer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « chrome/browser/about_flags.cc ('k') | chrome/browser/pdf/pdf_extension_util.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <vector> 5 #include <vector>
6 6
7 #include "base/base_paths.h" 7 #include "base/base_paths.h"
8 #include "base/files/file_enumerator.h" 8 #include "base/files/file_enumerator.h"
9 #include "base/files/file_util.h" 9 #include "base/files/file_util.h"
10 #include "base/hash.h" 10 #include "base/hash.h"
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
60 // Return false so that we iterate through all the guests and verify there is 60 // Return false so that we iterate through all the guests and verify there is
61 // only one. 61 // only one.
62 return false; 62 return false;
63 } 63 }
64 64
65 class PDFExtensionTest : public ExtensionApiTest, 65 class PDFExtensionTest : public ExtensionApiTest,
66 public testing::WithParamInterface<int> { 66 public testing::WithParamInterface<int> {
67 public: 67 public:
68 ~PDFExtensionTest() override {} 68 ~PDFExtensionTest() override {}
69 69
70 void SetUpCommandLine(base::CommandLine* command_line) override {
71 command_line->AppendSwitch(switches::kDisablePdfMaterialUI);
72 }
73
74 void SetUpOnMainThread() override { 70 void SetUpOnMainThread() override {
75 ExtensionApiTest::SetUpOnMainThread(); 71 ExtensionApiTest::SetUpOnMainThread();
76 ASSERT_TRUE(embedded_test_server()->Start()); 72 ASSERT_TRUE(embedded_test_server()->Start());
77 } 73 }
78 74
79 void TearDownOnMainThread() override { 75 void TearDownOnMainThread() override {
80 ASSERT_TRUE(embedded_test_server()->ShutdownAndWaitUntilComplete()); 76 ASSERT_TRUE(embedded_test_server()->ShutdownAndWaitUntilComplete());
81 ExtensionApiTest::TearDownOnMainThread(); 77 ExtensionApiTest::TearDownOnMainThread();
82 } 78 }
83 79
(...skipping 227 matching lines...) Expand 10 before | Expand all | Expand 10 after
311 ASSERT_EQ(1u, downloads.size()); 307 ASSERT_EQ(1u, downloads.size());
312 downloads[0]->Cancel(false); 308 downloads[0]->Cancel(false);
313 } 309 }
314 310
315 // We break PDFTest.Load up into kNumberLoadTestParts. 311 // We break PDFTest.Load up into kNumberLoadTestParts.
316 INSTANTIATE_TEST_CASE_P(PDFTestFiles, 312 INSTANTIATE_TEST_CASE_P(PDFTestFiles,
317 PDFExtensionTest, 313 PDFExtensionTest,
318 testing::Range(0, kNumberLoadTestParts)); 314 testing::Range(0, kNumberLoadTestParts));
319 315
320 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Basic) { 316 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Basic) {
321 RunTestsInFile("basic_test.js", "test.pdf"); 317 RunTestsInFile("basic_test_material.js", "test.pdf");
322 } 318 }
323 319
324 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, BasicPlugin) { 320 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, BasicPlugin) {
325 RunTestsInFile("basic_plugin_test.js", "test.pdf"); 321 RunTestsInFile("basic_plugin_test.js", "test.pdf");
326 } 322 }
327 323
328 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Viewport) { 324 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Viewport) {
329 RunTestsInFile("viewport_test.js", "test.pdf"); 325 RunTestsInFile("viewport_test.js", "test.pdf");
330 } 326 }
331 327
332 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Bookmark) { 328 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Bookmark) {
333 RunTestsInFile("bookmarks_test.js", "test-bookmarks.pdf"); 329 RunTestsInFile("bookmarks_test.js", "test-bookmarks.pdf");
334 } 330 }
335 331
336 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Navigator) { 332 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Navigator) {
337 RunTestsInFile("navigator_test.js", "test.pdf"); 333 RunTestsInFile("navigator_test.js", "test.pdf");
338 } 334 }
339 335
340 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, ParamsParser) { 336 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, ParamsParser) {
341 RunTestsInFile("params_parser_test.js", "test.pdf"); 337 RunTestsInFile("params_parser_test.js", "test.pdf");
342 } 338 }
343 339
344 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, ZoomManager) { 340 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, ZoomManager) {
345 RunTestsInFile("zoom_manager_test.js", "test.pdf"); 341 RunTestsInFile("zoom_manager_test.js", "test.pdf");
346 } 342 }
347 343
344 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Elements) {
345 // Although this test file does not require a PDF to be loaded, loading the
346 // elements without loading a PDF is difficult.
347 RunTestsInFile("material_elements_test.js", "test.pdf");
348 }
349
350 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, ToolbarManager) {
351 RunTestsInFile("toolbar_manager_test.js", "test.pdf");
352 }
353
348 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Title) { 354 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, Title) {
349 RunTestsInFile("title_test.js", "test-title.pdf"); 355 RunTestsInFile("title_test.js", "test-title.pdf");
350 } 356 }
351 357
352 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, WhitespaceTitle) { 358 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, WhitespaceTitle) {
353 RunTestsInFile("whitespace_title_test.js", "test-whitespace-title.pdf"); 359 RunTestsInFile("whitespace_title_test.js", "test-whitespace-title.pdf");
354 } 360 }
355 361
356 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, PageChange) { 362 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, PageChange) {
357 RunTestsInFile("page_change_test.js", "test-bookmarks.pdf"); 363 RunTestsInFile("page_change_test.js", "test-bookmarks.pdf");
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
495 #endif 501 #endif
496 ASSERT_TRUE( 502 ASSERT_TRUE(
497 content::ExecuteScript(guest_contents, "viewer.viewport.zoomIn();")); 503 content::ExecuteScript(guest_contents, "viewer.viewport.zoomIn();"));
498 504
499 watcher.Wait(); 505 watcher.Wait();
500 #if defined(TOOLKIT_VIEWS) && !defined(OS_MACOSX) 506 #if defined(TOOLKIT_VIEWS) && !defined(OS_MACOSX)
501 EXPECT_EQ(nullptr, ZoomBubbleView::GetZoomBubble()); 507 EXPECT_EQ(nullptr, ZoomBubbleView::GetZoomBubble());
502 #endif 508 #endif
503 } 509 }
504 510
505 class MaterialPDFExtensionTest : public PDFExtensionTest {
506 void SetUpCommandLine(base::CommandLine* command_line) override {
507 command_line->AppendSwitch(switches::kEnablePdfMaterialUI);
508 }
509 };
510
511 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, Basic) {
512 RunTestsInFile("basic_test_material.js", "test.pdf");
513 }
514
515 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, BasicPlugin) {
516 RunTestsInFile("basic_plugin_test.js", "test.pdf");
517 }
518
519 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, Viewport) {
520 RunTestsInFile("viewport_test.js", "test.pdf");
521 }
522
523 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, Bookmark) {
524 RunTestsInFile("bookmarks_test.js", "test-bookmarks.pdf");
525 }
526
527 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, Navigator) {
528 RunTestsInFile("navigator_test.js", "test.pdf");
529 }
530
531 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, ParamsParser) {
532 RunTestsInFile("params_parser_test.js", "test.pdf");
533 }
534
535 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, ZoomManager) {
536 RunTestsInFile("zoom_manager_test.js", "test.pdf");
537 }
538
539 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, Elements) {
540 // Although this test file does not require a PDF to be loaded, loading the
541 // elements without loading a PDF is difficult.
542 RunTestsInFile("material_elements_test.js", "test.pdf");
543 }
544
545 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, ToolbarManager) {
546 RunTestsInFile("toolbar_manager_test.js", "test.pdf");
547 }
548
549 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, Title) {
550 RunTestsInFile("title_test.js", "test-title.pdf");
551 }
552
553 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, WhitespaceTitle) {
554 RunTestsInFile("whitespace_title_test.js", "test-whitespace-title.pdf");
555 }
556
557 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, PageChange) {
558 RunTestsInFile("page_change_test.js", "test-bookmarks.pdf");
559 }
OLDNEW
« no previous file with comments | « chrome/browser/about_flags.cc ('k') | chrome/browser/pdf/pdf_extension_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698