Closed Bug 1231915 Opened 4 years ago Closed 4 years ago

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

Categories

(Firefox for Android :: Toolbar, defect, critical)

ARM
Android
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Firefox 46
Tracking Status
firefox45 --- affected
firefox46 --- fixed

People

(Reporter: wgianopoulos, Assigned: kats)

References

Details

(Keywords: crash)

Crash Data

Attachments

(1 file)

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
(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.
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.
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?
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.
Attached patch PatchSplinter Review
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+
https://hg.mozilla.org/mozilla-central/rev/568f91710ba3
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
You need to log in before you can comment on or make changes to this bug.