Closed Bug 691175 Opened 8 years ago Closed 8 years ago

Unable to back out of Awesome screen


(Firefox for Android Graveyard :: General, defect)

Firefox 10
Not set


(firefox10 fixed)

Firefox 10
Tracking Status
firefox10 --- fixed


(Reporter: aaronmt, Assigned: mbrubeck)



(Keywords: regression, Whiteboard: [pushed])


(1 file)

Pressing my devices back button will not back out of the awesome screen; a recent regression.

1. New profile, start Fennec
2. Tap the Awesomebar
3. Tap your devices back button twice

ER: Back looking at about:home
AR: Stuck in the Awesome screen

Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111002 Firefox/10.0a1 Fennec/10.0a1

Samsung Galaxy SII
Samsung Nexus S
Under the tablet UI, the panel remains open but one can tap outside of it to dismiss it

Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111003 Firefox/10.0a1 Fennec/10.0a1

Samsung Galaxy Tab 10.1
Assignee: nobody → mbrubeck
Regression from bug 684558.  Specifically, this broke because I changed the BrowserUI "keypress" handler from capture=true to capture=false.
Blocks: 684558
Version: Trunk → Firefox 10
Attached patch patchSplinter Review
This fixes the bug, and does not regress bug 684558 or bug 683736.

This also backs out a change I made to browser_awesomescreen.js that I think caught this bug but I thought it was a bug in the test.  Oops.  :(  Tests are passing on desktop except for browser_awesomescreen which is crashing on my desktop build for some unrelated reason (with or without this patch).  Pushed to try to test on device:

It feels kind of hacky that we have to do this.  I'd like to file a followup bug on making the platform side of this better so that we don't need to do our own event re-dispatching (though I don't know exactly how that would work).  But this is no hackier than the code that was removed by bug 682017, so I think the best way forward is to stick with the platform event forwarding, fix this bug, and try to clean up the platform to eventually remove the need for these hacks.  (The alternative is to back out bug 682017 and all its dependencies and just go back to doing all the forwarding with JavaScript messages.)
Attachment #564230 - Flags: review?(mark.finkle)
Comment on attachment 564230 [details] [diff] [review]

>       case "keypress":
>+        // Ignore events headed toward the browser; they will be
>+        // re-dispatched after content has a chance to handle them.
>+        if ( == Browser.selectedBrowser)
>+          break;

I suppose keypresses would only be targeted to the selected browser and not any browser. If we think it could be any browser, maybe use | == "browser" | check instead
Attachment #564230 - Flags: review?(mark.finkle) → review+
Closed: 8 years ago
Resolution: --- → FIXED
Retested bug with:
Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111004 Firefox/10.0a1 Fennec/10.0a1

Samsung Nexus S (Android 2.3)
LG Optimus 2X (Android 2.2)

Bug is no longer reproducible. Back button is working properly after Awesome screen is displayed.

Verifying bug.
Depends on: 692123
Target Milestone: --- → Firefox 10
You need to log in before you can comment on or make changes to this bug.