Bug 611466 - Fennec highlights the whole URL after I've entered a special character ('~'), inviting me to overwrite it (missed some changes) [r=vingtetun]
authorMark Finkle <mfinkle@mozilla.com>
Wed, 22 Dec 2010 15:31:12 -0500
changeset 67160 c0ab7b3eaa08fe7e071e4a80b03078162a3ef11f
parent 67159 5b544d1ed1126c6fba82de36a89a11ec22f7e5dd
child 67161 6f20af7c60854d376d4694ad760342e21886a223
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersvingtetun
bugs611466
Bug 611466 - Fennec highlights the whole URL after I've entered a special character ('~'), inviting me to overwrite it (missed some changes) [r=vingtetun]
mobile/chrome/content/bindings.xml
--- a/mobile/chrome/content/bindings.xml
+++ b/mobile/chrome/content/bindings.xml
@@ -38,19 +38,16 @@
           val ? this.setAttribute("readonly", "true")
               : this.removeAttribute("readonly");
 
           // This is a workaround needed to cycle focus for the IME state
           // to be set properly (bug 488420)
           input.blur();
           input.focus();
 
-          if (val)
-            input.selectionStart = input.selectionEnd = input.textLength;
-
           return val;
         ]]></setter>
       </property>
 
       <method name="openPopup">
         <body><![CDATA[
           this.popup.openAutocompletePopup(this, null);
         ]]></body>
@@ -108,16 +105,22 @@
 
       <handler event="blur" phase="capturing">
         <![CDATA[
           // Bug 583341 - suppress disconnect of autocomplete controller
           this._dontBlur = true;
         ]]>
       </handler>
 
+      <handler event="TapDouble" phase="capturing">
+        <![CDATA[
+          this.select();
+        ]]>
+      </handler>
+
       <handler event="TapLong" phase="capturing">
         <![CDATA[
           let box = this.inputField.parentNode;
           box.showContextMenu(this, true);
         ]]>
       </handler>
     </handlers>
   </binding>