518390 - n900: selecting a character from symbol map clears awesome bar

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Aakash Desai [:aakashd]]

Build Id:

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20090923
Fennec/1.0b4pre

Steps to Reproduce:
1. Go to the url bar and try to type in http://people.mozilla.org/~mfinkle/size-content.html

Actual Results:
When you select the "~" in the symbol map, fennec will clear the awesome bar and go back to the browser

Expected Results:
The tilde should show up and the cursor should still be blinking in the url bar.

Comment 1

[Ben Combee (:bcombee)]

The popup symbol list also doesn't work for fields embedded in web pages.  It gets dismissed, but no character is inserted into the field.

Comment 2

[Ben Combee (:bcombee)]

n810 "Chr" popup doesn't cause this bug; I suspect its due to focus.  On the n900, the awesome bar remains on screen while the symbols are displayed, its only on return from the dialog that it's dismissed.

Comment 3

[Ben Combee (:bcombee)]

Another observation: clicking outside symbol dialog to dismiss also causes aweseome bar dismissal.

Comment 4

[Ben Combee (:bcombee)]

I didn't get far into what else could be happening while debugging this, someone should look at this before I return on Nov 6th.

Comment 5

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 410582 [details] [diff] [review]
patch

Let's remove handleEvent entirely

>diff --git a/chrome/content/bindings.xml b/chrome/content/bindings.xml
>     <implementation implements="nsIAutoCompletePopup, nsIDOMEventListener">

Remove nsIDOMEventListener

>       <constructor><![CDATA[
>-        window.addEventListener("blur", this, true);
>       ]]></constructor>

Remove <constructor> since we don't use it

>-      <!-- Handles blur events on the window while the popup is open. -->
>       <method name="handleEvent">

>       </method>

Let's remove handleEvent altogether since we no longer use it

Comment 7

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch v.2, remove everything

Comment 8

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 410607 [details] [diff] [review]
patch v.2, remove everything

>     <implementation implements="nsIAutoCompletePopup, nsIDOMEventListener">

Remove "nsIDOMEventListener" too

Comment 9

[Brad Lassey [:blassey] (use needinfo?)]

pushed http://hg.mozilla.org/mobile-browser/rev/061d3ab4cc54

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

The bug is not fixed yet. The patch keeps the awesomebar from being dismissed, but no symbols are inserted into the urlbar

Comment 11

[Mark Finkle (:mfinkle) (use needinfo?)]

Leaving this bug as fixed - the awesomebar is not cleared anymore. Filing a new bug for the symbol editor not working.

Comment 12

[Al Billings [:abillings - ex-MoCo]]

The awesomebar isn't dismissed anymore, as noted, but, yeah, you can't enter a tilde into the field. 

What is the new bug number for that?