Remove extraneous frameView->setNeedsLayout() in LayoutPart

third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp

Index: third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
diff --git a/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp b/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
index 31aee794cdd7ee85c899f12e872794e9eb68a403..807d953be493a72521c6fde33daff02f0a9ac3f0 100644
--- a/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
+++ b/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
@@ -95,40 +95,6 @@ TEST_F(FrameSelectionTest, InvalidateCaretRect)
     EXPECT_FALSE(selection().isCaretBoundsDirty());
 }
 
-TEST_F(FrameSelectionTest, PaintCaretShouldNotLayout)
-{
-    Text* text = appendTextNode("Hello, World!");
-    document().view()->updateAllLifecyclePhases();
-
-    document().body()->setContentEditable("true", ASSERT_NO_EXCEPTION);
-    document().body()->focus();
-    EXPECT_TRUE(document().body()->focused());
-
-    VisibleSelection validSelection(Position(text, 0), Position(text, 0));
-    selection().setCaretVisible(true);
-    setSelection(validSelection);
-    EXPECT_TRUE(selection().isCaret());
-    EXPECT_TRUE(shouldPaintCaretForTesting());
-
-    int startCount = layoutCount();
-    {
-        // To force layout in next updateLayout calling, widen view.
-        FrameView& frameView = dummyPageHolder().frameView();
-        IntRect frameRect = frameView.frameRect();
-        frameRect.setWidth(frameRect.width() + 1);
-        frameRect.setHeight(frameRect.height() + 1);
-        dummyPageHolder().frameView().setFrameRect(frameRect);
-    }
-    std::unique_ptr<PaintController> paintController = PaintController::create();
-    {
-        GraphicsContext context(*paintController);
-        DrawingRecorder drawingRecorder(context, *dummyPageHolder().frameView().layoutView(), DisplayItem::Caret, LayoutRect::infiniteIntRect());
-        selection().paintCaret(context, LayoutPoint());
-    }
-    paintController->commitNewDisplayItems();
-    EXPECT_EQ(startCount, layoutCount());

[trchen, 2016/07/28 00:04:36]

This is bogus. Resizing the frame would invalidate layout, and painting requires
a clean layout. It is wrong to expect layoutCount() not increased.

The original CL that introduced this test to make sure caret painting code is
impossible to invoke layout, but I think the method in that CL was wrong. We
should have figured out why layout could be dirty during paint.

[bokan, 2016/07/28 13:24:56]

Agree, this is sketchy. Could you file a bug to follow up with your suggestion
of figuring out why layout can be dirty during paint?

[trchen, 2016/07/28 22:45:54]

I think the original bug is already solved. Xianzhu recently invested a lot of
time to investigate layout lifecycle bugs. We now have many more DCHECK and
fixed them. Here is the meta bug: crbug.com/590856

[bokan, 2016/07/28 22:48:41]

ok, sgtm, thanks.

-}
-
 TEST_F(FrameSelectionTest, InvalidatePreviousCaretAfterRemovingLastCharacter)
 {
     Text* text = appendTextNode("Hello, World!");