Closed Bug 1434925 Opened 5 years ago Closed 5 years ago

Crash in java.lang.IllegalArgumentException: width and height must be > 0 at


(Core Graveyard :: Widget: Android, defect)

Not set


(firefox-esr52 unaffected, firefox58 wontfix, firefox59+ fixed, firefox60+ fixed)

Tracking Status
firefox-esr52 --- unaffected
firefox58 --- wontfix
firefox59 + fixed
firefox60 + fixed


(Reporter: marcia, Assigned: jchen)


(Keywords: crash, regression)

Crash Data


(1 file)

This bug was filed from the Socorro interface and is
report bp-f9be70f7-2836-46f3-8b7e-62c9b0180201.

Seen while looking at release crash stats, but also present on other branches: In 58.0.1 this is the #11 overall browser crash. Almost 70% of those crashing crashed in under a minute. APIs from 16-27 are affected, and there is a wide spectrum of devices.

* Loading Firefox from external link (new tab) 
* just pasted and goed a link to a post on space battles. Started scrolling down and a brief freeze before crashing 
* I was just browsing Stack Overflow when it happened with some other tabs opened. 

Top 10 frames of crashing thread:

0 mozilla::jni::Accessor::EndAccess<mozilla::java::AndroidGamepadManager::OnGamepadAdded_t> 
1 mozilla::jni::Method<mozilla::java::LayerView::Compositor::RecvToolbarAnimatorMessage_t, void>::Call<int> widget/android/jni/Accessors.h:99
2 mozilla::java::LayerView::Compositor::RecvToolbarAnimatorMessage widget/android/GeneratedJNIWrappers.cpp:1340
3 nsWindow::LayerViewSupport::RecvToolbarAnimatorMessage widget/android/nsWindow.cpp:1095
4 nsWindow::RecvToolbarAnimatorMessageFromCompositor widget/android/nsWindow.cpp:2353
5 mozilla::layers::UiCompositorControllerChild::RecvToolbarAnimatorMessageFromCompositor gfx/layers/ipc/UiCompositorControllerChild.cpp:270
6 mozilla::layers::PUiCompositorControllerChild::OnMessageReceived ipc/ipdl/PUiCompositorControllerChild.cpp:379
7 mozilla::ipc::MessageChannel::DispatchAsyncMessage ipc/glue/MessageChannel.cpp:2119
8 mozilla::ipc::MessageChannel::DispatchMessage ipc/glue/MessageChannel.cpp:2049
9 mozilla::ipc::MessageChannel::AssertWorkerThread ipc/glue/MessageChannel.h:530

Jim, is this something you can take a look at?
Flags: needinfo?(nchen)
Sure. Looks like a simple fix.
Assignee: nobody → nchen
Flags: needinfo?(nchen)
wontfix for 58 giving timing. but maybe we should track for 59? pretty high volume.
Comment on attachment 8949482 [details]
Bug 1434925 - Skip drawing hidden toolbar elements;
Attachment #8949482 - Flags: review?(rbarker) → review+
Pushed by
Skip drawing hidden toolbar elements; r=rbarker
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Comment on attachment 8949482 [details]
Bug 1434925 - Skip drawing hidden toolbar elements;

Approval Request Comment
[Feature/Bug causing the regression]: n/a
[User impact if declined]: rare crash when using Fennec
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]: simple fix to avoid crash
[String changes made/needed]: none
Attachment #8949482 - Flags: approval-mozilla-beta?
Comment on attachment 8949482 [details]
Bug 1434925 - Skip drawing hidden toolbar elements;

Fairly low volume crash, but since the fix is verified in nightly, let's uplift for beta 11.
Attachment #8949482 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.