Commit 3cc2505d authored by Yoshifumi Inoue's avatar Yoshifumi Inoue Committed by Commit Bot

Get rid of unused functions from ElementInnerText

This patch gets rid of unused functions in ElementInnerText for improving code
health.

Change-Id: I61692011375770b2c263c9a82499d7d02c340ca2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2037304
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Auto-Submit: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: default avatarKent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#738488}
parent 80e9555e
...@@ -72,7 +72,6 @@ class ElementInnerTextCollector final { ...@@ -72,7 +72,6 @@ class ElementInnerTextCollector final {
static bool IsBeingRendered(const Node& node); static bool IsBeingRendered(const Node& node);
// Returns true if used value of "display" is block-level. // Returns true if used value of "display" is block-level.
static bool IsDisplayBlockLevel(const Node&); static bool IsDisplayBlockLevel(const Node&);
static LayoutObject* PreviousLeafOf(const LayoutObject& layout_object);
static bool ShouldEmitNewlineForTableRow( static bool ShouldEmitNewlineForTableRow(
const LayoutNGTableRowInterface& table_row); const LayoutNGTableRowInterface& table_row);
...@@ -81,7 +80,6 @@ class ElementInnerTextCollector final { ...@@ -81,7 +80,6 @@ class ElementInnerTextCollector final {
void ProcessChildrenWithRequiredLineBreaks(const Node& node, void ProcessChildrenWithRequiredLineBreaks(const Node& node,
int required_line_break_count); int required_line_break_count);
void ProcessLayoutText(const LayoutText& layout_text, const Text& text_node); void ProcessLayoutText(const LayoutText& layout_text, const Text& text_node);
void ProcessLayoutTextEmpty(const LayoutText& layout_text);
void ProcessNode(const Node& node); void ProcessNode(const Node& node);
void ProcessOptionElement(const HTMLOptionElement& element); void ProcessOptionElement(const HTMLOptionElement& element);
void ProcessSelectElement(const HTMLSelectElement& element); void ProcessSelectElement(const HTMLSelectElement& element);
...@@ -180,19 +178,6 @@ bool ElementInnerTextCollector::IsDisplayBlockLevel(const Node& node) { ...@@ -180,19 +178,6 @@ bool ElementInnerTextCollector::IsDisplayBlockLevel(const Node& node) {
return true; return true;
} }
// static
LayoutObject* ElementInnerTextCollector::PreviousLeafOf(
const LayoutObject& layout_object) {
LayoutObject* parent = layout_object.Parent();
for (LayoutObject* runner = layout_object.PreviousInPreOrder(); runner;
runner = runner->PreviousInPreOrder()) {
if (runner != parent)
return runner;
parent = runner->Parent();
}
return nullptr;
}
// static // static
bool ElementInnerTextCollector::ShouldEmitNewlineForTableRow( bool ElementInnerTextCollector::ShouldEmitNewlineForTableRow(
const LayoutNGTableRowInterface& table_row) { const LayoutNGTableRowInterface& table_row) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment