crash in java.lang.IllegalStateException: APZCCallbackHandler::NotifyDefaultPrevented should be getting called, not this! at org.mozilla.gecko.gfx.NativePanZoomController.notifyDefaultActionPrevented$1385ff(NativePanZoomController.java)

RESOLVED FIXED in Firefox 46

Status

()

--
critical
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: wgianopoulos, Assigned: kats)

Tracking

({crash})

Trunk
Firefox 46
ARM
Android
crash
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox45 affected, firefox46 fixed)

Details

(crash signature)

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
This bug was filed from the Socorro interface and is 
report bp-d2c75b73-5e00-481c-993d-f913b2151211.
=============================================================
Now that the fix for bug 1230522 has landed, I am now crashing here.  This happens when I use about:config and search, or about:crashes and try to view a crash report.
This should be fixed by bug 1230838. However - do you have accessibility enabled, or using any sort of non-touch input? That's the only scenario under which this should happen, so if that's not the case then I'd like to understand why.
Depends on: 1230838
(Reporter)

Comment 2

3 years ago
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #1)
> This should be fixed by bug 1230838. However - do you have accessibility
> enabled, or using any sort of non-touch input? That's the only scenario
> under which this should happen, so if that's not the case then I'd like to
> understand why.

As far as I know i have no alternate input devices or accessibility options enabled that should impact this.
(Reporter)

Comment 3

3 years ago
This crash occurs if you disable APZ form about config, which I had done yesterday to get around the previous crash.
Ah, on Fennec unfortunately you can't disable APZ just by flipping the pref, there is some build-time stuff too. You would have to do a local build with a modified confvars.sh to disable it. That explains why this crash was happening, thanks.
(Reporter)

Comment 5

3 years ago
So, we should guard somehow against people changing this.  There should not be visible preferences that changing them guarantees an inoperable product.  perhaps ifdef's around where it is defined?
(Reporter)

Comment 6

3 years ago
The major problem here is that once you get into this state you cannot use about:config to undo the change.
Once bug 1230838 lands you should be able to use about:config again. But yeah we should probably prevent people from flipping this somehow.
Created attachment 8697771 [details] [diff] [review]
Patch

This ignores the pref if MOZ_ANDROID_APZ is enabled, so flipping the pref shouldn't result in some weird hybrid configuration.
Assignee: nobody → bugmail.mozilla
Attachment #8697771 - Flags: review?(rbarker)
Attachment #8697771 - Flags: review?(rbarker) → review+

Comment 10

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/568f91710ba3
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
You need to log in before you can comment on or make changes to this bug.