Chromium Code Reviews| Index: third_party/WebKit/Source/core/layout/ng/ng_fragment.h |
| diff --git a/third_party/WebKit/Source/core/layout/ng/ng_fragment.h b/third_party/WebKit/Source/core/layout/ng/ng_fragment.h |
| index 5272b4357e067ba5c5faa0bdf6197497b6807407..e89deeb784b195fc93a0793aa1a445b17e7ca4e3 100644 |
| --- a/third_party/WebKit/Source/core/layout/ng/ng_fragment.h |
| +++ b/third_party/WebKit/Source/core/layout/ng/ng_fragment.h |
| @@ -13,14 +13,13 @@ |
| namespace blink { |
| -class CORE_EXPORT NGFragment : public GarbageCollected<NGFragment> { |
| +class CORE_EXPORT NGFragment { |
| + STACK_ALLOCATED(); |
| + |
| public: |
| NGWritingMode WritingMode() const { |
| return static_cast<NGWritingMode>(writing_mode_); |
| } |
| - TextDirection Direction() const { |
|
cbiesinger
2017/02/03 19:43:40
Don't you need the direction for the inline offset
ikilpatrick
2017/02/04 09:45:57
No you're right, I've just removed this for now as
|
| - return static_cast<TextDirection>(direction_); |
| - } |
| // Returns the border-box size. |
| LayoutUnit InlineSize() const; |
| @@ -36,22 +35,13 @@ class CORE_EXPORT NGFragment : public GarbageCollected<NGFragment> { |
| NGPhysicalFragment::NGFragmentType Type() const; |
| - NGPhysicalFragment* PhysicalFragment() const { return physical_fragment_; }; |
| - |
| - DECLARE_TRACE(); |
| - |
| protected: |
| - NGFragment(NGWritingMode writing_mode, |
| - TextDirection direction, |
| - NGPhysicalFragment* physical_fragment) |
| - : physical_fragment_(physical_fragment), |
| - writing_mode_(writing_mode), |
| - direction_(static_cast<unsigned>(direction)) {} |
| + NGFragment(NGWritingMode writing_mode, NGPhysicalFragment* physical_fragment) |
| + : physical_fragment_(physical_fragment), writing_mode_(writing_mode) {} |
| - Member<NGPhysicalFragment> physical_fragment_; |
| + NGPhysicalFragment* physical_fragment_; |
| unsigned writing_mode_ : 3; |
| - unsigned direction_ : 1; |
| }; |
| } // namespace blink |