Merge "Fix typos related to sortSelectionIndices"
This commit is contained in:
@@ -109,7 +109,7 @@ public final class SelectionActionModeHelper {
|
|||||||
*
|
*
|
||||||
* @return the swap result, index 0 is the start index and index 1 is the end index.
|
* @return the swap result, index 0 is the start index and index 1 is the end index.
|
||||||
*/
|
*/
|
||||||
private static int[] sortSelctionIndices(int selectionStart, int selectionEnd) {
|
private static int[] sortSelectionIndices(int selectionStart, int selectionEnd) {
|
||||||
if (selectionStart < selectionEnd) {
|
if (selectionStart < selectionEnd) {
|
||||||
return new int[]{selectionStart, selectionEnd};
|
return new int[]{selectionStart, selectionEnd};
|
||||||
}
|
}
|
||||||
@@ -123,11 +123,11 @@ public final class SelectionActionModeHelper {
|
|||||||
* @param textView the selected TextView.
|
* @param textView the selected TextView.
|
||||||
* @return the swap result, index 0 is the start index and index 1 is the end index.
|
* @return the swap result, index 0 is the start index and index 1 is the end index.
|
||||||
*/
|
*/
|
||||||
private static int[] sortSelctionIndicesFromTextView(TextView textView) {
|
private static int[] sortSelectionIndicesFromTextView(TextView textView) {
|
||||||
int selectionStart = textView.getSelectionStart();
|
int selectionStart = textView.getSelectionStart();
|
||||||
int selectionEnd = textView.getSelectionEnd();
|
int selectionEnd = textView.getSelectionEnd();
|
||||||
|
|
||||||
return sortSelctionIndices(selectionStart, selectionEnd);
|
return sortSelectionIndices(selectionStart, selectionEnd);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -136,7 +136,7 @@ public final class SelectionActionModeHelper {
|
|||||||
public void startSelectionActionModeAsync(boolean adjustSelection) {
|
public void startSelectionActionModeAsync(boolean adjustSelection) {
|
||||||
// Check if the smart selection should run for editable text.
|
// Check if the smart selection should run for editable text.
|
||||||
adjustSelection &= getTextClassificationSettings().isSmartSelectionEnabled();
|
adjustSelection &= getTextClassificationSettings().isSmartSelectionEnabled();
|
||||||
int[] sortedSelectionIndices = sortSelctionIndicesFromTextView(mTextView);
|
int[] sortedSelectionIndices = sortSelectionIndicesFromTextView(mTextView);
|
||||||
|
|
||||||
mSelectionTracker.onOriginalSelection(
|
mSelectionTracker.onOriginalSelection(
|
||||||
getText(mTextView),
|
getText(mTextView),
|
||||||
@@ -166,7 +166,7 @@ public final class SelectionActionModeHelper {
|
|||||||
* Starts Link ActionMode.
|
* Starts Link ActionMode.
|
||||||
*/
|
*/
|
||||||
public void startLinkActionModeAsync(int start, int end) {
|
public void startLinkActionModeAsync(int start, int end) {
|
||||||
int[] indexResult = sortSelctionIndices(start, end);
|
int[] indexResult = sortSelectionIndices(start, end);
|
||||||
mSelectionTracker.onOriginalSelection(getText(mTextView), indexResult[0], indexResult[1],
|
mSelectionTracker.onOriginalSelection(getText(mTextView), indexResult[0], indexResult[1],
|
||||||
true /*isLink*/);
|
true /*isLink*/);
|
||||||
cancelAsyncTask();
|
cancelAsyncTask();
|
||||||
@@ -202,21 +202,21 @@ public final class SelectionActionModeHelper {
|
|||||||
|
|
||||||
/** Reports a selection action event. */
|
/** Reports a selection action event. */
|
||||||
public void onSelectionAction(int menuItemId, @Nullable String actionLabel) {
|
public void onSelectionAction(int menuItemId, @Nullable String actionLabel) {
|
||||||
int[] sortedSelectionIndices = sortSelctionIndicesFromTextView(mTextView);
|
int[] sortedSelectionIndices = sortSelectionIndicesFromTextView(mTextView);
|
||||||
mSelectionTracker.onSelectionAction(
|
mSelectionTracker.onSelectionAction(
|
||||||
sortedSelectionIndices[0], sortedSelectionIndices[1],
|
sortedSelectionIndices[0], sortedSelectionIndices[1],
|
||||||
getActionType(menuItemId), actionLabel, mTextClassification);
|
getActionType(menuItemId), actionLabel, mTextClassification);
|
||||||
}
|
}
|
||||||
|
|
||||||
public void onSelectionDrag() {
|
public void onSelectionDrag() {
|
||||||
int[] sortedSelectionIndices = sortSelctionIndicesFromTextView(mTextView);
|
int[] sortedSelectionIndices = sortSelectionIndicesFromTextView(mTextView);
|
||||||
mSelectionTracker.onSelectionAction(
|
mSelectionTracker.onSelectionAction(
|
||||||
sortedSelectionIndices[0], sortedSelectionIndices[1],
|
sortedSelectionIndices[0], sortedSelectionIndices[1],
|
||||||
SelectionEvent.ACTION_DRAG, /* actionLabel= */ null, mTextClassification);
|
SelectionEvent.ACTION_DRAG, /* actionLabel= */ null, mTextClassification);
|
||||||
}
|
}
|
||||||
|
|
||||||
public void onTextChanged(int start, int end) {
|
public void onTextChanged(int start, int end) {
|
||||||
int[] sortedSelectionIndices = sortSelctionIndices(start, end);
|
int[] sortedSelectionIndices = sortSelectionIndices(start, end);
|
||||||
mSelectionTracker.onTextChanged(sortedSelectionIndices[0], sortedSelectionIndices[1],
|
mSelectionTracker.onTextChanged(sortedSelectionIndices[0], sortedSelectionIndices[1],
|
||||||
mTextClassification);
|
mTextClassification);
|
||||||
}
|
}
|
||||||
@@ -335,7 +335,7 @@ public final class SelectionActionModeHelper {
|
|||||||
startSelectionActionMode(startSelectionResult);
|
startSelectionActionMode(startSelectionResult);
|
||||||
};
|
};
|
||||||
// TODO do not trigger the animation if the change included only non-printable characters
|
// TODO do not trigger the animation if the change included only non-printable characters
|
||||||
int[] sortedSelectionIndices = sortSelctionIndicesFromTextView(mTextView);
|
int[] sortedSelectionIndices = sortSelectionIndicesFromTextView(mTextView);
|
||||||
final boolean didSelectionChange =
|
final boolean didSelectionChange =
|
||||||
result != null && (sortedSelectionIndices[0] != result.mStart
|
result != null && (sortedSelectionIndices[0] != result.mStart
|
||||||
|| sortedSelectionIndices[1] != result.mEnd);
|
|| sortedSelectionIndices[1] != result.mEnd);
|
||||||
@@ -488,7 +488,7 @@ public final class SelectionActionModeHelper {
|
|||||||
if (actionMode != null) {
|
if (actionMode != null) {
|
||||||
actionMode.invalidate();
|
actionMode.invalidate();
|
||||||
}
|
}
|
||||||
final int[] sortedSelectionIndices = sortSelctionIndicesFromTextView(mTextView);
|
final int[] sortedSelectionIndices = sortSelectionIndicesFromTextView(mTextView);
|
||||||
mSelectionTracker.onSelectionUpdated(
|
mSelectionTracker.onSelectionUpdated(
|
||||||
sortedSelectionIndices[0], sortedSelectionIndices[1], mTextClassification);
|
sortedSelectionIndices[0], sortedSelectionIndices[1], mTextClassification);
|
||||||
mTextClassificationAsyncTask = null;
|
mTextClassificationAsyncTask = null;
|
||||||
@@ -497,7 +497,7 @@ public final class SelectionActionModeHelper {
|
|||||||
private void resetTextClassificationHelper(int selectionStart, int selectionEnd) {
|
private void resetTextClassificationHelper(int selectionStart, int selectionEnd) {
|
||||||
if (selectionStart < 0 || selectionEnd < 0) {
|
if (selectionStart < 0 || selectionEnd < 0) {
|
||||||
// Use selection indices
|
// Use selection indices
|
||||||
int[] sortedSelectionIndices = sortSelctionIndicesFromTextView(mTextView);
|
int[] sortedSelectionIndices = sortSelectionIndicesFromTextView(mTextView);
|
||||||
selectionStart = sortedSelectionIndices[0];
|
selectionStart = sortedSelectionIndices[0];
|
||||||
selectionEnd = sortedSelectionIndices[1];
|
selectionEnd = sortedSelectionIndices[1];
|
||||||
}
|
}
|
||||||
@@ -639,7 +639,7 @@ public final class SelectionActionModeHelper {
|
|||||||
mAllowReset = false;
|
mAllowReset = false;
|
||||||
boolean selected = editor.selectCurrentWord();
|
boolean selected = editor.selectCurrentWord();
|
||||||
if (selected) {
|
if (selected) {
|
||||||
final int[] sortedSelectionIndices = sortSelctionIndicesFromTextView(textView);
|
final int[] sortedSelectionIndices = sortSelectionIndicesFromTextView(textView);
|
||||||
mSelectionStart = sortedSelectionIndices[0];
|
mSelectionStart = sortedSelectionIndices[0];
|
||||||
mSelectionEnd = sortedSelectionIndices[1];
|
mSelectionEnd = sortedSelectionIndices[1];
|
||||||
mLogger.logSelectionAction(
|
mLogger.logSelectionAction(
|
||||||
@@ -1216,7 +1216,7 @@ public final class SelectionActionModeHelper {
|
|||||||
|
|
||||||
SelectionResult(int start, int end,
|
SelectionResult(int start, int end,
|
||||||
@Nullable TextClassification classification, @Nullable TextSelection selection) {
|
@Nullable TextClassification classification, @Nullable TextSelection selection) {
|
||||||
int[] sortedIndices = sortSelctionIndices(start, end);
|
int[] sortedIndices = sortSelectionIndices(start, end);
|
||||||
mStart = sortedIndices[0];
|
mStart = sortedIndices[0];
|
||||||
mEnd = sortedIndices[1];
|
mEnd = sortedIndices[1];
|
||||||
mClassification = classification;
|
mClassification = classification;
|
||||||
|
|||||||
Reference in New Issue
Block a user