Closed
Bug 598363
Opened 13 years ago
Closed 12 years ago
Context menu cannot be dismissed after opening a link in a new tab.
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(fennec2.0b3+)
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
fennec | 2.0b3+ | --- |
People
(Reporter: nhirata, Assigned: mbrubeck)
Details
(Whiteboard: [fennec-checkin-postb2])
Attachments
(2 files)
6.74 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
687 bytes,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Maemo;Linux armv71; rv:2.0b7pre)Gecko/20100921 Firefox/4.0b7pre Fennec/2.0b1pre Mozilla/5.0 (Android; Linux armv71; rv2.0b7pre) Gecko/20100921 Firefox/4.0b7pre Fennec/2.0b1pre 1. go to url: http://people.mozilla.com/~nhirata/ 2. long tap on html_tp link 3. select : open in a new tab in the context menu 4. long tap on html_tp link again 5. click outside the context menu Expected: Context menu dismisses Actual: context menu does not dismiss.
![]() |
Reporter | |
Updated•13 years ago
|
OS: Mac OS X → All
Hardware: x86 → ARM
![]() |
Reporter | |
Updated•13 years ago
|
tracking-fennec: --- → ?
![]() |
Reporter | |
Updated•13 years ago
|
Flags: in-litmus?(nhirata.bugzilla)
![]() |
Reporter | |
Comment 1•13 years ago
|
||
It looks like this has to do with either memory or a page loading at the same time. It's harder to reproduce now, you need to do step 1 ~ 3 multiple times to have around 8 tabs or so to get this issue to occur. It still does occur on both. May need to take a relook after bug 598331 lands.
![]() |
Reporter | |
Comment 2•13 years ago
|
||
litmus test case created : https://litmus.mozilla.org/show_test.cgi?searchType=by_id&id=13540
Flags: in-litmus?(nhirata.bugzilla) → in-litmus+
Updated•13 years ago
|
Assignee: nobody → mbrubeck
Comment 3•13 years ago
|
||
Matt - can you repo this?
Comment 4•13 years ago
|
||
in order to reproduce this, I need to have the tab bar hidden and open the context menu while the "New tab opened" alert is visible
tracking-fennec: ? → 2.0+
Updated•13 years ago
|
tracking-fennec: 2.0+ → 2.0b3+
Assignee | ||
Updated•13 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•13 years ago
|
||
ContextHelper.hide() is called twice: Once when NewTabPopup.show calls BrowserUI.pushPopup, and once by the context-commands onclick handler. The second time it's called, its popPopup pops the wrong popup. This patch fixes this in two ways. First, it fixes ContextHelper.hide() so it doesn't do anything if the menu is already hidden. That's enough to fix this bug, but this could easily happen again if some other code calls popPopup() while its popup is no longer pushed. To prevent this, I also added an argument to popPopup() so it can check that the right popup is popped.
Attachment #487076 -
Flags: review?(mark.finkle)
Comment 6•13 years ago
|
||
Comment on attachment 487076 [details] [diff] [review] patch Nice
Attachment #487076 -
Flags: review?(mark.finkle) → review+
Updated•13 years ago
|
Whiteboard: [fennec-checkin-postb2]
Assignee | ||
Comment 7•12 years ago
|
||
http://hg.mozilla.org/mobile-browser/rev/df1c79eb047e
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 8•12 years ago
|
||
A bug in the patch is causing an uncaught exception when showing a notification on desktop/Maemo.
Attachment #487468 -
Flags: review?(mark.finkle)
Updated•12 years ago
|
Attachment #487468 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 9•12 years ago
|
||
followup pushed: http://hg.mozilla.org/mobile-browser/rev/8d0f0009361d
Updated•12 years ago
|
Whiteboard: [fennec-checkin-postb2]
Comment 10•12 years ago
|
||
verified fixed on: Mozilla/5.0(Android; Linux armv7l;rv2.0b8pre) Gecko/20101102 Firefox/4.0b8pre Fennec/4.0b3pre And Mozilla/5.0(Maemo; Linux armv7l;rv2.0b8pre) Gecko/20101102 Firefox/4.0b8pre Fennec/4.0b3pre
Status: RESOLVED → VERIFIED
Whiteboard: [fennec-checkin-postb2]
You need to log in
before you can comment on or make changes to this bug.
Description
•