wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 767626 - We don't need to be caching the selectedText. r=mbrubeck

Browse Source
This commit is contained in:
Margaret Leibovic 2012-06-23 15:46:21 -07:00
parent c23b5f06de
commit c92853bb54
1 changed files with 33 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
mobile/android/chrome/content/browser.js
View File

@ -1358,7 +1358,6 @@ var NativeWindow = {
var SelectionHandler = {
// Keeps track of data about the dimensions of the selection
cache: null,
selectedText: "",
// The window that holds the selection (can be a sub-frame)
get _view() {
@ -1425,9 +1424,6 @@ var SelectionHandler = {
this._view = aElement.ownerDocument.defaultView;
this._isRTL = (this._view.getComputedStyle(aElement, "").direction == "rtl");
// Clear out the text cache
this.selectedText = "";
// Remove any previous selected or created ranges. Tapping anywhere on a
// page will create an empty range.
let selection = this._view.getSelection();
@ -1457,24 +1453,17 @@ var SelectionHandler = {
return;
}
// Find the selected text rect and send it back so the handles can position correctly
if (selection.rangeCount == 0 || !selection.getRangeAt(0))
// If there isn't an appropriate selection, bail
if (!selection.rangeCount || !selection.getRangeAt(0) || !selection.toString().trim().length) {
selection.collapseToStart();
return;
}
// Initialize the cache
this.cache = {};
this.updateCacheForSelection();
this.updateCacheOffset();
// Cache the selected text since the selection might be gone by the time we get the "end" message
this.selectedText = selection.toString().trim();
// If the range didn't have any text, let's bail
if (!this.selectedText.length) {
selection.collapseToStart();
return;
}
this.showHandles();
},
@ -1588,8 +1577,18 @@ var SelectionHandler = {
// aX/aY are in top-level window browser coordinates
endSelection: function sh_endSelection(aX, aY) {
this.hideHandles();
this._view.getSelection().removeAllRanges();
let selectedText = "";
if (this._view) {
let selection = this._view.getSelection();
if (selection) {
selectedText = selection.toString().trim();
selection.removeAllRanges();
}
}
// Only try copying text if there's text to copy!
if (selectedText.length) {
let contentWindow = BrowserApp.selectedBrowser.contentWindow;
let element = ElementTouchHelper.elementFromPoint(contentWindow, aX, aY);
if (!element)
@ -1606,13 +1605,13 @@ var SelectionHandler = {
aX - this.cache.offset.x + scrollX.value < this.cache.rect.right) &&
(aY - this.cache.offset.y + scrollY.value > this.cache.rect.top &&
aY - this.cache.offset.y + scrollY.value < this.cache.rect.bottom);
if (pointInSelection && this.selectedText.length) {
if (pointInSelection) {
let clipboard = Cc["@mozilla.org/widget/clipboardhelper;1"].getService(Ci.nsIClipboardHelper);
clipboard.copyString(this.selectedText);
clipboard.copyString(selectedText);
NativeWindow.toast.show(Strings.browser.GetStringFromName("selectionHelper.textCopied"), "short");
}
}
}
this._isRTL = false;
this._view = null;
@ -1746,10 +1745,6 @@ var SelectionHandler = {
this._touchId = null;
this._touchDelta = null;
// Update the cached selected text
let selection = this._view.getSelection();
this.selectedText = selection.toString().trim();
// Adjust the handles to be in the correct spot relative to the text selection
this.positionHandles();
break;