Commit a3925d24 authored by Yoichi Osato's avatar Yoichi Osato Committed by Commit Bot

Remove unused LayoutSelection::HasPendingSelection()

This patch just removes the functions.

Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
Change-Id: Ie0bcda4080ff4390cf29cad4661dca67ead9ad44
Reviewed-on: https://chromium-review.googlesource.com/c/1264156Reviewed-by: default avatarYoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597034}
parent 9b7961c2
...@@ -857,10 +857,6 @@ bool FrameSelection::FrameIsFocusedAndActive() const { ...@@ -857,10 +857,6 @@ bool FrameSelection::FrameIsFocusedAndActive() const {
frame_->GetPage()->GetFocusController().IsActive(); frame_->GetPage()->GetFocusController().IsActive();
} }
bool FrameSelection::NeedsLayoutSelectionUpdate() const {
return layout_selection_->HasPendingSelection();
}
void FrameSelection::CommitAppearanceIfNeeded() { void FrameSelection::CommitAppearanceIfNeeded() {
return layout_selection_->Commit(); return layout_selection_->Commit();
} }
......
...@@ -210,7 +210,6 @@ class CORE_EXPORT FrameSelection final ...@@ -210,7 +210,6 @@ class CORE_EXPORT FrameSelection final
void DocumentAttached(Document*); void DocumentAttached(Document*);
void DidLayout(); void DidLayout();
bool NeedsLayoutSelectionUpdate() const;
void CommitAppearanceIfNeeded(); void CommitAppearanceIfNeeded();
void SetCaretVisible(bool caret_is_visible); void SetCaretVisible(bool caret_is_visible);
void ScheduleVisualUpdate() const; void ScheduleVisualUpdate() const;
......
...@@ -630,7 +630,7 @@ static LayoutTextSelectionStatus ComputeSelectionStatusForNode( ...@@ -630,7 +630,7 @@ static LayoutTextSelectionStatus ComputeSelectionStatusForNode(
LayoutTextSelectionStatus LayoutSelection::ComputeSelectionStatus( LayoutTextSelectionStatus LayoutSelection::ComputeSelectionStatus(
const LayoutText& layout_text) const { const LayoutText& layout_text) const {
DCHECK(!HasPendingSelection()); DCHECK(!has_pending_selection_);
const SelectionState selection_state = GetSelectionStateFor(layout_text); const SelectionState selection_state = GetSelectionStateFor(layout_text);
if (selection_state == SelectionState::kNone) if (selection_state == SelectionState::kNone)
return {0, 0, SelectionIncludeEnd::kNotInclude}; return {0, 0, SelectionIncludeEnd::kNotInclude};
...@@ -791,7 +791,7 @@ static void AssertLayoutClean(Document* document) { ...@@ -791,7 +791,7 @@ static void AssertLayoutClean(Document* document) {
} }
void LayoutSelection::Commit() { void LayoutSelection::Commit() {
if (!HasPendingSelection()) if (!has_pending_selection_)
return; return;
has_pending_selection_ = false; has_pending_selection_ = false;
......
...@@ -44,7 +44,6 @@ class LayoutSelection final : public GarbageCollected<LayoutSelection> { ...@@ -44,7 +44,6 @@ class LayoutSelection final : public GarbageCollected<LayoutSelection> {
return new LayoutSelection(frame_selection); return new LayoutSelection(frame_selection);
} }
bool HasPendingSelection() const { return has_pending_selection_; }
void SetHasPendingSelection(); void SetHasPendingSelection();
void Commit(); void Commit();
......
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