895962 - defect - find bar not dismissed with CTRL-L

Status: VERIFIED FIXED

(Firefox for Metro Graveyard :: Browser, defect, P2)

Description

[juan becerra [:juanb]]

Tested on 2013-07-19 using latest nightly while doing regression testing for story in bug 867641.

I noticed that after having brought up and used the find bar, it wasn't dismissed when I then used the hardware keyboard to focus on the URL bar.

Steps:
1. Go to www.google.com
2. Swipe from the bottom to bring up the navigation app bar
3. Tap on the settings and select Find in Page
4. Type "google" and tap any of the arrows to cycle through the results
5. Using the hardware keyboard press ctrl-L

Expected: Find bar is dismissed and you can see the navigation bar where you can type a URL.

Actual: Find in bar is visible, and behind it you can see what should be the top navigation bar.

Comment 1

[Matt Brubeck (:mbrubeck)]

Attachment: find-blur

We previously fixed this (bug 771255) in appbar.js but that code has been removed:
https://hg.mozilla.org/users/mbrubeck_mozilla.com/elm-metro-firefox/rev/0867537267f6

There's also code that does this on MozContextUIShow, but that event is no longer what we want, and in fact we no longer have any use for the MozContextUIShow or Dismiss events.  This patch removes them and uses MozAppbarShowing instead.

Also adds a test for this bug, and changes some errant tabs to spaces.

Comment 2

[Rodrigo Silveira [:rsilveira] [:rodms]]

Comment on attachment 778596 [details] [diff] [review]
find-blur

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

Looks great.

::: browser/metro/base/content/helperui/FindHelperUI.js
@@ +56,5 @@
>  
>      // Listen for events where form assistant should be closed
>      Elements.tabList.addEventListener("TabSelect", this, true);
>      Elements.browsers.addEventListener("URLChanged", this, true);
> +    window.addEventListener('MozAppbarShowing', this);

Nit: use double quotes for consistency of change the other to single.

Comment 3

[Matt Brubeck (:mbrubeck)]

Pushed with nits fixed:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ed1a14f6ef85

was green on Try:
https://tbpl.mozilla.org/?tree=Try&rev=5c7401c108be

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/ed1a14f6ef85

Comment 5

[Manuela Muntean [Away]]

Mozilla/5.0 (Windows NT 6.2; rv:25.0) Gecko/20130728 Firefox/25.0

Verified as fixed on the latest Nightly build (on desktop) using the steps from the description. Find bar dismissed with CTRL-L.

This still needs verification for mobile devices.

Comment 6

[Samvedana (:Samvedana)]

User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Build ID: 20130826074752
Built from http://hg.mozilla.org/mozilla-central/rev/14b1e8c2957e

WFM
Tested on windows 8 using latest nightly for iteration-12. Followed steps provided in comment0 and got expected result.