third_party/WebKit/Source/core/dom/DocumentTest.cpp - Issue 2726593002: Use mayNotBeMainThread() for wrapper optimization

Side by Side Diff: third_party/WebKit/Source/core/dom/DocumentTest.cpp

Issue 2726593002: Use mayNotBeMainThread() for wrapper optimization (Closed)

Patch Set: Renaming + move decrement to destructor Created 3 years, 9 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:

« third_party/WebKit/Source/bindings/core/v8/DOMWrapperWorld.cpp ('K') | « third_party/WebKit/Source/bindings/core/v8/V8DOMWrapper.h ('k') | third_party/WebKit/Source/core/dom/Node.h » ('j') | third_party/WebKit/Source/core/dom/RangeTest.cpp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 /*	1 /*

2 * Copyright (c) 2014, Google Inc. All rights reserved.	2 * Copyright (c) 2014, Google Inc. All rights reserved.

3 *	3 *

4 * Redistribution and use in source and binary forms, with or without	4 * Redistribution and use in source and binary forms, with or without

5 * modification, are permitted provided that the following conditions are	5 * modification, are permitted provided that the following conditions are

6 * met:	6 * met:

7 *	7 *

8 * * Redistributions of source code must retain the above copyright	8 * * Redistributions of source code must retain the above copyright

9 * notice, this list of conditions and the following disclaimer.	9 * notice, this list of conditions and the following disclaimer.

10 * * Redistributions in binary form must reproduce the above	10 * * Redistributions in binary form must reproduce the above

(...skipping 35 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
46 #include <memory>	46 #include <memory>

47	47

48 namespace blink {	48 namespace blink {

49	49

50 class DocumentTest : public ::testing::Test {	50 class DocumentTest : public ::testing::Test {

51 protected:	51 protected:

52 void SetUp() override;	52 void SetUp() override;

53	53

54 void TearDown() override { ThreadState::current()->collectAllGarbage(); }	54 void TearDown() override { ThreadState::current()->collectAllGarbage(); }

55	55

	56 LocalFrame& frame() const { return m_dummyPageHolder->frame(); }

56 Document& document() const { return m_dummyPageHolder->document(); }	57 Document& document() const { return m_dummyPageHolder->document(); }

57 Page& page() const { return m_dummyPageHolder->page(); }	58 Page& page() const { return m_dummyPageHolder->page(); }

58	59

59 void setHtmlInnerHTML(const char*);	60 void setHtmlInnerHTML(const char*);

60	61

61 private:	62 private:

62 std::unique_ptr<DummyPageHolder> m_dummyPageHolder;	63 std::unique_ptr<DummyPageHolder> m_dummyPageHolder;

63 };	64 };

64	65

65 void DocumentTest::SetUp() {	66 void DocumentTest::SetUp() {

(...skipping 502 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
568 document().body());	569 document().body());

569 ASSERT_EQ(2u, observer.childrenChangedNodes().size());	570 ASSERT_EQ(2u, observer.childrenChangedNodes().size());

570 EXPECT_EQ(document().documentElement(), observer.childrenChangedNodes()[0]);	571 EXPECT_EQ(document().documentElement(), observer.childrenChangedNodes()[0]);

571 EXPECT_EQ(document().documentElement(), observer.childrenChangedNodes()[1]);	572 EXPECT_EQ(document().documentElement(), observer.childrenChangedNodes()[1]);

572	573

573 ASSERT_EQ(1u, observer.removedNodes().size());	574 ASSERT_EQ(1u, observer.removedNodes().size());

574 EXPECT_EQ(replacedNode, observer.removedNodes()[0]);	575 EXPECT_EQ(replacedNode, observer.removedNodes()[0]);

575 }	576 }

576	577

577 TEST_F(DocumentTest, SynchronousMutationNotifierSplitTextNode) {	578 TEST_F(DocumentTest, SynchronousMutationNotifierSplitTextNode) {

	579 ScriptState* scriptState = ScriptState::forMainWorld(&frame());

	580 ScriptState::Scope scope(scriptState);

	581

578 auto& observer = *new TestSynchronousMutationObserver(document());	582 auto& observer = *new TestSynchronousMutationObserver(document());

579	583

580 Text* splitSample = document().createTextNode("0123456789");	584 Text* splitSample = document().createTextNode("0123456789");

581 document().body()->appendChild(splitSample);	585 document().body()->appendChild(splitSample);

582	586

583 splitSample->splitText(4, ASSERT_NO_EXCEPTION);	587 splitSample->splitText(4, ASSERT_NO_EXCEPTION);

584 ASSERT_EQ(1u, observer.splitTextNodes().size());	588 ASSERT_EQ(1u, observer.splitTextNodes().size());

585 EXPECT_EQ(splitSample, observer.splitTextNodes()[0]);	589 EXPECT_EQ(splitSample, observer.splitTextNodes()[0]);

586 }	590 }

587	591

(...skipping 56 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
644 {	648 {

645 setHtmlInnerHTML(	649 setHtmlInnerHTML(

646 "<body>"	650 "<body>"

647 "<meta name=\"theme-color\" content=\"#00ff00\">");	651 "<meta name=\"theme-color\" content=\"#00ff00\">");

648 EXPECT_EQ(Color(0, 255, 0), document().themeColor())	652 EXPECT_EQ(Color(0, 255, 0), document().themeColor())

649 << "Theme color should be bright green.";	653 << "Theme color should be bright green.";

650 }	654 }

651 }	655 }

652	656

653 } // namespace blink	657 } // namespace blink

OLD	NEW