commit 616100e9bd4284dfb3fcc62c66ae966be1e042aa
author Koji Ishii <kojii@chromium.org> Sat Sep 16 12:36:06 2017
committer Commit Bot <commit-bot@chromium.org> Sat Sep 16 12:36:06 2017
tree 8839656d5e80fe485f18915632b56d2b75ac29a8
parent 36125a320bf94d4f19e19f5f3176035b9095fd1c

[LayoutNG] Fix ComputedStyle::ShouldUseTextIndent()

This patch fixes a typo that calls the default constructor instead of
the getter when the logic was copied from BreakingContextInlineHeaders.

This code path is used only in LayoutNG. Tests in fast/css3-text/css3-
text-indent/ still fail for other reasons when LayoutNG is enabled.

Bug: 636993, 346399
Change-Id: I9beea46bd11d3486bd9047ad98d14d960df8bad3
Reviewed-on: https://chromium-review.googlesource.com/670199
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502501}

third_party/WebKit/Source/core/style/ComputedStyle.cpp

diff --git a/third_party/WebKit/Source/core/style/ComputedStyle.cpp b/third_party/WebKit/Source/core/style/ComputedStyle.cpp
index ebb63665..898d782 100644
--- a/third_party/WebKit/Source/core/style/ComputedStyle.cpp
+++ b/third_party/WebKit/Source/core/style/ComputedStyle.cpp
@@ -1256,7 +1256,8 @@
   bool should_use =
       is_first_line || (is_after_forced_break &&
                         GetTextIndentLine() != TextIndentLine::kFirstLine);
-  return TextIndentType() == TextIndentType::kNormal ? should_use : !should_use;
+  return GetTextIndentType() == TextIndentType::kNormal ? should_use
+                                                        : !should_use;
 }
 
 const AtomicString& ComputedStyle::TextEmphasisMarkString() const {