Commit d33d00d5 authored by Yoshifumi Inoue's avatar Yoshifumi Inoue Committed by Commit Bot

Get rid of redundant equality checks from EqualSelectionsAlgorithm() for VisibleSelectionTemplate

This patch gets rid of redundant equality checks of |Start()| and |End()| of
|VisibleSelectionTemplate| from |EqualSelectionsAlgorithm()| for improving
code health.

Change-Id: I0fd5949f8356b0527fa5e67426818434efedbee1
Reviewed-on: https://chromium-review.googlesource.com/597510Reviewed-by: default avatarXiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491642}
parent 2a92a47c
......@@ -813,9 +813,7 @@ static bool EqualSelectionsAlgorithm(
const VisibleSelectionTemplate<Strategy> selection_wrapper1(selection1);
const VisibleSelectionTemplate<Strategy> selection_wrapper2(selection2);
return selection_wrapper1.Start() == selection_wrapper2.Start() &&
selection_wrapper1.End() == selection_wrapper2.End() &&
selection_wrapper1.Base() == selection_wrapper2.Base() &&
return selection_wrapper1.Base() == selection_wrapper2.Base() &&
selection_wrapper1.Extent() == selection_wrapper2.Extent();
}
......
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