diff options
author | Leon Scroggins <scroggo@google.com> | 2009-07-06 14:32:49 -0400 |
---|---|---|
committer | Leon Scroggins <scroggo@google.com> | 2009-07-07 10:07:17 -0400 |
commit | 43488fc78c927c25936e7b90030c5f10d2db94e5 (patch) | |
tree | 284c956b2306cd8f740de836a02fbd1d856104ed /core/java/android/webkit | |
parent | 6e2863ec16828b77c494592410e6b01281596e87 (diff) | |
download | frameworks_base-43488fc78c927c25936e7b90030c5f10d2db94e5.zip frameworks_base-43488fc78c927c25936e7b90030c5f10d2db94e5.tar.gz frameworks_base-43488fc78c927c25936e7b90030c5f10d2db94e5.tar.bz2 |
Update text webcore thread's text generation number.
When the user edits text in a textfield, we increase a generation
number so we can mark changes from webkit to be out of date. With
this change, update webcore's notion of the text generation number
in deleteSelection and replaceTextfieldText, in addition to
passToJs. Requires a change in external/webkit.
Diffstat (limited to 'core/java/android/webkit')
-rw-r--r-- | core/java/android/webkit/WebView.java | 8 | ||||
-rw-r--r-- | core/java/android/webkit/WebViewCore.java | 23 |
2 files changed, 26 insertions, 5 deletions
diff --git a/core/java/android/webkit/WebView.java b/core/java/android/webkit/WebView.java index b2f2e51..899d636 100644 --- a/core/java/android/webkit/WebView.java +++ b/core/java/android/webkit/WebView.java @@ -3024,7 +3024,12 @@ public class WebView extends AbsoluteLayout */ /* package */ void deleteSelection(int start, int end) { mTextGeneration++; - mWebViewCore.sendMessage(EventHub.DELETE_SELECTION, start, end); + WebViewCore.DeleteSelectionData data + = new WebViewCore.DeleteSelectionData(); + data.mStart = start; + data.mEnd = end; + data.mTextGeneration = mTextGeneration; + mWebViewCore.sendMessage(EventHub.DELETE_SELECTION, data); } /** @@ -4672,6 +4677,7 @@ public class WebView extends AbsoluteLayout arg.mNewStart = newStart; arg.mNewEnd = newEnd; mTextGeneration++; + arg.mTextGeneration = mTextGeneration; mWebViewCore.sendMessage(EventHub.REPLACE_TEXT, oldStart, oldEnd, arg); } diff --git a/core/java/android/webkit/WebViewCore.java b/core/java/android/webkit/WebViewCore.java index 44d8b7e..20f7239 100644 --- a/core/java/android/webkit/WebViewCore.java +++ b/core/java/android/webkit/WebViewCore.java @@ -370,7 +370,8 @@ final class WebViewCore { // Start: functions that deal with text editing private native void nativeReplaceTextfieldText( - int oldStart, int oldEnd, String replace, int newStart, int newEnd); + int oldStart, int oldEnd, String replace, int newStart, int newEnd, + int textGeneration); private native void passToJs(int gen, String currentText, int keyCode, int keyValue, boolean down, @@ -415,8 +416,10 @@ final class WebViewCore { * order, swap them. * @param start Beginning of selection to delete. * @param end End of selection to delete. + * @param textGeneration Text generation number when delete was pressed. */ - private native void nativeDeleteSelection(int start, int end); + private native void nativeDeleteSelection(int start, int end, + int textGeneration); /** * Set the selection to (start, end) in the focused textfield. If start and @@ -551,10 +554,17 @@ final class WebViewCore { byte[] mPostData; } + static class DeleteSelectionData { + int mStart; + int mEnd; + int mTextGeneration; + } + static class ReplaceTextData { String mReplace; int mNewStart; int mNewEnd; + int mTextGeneration; } static class TouchUpData { @@ -920,7 +930,8 @@ final class WebViewCore { case REPLACE_TEXT: ReplaceTextData rep = (ReplaceTextData) msg.obj; nativeReplaceTextfieldText(msg.arg1, msg.arg2, - rep.mReplace, rep.mNewStart, rep.mNewEnd); + rep.mReplace, rep.mNewStart, rep.mNewEnd, + rep.mTextGeneration); break; case PASS_TO_JS: { @@ -1029,7 +1040,11 @@ final class WebViewCore { break; case DELETE_SELECTION: - nativeDeleteSelection(msg.arg1, msg.arg2); + DeleteSelectionData deleteSelectionData + = (DeleteSelectionData) msg.obj; + nativeDeleteSelection(deleteSelectionData.mStart, + deleteSelectionData.mEnd, + deleteSelectionData.mTextGeneration); break; case SET_SELECTION: |