From b2a7b29612142f37b4f1df04ccbe1d561be15a92 Mon Sep 17 00:00:00 2001 From: "varga%nixcorp.com" Date: Sat, 29 Jul 2006 05:41:47 +0000 Subject: [PATCH] Fix for bug 221361. right click context speed is very slow with large selections r=jan,neil sr=alecf patch by gemal git-svn-id: svn://10.0.0.236/trunk@205711 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/suite/common/nsContextMenu.js | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/mozilla/suite/common/nsContextMenu.js b/mozilla/suite/common/nsContextMenu.js index 38cbe1b5bfc..0e8b3679446 100644 --- a/mozilla/suite/common/nsContextMenu.js +++ b/mozilla/suite/common/nsContextMenu.js @@ -784,7 +784,7 @@ nsContextMenu.prototype = { //selected text. Only use the first 15 chars. isTextSelection : function() { var result = false; - var selection = this.searchSelected(); + var selection = this.searchSelected(16); var bundle = srGetStrBundle("chrome://communicator/locale/contentAreaCommands.properties"); @@ -803,13 +803,21 @@ nsContextMenu.prototype = { return result; }, - searchSelected : function() { + searchSelected : function( charlen ) { var focusedWindow = document.commandDispatcher.focusedWindow; var searchStr = focusedWindow.__proto__.getSelection.call(focusedWindow); searchStr = searchStr.toString(); - searchStr = searchStr.replace( /^\s+/, "" ); - searchStr = searchStr.replace(/(\n|\r|\t)+/g, " "); - searchStr = searchStr.replace(/\s+$/,""); + // searching for more than 150 chars makes no sense + if (!charlen) + charlen = 150; + if (charlen < searchStr.length) { + // only use the first charlen important chars. see bug 221361 + var pattern = new RegExp("^(?:\\s*.){0," + charlen + "}"); + pattern.test(searchStr); + searchStr = RegExp.lastMatch; + } + searchStr = searchStr.replace(/\s*(.*?)\s*$/, "$1"); + searchStr = searchStr.replace(/\s+/g, " "); return searchStr; },