Cleanup: remove state variables from inline_algorithm

third_party/WebKit/Source/core/layout/ng/ng_inline_layout_algorithm.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NGInlineLayoutAlgorithm_h
 #define NGInlineLayoutAlgorithm_h
 
 #include "core/CoreExport.h"
 #include "core/layout/ng/ng_layout_algorithm.h"
 #include "wtf/RefPtr.h"
 
 namespace blink {
 
 class ComputedStyle;
 class NGBreakToken;
 class NGConstraintSpace;
 class NGFragmentBuilder;
 class NGInlineNode;
-class NGLineBuilder;
 
 // A class for inline layout (e.g. a anonymous block with inline-level children
 // only).
 //
 // This algorithm may at some point be merged with NGBlockLayoutAlgorithm in
 // the future. Currently it exists as its own class to simplify the LayoutNG
 // transition period.
 class CORE_EXPORT NGInlineLayoutAlgorithm : public NGLayoutAlgorithm {
  public:
   // Default constructor.
   // @param style Style reference of the block that is being laid out.
   // @param first_child Our first child; the algorithm will use its NextSibling
   //                    method to access all the children.
   // @param space The constraint space which the algorithm should generate a
   //              fragment within.
   NGInlineLayoutAlgorithm(PassRefPtr<const ComputedStyle>,
                           NGInlineNode* first_child,
                           NGConstraintSpace* space,
                           NGBreakToken* break_token = nullptr);
 
   NGPhysicalFragment* Layout() override;
 
   DECLARE_VIRTUAL_TRACE();
 
  private:
   // Read-only Getters.
   const ComputedStyle& Style() const { return *style_; }
 
   bool LayoutCurrentChild();

[ikilpatrick, 2017/01/23 23:21:36]

can you remove this? i forgot to in an earlier patch.

[atotic, 2017/01/23 23:31:21]

done

-  NGConstraintSpace* CreateConstraintSpaceForCurrentChild() const;
+  NGConstraintSpace* CreateConstraintSpaceForCurrentChild(

[ikilpatrick, 2017/01/23 23:21:36]

can you rename this to:
CreateConstraintSpaceForChild(const NGInlineNode& child)
?

[atotic, 2017/01/23 23:31:21]

done

+      const NGInlineNode& current_child) const;
 
   RefPtr<const ComputedStyle> style_;
   Member<NGInlineNode> first_child_;
   Member<NGConstraintSpace> constraint_space_;
   Member<NGBreakToken> break_token_;
   Member<NGFragmentBuilder> builder_;
-  Member<NGConstraintSpace> space_for_current_child_;
-  Member<NGInlineNode> current_child_;
-  Member<NGLineBuilder> line_builder_;
 };
 
 }  // namespace blink
 
 #endif  // NGInlineLayoutAlgorithm_h