Delete selection gap code.

third_party/WebKit/Source/core/layout/LayoutBR.cpp

Index: third_party/WebKit/Source/core/layout/LayoutBR.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBR.cpp b/third_party/WebKit/Source/core/layout/LayoutBR.cpp
index 3d3fcc6fb5cb3cdf3680bd99c22149a305105644..25ba1c9c3983370171a942119647bf5cffe35a1d 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBR.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBR.cpp
@@ -45,9 +45,6 @@ LayoutBR::~LayoutBR()
 
 LayoutRect LayoutBR::selectionRectForPaintInvalidation(const LayoutBoxModelObject* paintInvalidationContainer) const
 {
-    if (!RuntimeEnabledFeatures::selectionPaintingWithoutSelectionGapsEnabled())

[chrishtr, 2016/01/28 22:06:22]

Why was this a negation of the feature when early-returning?

[wkorman, 2016/01/28 22:54:32]

This code was added as part of the new selection gap behavior:

https://codereview.chromium.org/1350973003

Previously line breaks did not need any special selection gap related
invalidation since they did not actually paint anything.

So, it was easiest to just do this early return with an empty rect when the new
behavior was not enabled.

-        return LayoutRect();
-
     // Although line breaks contain no actual text, if we're selected we need
     // to return a rect that includes space to illustrate a newline.
     return LayoutText::selectionRectForPaintInvalidation(paintInvalidationContainer);