Regression: Context menus do not pop up on links

VERIFIED FIXED

Status

defect
--
major
VERIFIED FIXED
9 years ago
8 years ago

People

(Reporter: aakashd, Assigned: stechz)

Tracking

({regression})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Reporter

Description

9 years ago
Build Id:
Mozilla/5.0 (Android; Linux armv71; rv:2.0b10pre) Gecko/20100201 Firefox/4.0b10pre Fennec/4.0b4pre

Steps to Reproduce:
1. Go to bugzilla.mozilla.org
2. Tap and hold on one of the bugzilla icons

Actual Results:
Context menu does not pop up

Expected Results:
Context menu pops up
Reporter

Comment 1

9 years ago
This is working on Mozilla/5.0 (Macintosh; Intel Mac OS X 10.5; rv:2.0b11pre) Gecko/20110131 Firefox/4.0b11pre Fennec/4.0b5pre ID:20110131120454
OS: All → Android
Hardware: All → ARM
Posted patch patchSplinter Review
Regression from bug 630145. The code must be throwing an exception which stops the rest of the content.js file objects from initializing.

Firefox desktop wraps this in a try/catch, now we do too.
Assignee: nobody → mark.finkle
Attachment #508808 - Flags: review?(ben)
Assignee

Comment 3

9 years ago
Assigning bug to myself at the request of Mark.
Assignee: mark.finkle → ben
Status: NEW → ASSIGNED
useGlobalHistory will init the nsIGlobalHistory2 service, which is actually, nsNavHistory.cpp and that tries to do something and fails
Assignee

Comment 5

9 years ago
Comment on attachment 508808 [details] [diff] [review]
patch

r+ with a followup to remove try-catch in bug 630593.
Attachment #508808 - Flags: review?(ben) → review+
pushed:
http://hg.mozilla.org/mobile-browser/rev/6987c7f6e252
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED

Comment 7

8 years ago
Verified on build: Mozilla /5.0 (Android;Linux armv7l;rv:2.2a1pre)
Gecko/20110406 Firefox/4.2a1pre Fennec /4.1a1pre 
Device: Samsung Captivate (Android 2.2 update 1)
Status: RESOLVED → VERIFIED
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.