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

Side by Side Diff: headless/lib/headless_devtools_client_browsertest.cc

Issue 2978473002: Disable headless TargetDomain tests on Windows. (Closed)
Patch Set: add disabling macro Created 3 years, 5 months 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 | « no previous file | headless/test/headless_browser_test.h » ('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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 <memory> 5 #include <memory>
6 6
7 #include "base/base_paths.h" 7 #include "base/base_paths.h"
8 #include "base/files/file_util.h" 8 #include "base/files/file_util.h"
9 #include "base/json/json_reader.h" 9 #include "base/json/json_reader.h"
10 #include "base/json/json_writer.h" 10 #include "base/json/json_writer.h"
11 #include "base/message_loop/message_loop.h" 11 #include "base/message_loop/message_loop.h"
12 #include "base/path_service.h" 12 #include "base/path_service.h"
13 #include "base/run_loop.h" 13 #include "base/run_loop.h"
14 #include "base/strings/string_util.h" 14 #include "base/strings/string_util.h"
15 #include "base/threading/thread_restrictions.h" 15 #include "base/threading/thread_restrictions.h"
16 #include "build/build_config.h"
16 #include "content/public/browser/render_widget_host_view.h" 17 #include "content/public/browser/render_widget_host_view.h"
17 #include "content/public/browser/web_contents.h" 18 #include "content/public/browser/web_contents.h"
18 #include "content/public/common/url_constants.h" 19 #include "content/public/common/url_constants.h"
19 #include "content/public/test/browser_test.h" 20 #include "content/public/test/browser_test.h"
20 #include "headless/lib/browser/headless_web_contents_impl.h" 21 #include "headless/lib/browser/headless_web_contents_impl.h"
21 #include "headless/public/devtools/domains/browser.h" 22 #include "headless/public/devtools/domains/browser.h"
22 #include "headless/public/devtools/domains/dom.h" 23 #include "headless/public/devtools/domains/dom.h"
23 #include "headless/public/devtools/domains/dom_snapshot.h" 24 #include "headless/public/devtools/domains/dom_snapshot.h"
24 #include "headless/public/devtools/domains/emulation.h" 25 #include "headless/public/devtools/domains/emulation.h"
25 #include "headless/public/devtools/domains/inspector.h" 26 #include "headless/public/devtools/domains/inspector.h"
(...skipping 425 matching lines...) Expand 10 before | Expand all | Expand 10 after
451 base::Unretained(this))); 452 base::Unretained(this)));
452 } 453 }
453 454
454 void OnCloseTargetResult(std::unique_ptr<target::CloseTargetResult> result) { 455 void OnCloseTargetResult(std::unique_ptr<target::CloseTargetResult> result) {
455 EXPECT_TRUE(result->GetSuccess()); 456 EXPECT_TRUE(result->GetSuccess());
456 EXPECT_EQ(1u, GetAllWebContents(browser()).size()); 457 EXPECT_EQ(1u, GetAllWebContents(browser()).size());
457 FinishAsynchronousTest(); 458 FinishAsynchronousTest();
458 } 459 }
459 }; 460 };
460 461
462 #if defined(OS_WIN)
463 DISABLED_HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainCreateAndDeletePageTest);
464 #else
461 HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainCreateAndDeletePageTest); 465 HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainCreateAndDeletePageTest);
466 #endif
462 467
463 class TargetDomainCreateAndDeleteBrowserContextTest 468 class TargetDomainCreateAndDeleteBrowserContextTest
464 : public HeadlessAsyncDevTooledBrowserTest { 469 : public HeadlessAsyncDevTooledBrowserTest {
465 void RunDevTooledTest() override { 470 void RunDevTooledTest() override {
466 EXPECT_TRUE(embedded_test_server()->Start()); 471 EXPECT_TRUE(embedded_test_server()->Start());
467 472
468 EXPECT_EQ(1u, GetAllWebContents(browser()).size()); 473 EXPECT_EQ(1u, GetAllWebContents(browser()).size());
469 474
470 devtools_client_->GetTarget()->GetExperimental()->CreateBrowserContext( 475 devtools_client_->GetTarget()->GetExperimental()->CreateBrowserContext(
471 target::CreateBrowserContextParams::Builder().Build(), 476 target::CreateBrowserContextParams::Builder().Build(),
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
519 void OnDisposeBrowserContextResult( 524 void OnDisposeBrowserContextResult(
520 std::unique_ptr<target::DisposeBrowserContextResult> result) { 525 std::unique_ptr<target::DisposeBrowserContextResult> result) {
521 EXPECT_TRUE(result->GetSuccess()); 526 EXPECT_TRUE(result->GetSuccess());
522 FinishAsynchronousTest(); 527 FinishAsynchronousTest();
523 } 528 }
524 529
525 private: 530 private:
526 std::string browser_context_id_; 531 std::string browser_context_id_;
527 }; 532 };
528 533
534 #if defined(OS_WIN)
535 DISABLED_HEADLESS_ASYNC_DEVTOOLED_TEST_F(
536 TargetDomainCreateAndDeleteBrowserContextTest);
537 #else
529 HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainCreateAndDeleteBrowserContextTest); 538 HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainCreateAndDeleteBrowserContextTest);
539 #endif
530 540
531 class TargetDomainDisposeContextFailsIfInUse 541 class TargetDomainDisposeContextFailsIfInUse
532 : public HeadlessAsyncDevTooledBrowserTest { 542 : public HeadlessAsyncDevTooledBrowserTest {
533 void RunDevTooledTest() override { 543 void RunDevTooledTest() override {
534 EXPECT_TRUE(embedded_test_server()->Start()); 544 EXPECT_TRUE(embedded_test_server()->Start());
535 545
536 EXPECT_EQ(1u, GetAllWebContents(browser()).size()); 546 EXPECT_EQ(1u, GetAllWebContents(browser()).size());
537 devtools_client_->GetTarget()->GetExperimental()->CreateBrowserContext( 547 devtools_client_->GetTarget()->GetExperimental()->CreateBrowserContext(
538 target::CreateBrowserContextParams::Builder().Build(), 548 target::CreateBrowserContextParams::Builder().Build(),
539 base::Bind(&TargetDomainDisposeContextFailsIfInUse::OnContextCreated, 549 base::Bind(&TargetDomainDisposeContextFailsIfInUse::OnContextCreated,
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
595 std::unique_ptr<target::DisposeBrowserContextResult> result) { 605 std::unique_ptr<target::DisposeBrowserContextResult> result) {
596 EXPECT_TRUE(result->GetSuccess()); 606 EXPECT_TRUE(result->GetSuccess());
597 FinishAsynchronousTest(); 607 FinishAsynchronousTest();
598 } 608 }
599 609
600 private: 610 private:
601 std::string context_id_; 611 std::string context_id_;
602 std::string page_id_; 612 std::string page_id_;
603 }; 613 };
604 614
615 #if defined(OS_WIN)
616 DISABLED_HEADLESS_ASYNC_DEVTOOLED_TEST_F(
617 TargetDomainDisposeContextFailsIfInUse);
618 #else
605 HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainDisposeContextFailsIfInUse); 619 HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainDisposeContextFailsIfInUse);
620 #endif
606 621
607 class TargetDomainCreateTwoContexts : public HeadlessAsyncDevTooledBrowserTest, 622 class TargetDomainCreateTwoContexts : public HeadlessAsyncDevTooledBrowserTest,
608 public target::ExperimentalObserver, 623 public target::ExperimentalObserver,
609 public page::Observer { 624 public page::Observer {
610 public: 625 public:
611 void RunDevTooledTest() override { 626 void RunDevTooledTest() override {
612 EXPECT_TRUE(embedded_test_server()->Start()); 627 EXPECT_TRUE(embedded_test_server()->Start());
613 628
614 base::RunLoop run_loop; 629 base::RunLoop run_loop;
615 devtools_client_->GetPage()->AddObserver(this); 630 devtools_client_->GetPage()->AddObserver(this);
(...skipping 251 matching lines...) Expand 10 before | Expand all | Expand 10 after
867 std::string context_id_one_; 882 std::string context_id_one_;
868 std::string context_id_two_; 883 std::string context_id_two_;
869 std::string page_id_one_; 884 std::string page_id_one_;
870 std::string page_id_two_; 885 std::string page_id_two_;
871 bool page_one_loaded_ = false; 886 bool page_one_loaded_ = false;
872 bool page_two_loaded_ = false; 887 bool page_two_loaded_ = false;
873 int page_close_count_ = 0; 888 int page_close_count_ = 0;
874 int context_closed_count_ = 0; 889 int context_closed_count_ = 0;
875 }; 890 };
876 891
892 #if defined(OS_WIN)
893 DISABLED_HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainCreateTwoContexts);
894 #else
877 HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainCreateTwoContexts); 895 HEADLESS_ASYNC_DEVTOOLED_TEST_F(TargetDomainCreateTwoContexts);
896 #endif
878 897
879 class HeadlessDevToolsNavigationControlTest 898 class HeadlessDevToolsNavigationControlTest
880 : public HeadlessAsyncDevTooledBrowserTest, 899 : public HeadlessAsyncDevTooledBrowserTest,
881 page::ExperimentalObserver { 900 page::ExperimentalObserver {
882 public: 901 public:
883 void RunDevTooledTest() override { 902 void RunDevTooledTest() override {
884 EXPECT_TRUE(embedded_test_server()->Start()); 903 EXPECT_TRUE(embedded_test_server()->Start());
885 base::RunLoop run_loop; 904 base::RunLoop run_loop;
886 devtools_client_->GetPage()->GetExperimental()->AddObserver(this); 905 devtools_client_->GetPage()->GetExperimental()->AddObserver(this);
887 devtools_client_->GetPage()->Enable(run_loop.QuitClosure()); 906 devtools_client_->GetPage()->Enable(run_loop.QuitClosure());
(...skipping 544 matching lines...) Expand 10 before | Expand all | Expand 10 after
1432 } 1451 }
1433 1452
1434 private: 1453 private:
1435 base::Lock lock_; 1454 base::Lock lock_;
1436 std::string url_that_failed_to_load_; 1455 std::string url_that_failed_to_load_;
1437 }; 1456 };
1438 1457
1439 HEADLESS_ASYNC_DEVTOOLED_TEST_F(FailedUrlRequestTest); 1458 HEADLESS_ASYNC_DEVTOOLED_TEST_F(FailedUrlRequestTest);
1440 1459
1441 } // namespace headless 1460 } // namespace headless
OLDNEW
« no previous file with comments | « no previous file | headless/test/headless_browser_test.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698