The default bug view has changed. See this FAQ.

Fix deadlocks caused by Flash

RESOLVED FIXED

Status

()

Firefox for Android
General
P1
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: snorp, Assigned: snorp)

Tracking

unspecified
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(1 attachment)

Created attachment 574334 [details] [diff] [review]
Request java plugin surface asynchronously (and other small fixes)

Gecko does a synchronous round trip to request the plugin surface from the main Android thread. This deadlocks frequently. We should just make it asynchronous, since it's not imperative that the plugin is displayed on the very first paint.
Assignee: nobody → snorp
Blocks: 695452
Attachment #574334 - Flags: review?(blassey.bugs)

Updated

5 years ago
Priority: -- → P1
Comment on attachment 574334 [details] [diff] [review]
Request java plugin surface asynchronously (and other small fixes)

Review of attachment 574334 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/plugins/base/nsNPAPIPluginInstance.h
@@ +49,4 @@
>  #include "nsIChannel.h"
>  #include "nsInterfaceHashtable.h"
>  #include "nsHashKeys.h"
> +#include "nsIRunnable.h"

put this in a #ifdef ANDROID
Attachment #574334 - Flags: review?(blassey.bugs) → review+
https://hg.mozilla.org/projects/birch/rev/924e2318089c
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
You need to log in before you can comment on or make changes to this bug.