Commit c460de5c authored by yosin's avatar yosin Committed by Commit bot

Get rid of unused parameter from CaretBase::invalidateCaretRect()

This patch gets rid of unused parameter |caretRectChanged| from
|CaretBase::invalidateCaretRect()|, since it is always |false| as default
parameter, for improving code health.

BUG=n/a
TEST=n/a; no behavior changes

Review-Url: https://codereview.chromium.org/2320933002
Cr-Commit-Position: refs/heads/master@{#417239}
parent 16061dc3
...@@ -181,11 +181,8 @@ bool CaretBase::shouldRepaintCaret(const LayoutViewItem view) const ...@@ -181,11 +181,8 @@ bool CaretBase::shouldRepaintCaret(const LayoutViewItem view) const
return false; return false;
} }
void CaretBase::invalidateCaretRect(Node* node, bool caretRectChanged) void CaretBase::invalidateCaretRect(Node* node)
{ {
if (caretRectChanged)
return;
if (LayoutViewItem view = node->document().layoutViewItem()) { if (LayoutViewItem view = node->document().layoutViewItem()) {
node->document().updateStyleAndLayoutTree(); node->document().updateStyleAndLayoutTree();
if (hasEditableStyle(*node) || shouldRepaintCaret(view)) if (hasEditableStyle(*node) || shouldRepaintCaret(view))
......
...@@ -50,7 +50,7 @@ public: ...@@ -50,7 +50,7 @@ public:
explicit CaretBase(CaretVisibility = CaretVisibility::Hidden); explicit CaretBase(CaretVisibility = CaretVisibility::Hidden);
virtual ~CaretBase(); virtual ~CaretBase();
void invalidateCaretRect(Node*, bool caretRectChanged = false); void invalidateCaretRect(Node*);
void clearCaretRect(); void clearCaretRect();
// Creating VisiblePosition causes synchronous layout so we should use the // Creating VisiblePosition causes synchronous layout so we should use the
// PositionWithAffinity version if possible. // PositionWithAffinity version if possible.
......
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