Commit 318a6866 authored by Hikaru Nishida's avatar Hikaru Nishida Committed by Commit Bot

Simplify redundant member function call in SetSelectionStateIfNeeded

This patch removes a redundant class specifier |LayoutObject::| for
|SetSelectionState()| call in |SetSelectionStateIfNeeded()| because
it is non-virtual member function for improving code health.

Change-Id: I68b58d07c5452e36e05b1a20a3e5d20eeee90237
Reviewed-on: https://chromium-review.googlesource.com/903524Reviewed-by: default avatarYoshifumi Inoue <yosin@chromium.org>
Reviewed-by: default avatarYoichi Osato <yoichio@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534662}
parent 744ac3cb
......@@ -278,9 +278,7 @@ static void SetSelectionStateIfNeeded(LayoutObject* layout_object,
DCHECK_NE(state, SelectionState::kNone) << layout_object;
if (layout_object->GetSelectionState() == state)
return;
// TODO(yoichio): Once we make LayoutObject::SetSelectionState() tribial, use
// it directly.
layout_object->LayoutObject::SetSelectionState(state);
layout_object->SetSelectionState(state);
// Set containing block SelectionState kContain for CSS ::selection style.
// See LayoutObject::InvalidatePaintForSelection().
......
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