943735 - Defect - Typing backspace in the urlbar should de-select the selected autocomplete tile

Status: VERIFIED FIXED

(Firefox for Metro Graveyard :: App Bar, defect, P2)

Description

[Matt Brubeck (:mbrubeck)]

Attachment: patch

Steps to reproduce:
1. Begin typing in the urlbar
2. Press tab or down to select an autocomplete tile
3. Press backspace
4. Press enter

Expected results: The autocomplete tile is de-selected after step 3.  After step 4, the address in the urlbar loads.

Actual results: The autocomplete tile is still selected after step 3.  After step 4, the address of the selected tile loads.

This happens because our autocomplete popup has broken selectedIndex and selectedItem setters.

Comment 1

[Matt Brubeck (:mbrubeck)]

Attachment: patch v2

Now with an automated test!

Comment 2

[Marina Samuel [:emtwo]]

Comment on attachment 8339054 [details] [diff] [review]
patch v2

Review of attachment 8339054 [details] [diff] [review]:
-----------------------------------------------------------------

I tried this out and it works well for autocomplete de-selecting, but it doesn't de-select the bing/google/yahoo tiles if you tab to those. Is this something we want to do as well?

Comment 3

[Matt Brubeck (:mbrubeck)]

(In reply to Marina Samuel [:emtwo] from comment #2)
> I tried this out and it works well for autocomplete de-selecting, but it
> doesn't de-select the bing/google/yahoo tiles if you tab to those. Is this
> something we want to do as well?

Ah, good catch.  I'm not sure what we should do for that case.  On the one hand having the same behavior would be good for consistency; on the other hand the interaction is different.  (The search section always updates to reflect what you are typing.)  I'm going to land this basic fix for now, and we can file a new bug if we find the search tiles are a problem.

https://hg.mozilla.org/integration/fx-team/rev/61dfe6cb955d

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/61dfe6cb955d

Comment 5

[Cornel Ionce [:noni] [Hubs QA]]

Mozilla/5.0 (Windows NT 6.3; rv:29.0) Gecko/20100101 Firefox/29.0
Mozilla/5.0 (Windows NT 6.3; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0

Verified as fixed on latest Nightly (build ID: 20131215030202) using the the STR from comment 0 and comment 2 and got the expected results.