Simplify SVG layout attribute reordering

third_party/WebKit/Source/core/layout/line/InlineFlowBox.cpp

Index: third_party/WebKit/Source/core/layout/line/InlineFlowBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/line/InlineFlowBox.cpp b/third_party/WebKit/Source/core/layout/line/InlineFlowBox.cpp
index 18e37831e5e76d6fcce75e04d7bcb52fd593fcd8..1fbc2b47065ef89514501a0336e596e0ff869211 100644
--- a/third_party/WebKit/Source/core/layout/line/InlineFlowBox.cpp
+++ b/third_party/WebKit/Source/core/layout/line/InlineFlowBox.cpp
@@ -1279,12 +1279,10 @@ void InlineFlowBox::collectLeafBoxesInLogicalOrder(Vector<InlineBox*>& leafBoxes
                 ++it;
             }
             Vector<InlineBox*>::iterator last = it;
-            if (customReverseImplementation) {
-                ASSERT(userData);
+            if (customReverseImplementation)
                 (*customReverseImplementation)(userData, first, last);

[Stephen Chennney, 2016/04/29 18:05:19]

Do we document the fact that userData may be null in this callback method, in
the place we declare the CustomInlineBoxRangeReverse type? If not, we should.

Is userData even needed at all now?

[fs, 2016/04/29 18:44:15]

It's dead AFAIK. reorderValueLists is the only thing that uses the
callback/closure, and I hope to be able to get rid of it there as well. Removed
userData.

-            } else {
+            else
                 std::reverse(first, last);
-            }
         }
         ++minLevel;
     }