Note: There are a few cases of duplicates in user autocompletion which are being worked on.

The Context Menu and Awesomelist are dismissed in the same time, if a tap is performed outside the Context Menu UI

VERIFIED FIXED in Firefox 10

Status

Fennec Graveyard
General
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: xti, Assigned: lucasr)

Tracking

({regression})

Trunk
Firefox 10
ARM
Android
regression
Bug Flags:
in-testsuite +

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Reporter)

Description

6 years ago
Build ID: Mozilla/5.0 (Android;Linux armv7l;rv:9.0a2)Gecko/20111004
Firefox/9.0a2 Fennec/9.0a2
Device: Samsung Galaxy S
OS: Android 2.2

Build ID: Mozilla/5.0 (Android;Linux armv7l;rv:10.0a1)Gecko/20111004
Firefox/10.0a1 Fennec/10.0a1
Device: Samsung Galaxy S
OS: Android 2.2

Steps to reproduce:
1. Open Fennec App
2. Tap on URL Bar for Awesomescreen
3. Long tap on any page for the Context Menu
4. Tap outside the Context Menu UI to dismiss it

Expected result:
After step 4, the Context Menu is dismissed and the Awesomescreen remains on the screen.

Actual result:
After step 4, the Context Menu and the Awesomescreen are dismissed in the same time.
Assignee: nobody → lucasr.at.mozilla
Aurora is also affected
Mozilla/5.0 (Android; Linux armv7l; rv:9.0a2) Gecko/20111005 Firefox/9.0a2 Fennec/9.0a2
status-firefox10: --- → affected
status-firefox9: --- → affected
Keywords: regression, regressionwindow-wanted
(Assignee)

Comment 2

6 years ago
Created attachment 564856 [details] [diff] [review]
Keep awesome panel open when context menu is dismissed
Attachment #564856 - Flags: review?(mark.finkle)
Comment on attachment 564856 [details] [diff] [review]
Keep awesome panel open when context menu is dismissed

Why isn't handleEscape handling this correctly?

http://mxr.mozilla.org/mozilla-central/source/mobile/chrome/content/browser-ui.js#850
(In reply to Mark Finkle (:mfinkle) from comment #3)
> Why isn't handleEscape handling this correctly?

This bug happens when the popup is dismissed by tapping, not when it's dismissed by the escape key.
Comment on attachment 564856 [details] [diff] [review]
Keep awesome panel open when context menu is dismissed

* _hasPopup -> _popupShowing
* Should we worry that _popupShowing is not ref counted and could be messed up by more than 1 popup open?

r+, but fix the nit and think about the question

do we have any tests for this situation?
Attachment #564856 - Flags: review?(mark.finkle) → review+
(Reporter)

Comment 6

6 years ago
This issue doesn't occur on:
Build ID : Mozilla /5.0 (Android;Linux armv7l;rv:9.0a1) Gecko/20110902 Firefox/9.0a1 Fennec/9.0a1 
http://hg.mozilla.org/mozilla-central/rev/0664108eb19d

but it occurs on:
Build ID : Mozilla /5.0 (Android;Linux armv7l;rv:9.0a1) Gecko/20110903 Firefox/9.0a1 Fennec/9.0a1
http://hg.mozilla.org/mozilla-central/rev/472716252ea3

Possible range:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=0664108eb19d&tochange=472716252ea3
(Assignee)

Comment 7

6 years ago
(In reply to Mark Finkle (:mfinkle) from comment #5)
> Comment on attachment 564856 [details] [diff] [review] [diff] [details] [review]
> Keep awesome panel open when context menu is dismissed
> 
> * _hasPopup -> _popupShowing
> * Should we worry that _popupShowing is not ref counted and could be messed
> up by more than 1 popup open?
> 
> r+, but fix the nit and think about the question

My understanding of popup handling in browser is that there's always only one popup open at a time. When a new popup is pushed, the previous one is dismissed. So, we only need to care if there's a popup on top of awesome panel or not.

> do we have any tests for this situation?

No. Writing one now.
(Assignee)

Comment 8

6 years ago
Created attachment 565207 [details] [diff] [review]
(1/3) Keep awesome panel open when context menu is dismissed

Same patch with nits fixed. Keeping the review+.
Attachment #564856 - Attachment is obsolete: true
Attachment #565207 - Flags: review+
(Assignee)

Comment 9

6 years ago
Created attachment 565208 [details] [diff] [review]
(2/3) Ensure awesome panel is closed before each awesome screen test
Attachment #565208 - Flags: review?(mark.finkle)
(Assignee)

Comment 10

6 years ago
Created attachment 565209 [details] [diff] [review]
(3/3) Add browser test for popup being dismissed on top of awesome panel
Attachment #565209 - Flags: review?(mark.finkle)
Attachment #565208 - Flags: review?(mark.finkle) → review+
Comment on attachment 565209 [details] [diff] [review]
(3/3) Add browser test for popup being dismissed on top of awesome panel

Just add a comment about using setTimeout to simulate a long tap. We try to avoid using setTimeout with non-zero values, but in this case it's kinda needed.
Attachment #565209 - Flags: review?(mark.finkle) → review+
(Assignee)

Comment 12

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/e5cfda41f2e8
http://hg.mozilla.org/integration/mozilla-inbound/rev/2d17b3815e1c
http://hg.mozilla.org/integration/mozilla-inbound/rev/4749a3d2d66b
Target Milestone: --- → Firefox 10

Updated

6 years ago
Flags: in-testsuite+
Oops, a little bit too early to set that flag yet.
Flags: in-testsuite+
No, you can go ahead and set in-testsuite+ when pushing to inbound -
https://wiki.mozilla.org/Inbound_Sheriff_Duty
Flags: in-testsuite+
Keywords: regressionwindow-wanted
https://hg.mozilla.org/mozilla-central/rev/e5cfda41f2e8
https://hg.mozilla.org/mozilla-central/rev/2d17b3815e1c
https://hg.mozilla.org/mozilla-central/rev/4749a3d2d66b
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
This issue is verified fixed on Nightly build:
Mozilla/5.0 (Android;Linux armv7l;rv:10.0a1)Gecko/20111009 Firefox/10.0a1 Fennec/10.0a1
Device: HTC Desire
OS: Android 2.2
status-firefox10: affected → fixed

Updated

6 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.