Created attachment 586275 [details] logcat.txt 1. go to menu -> more -> settings -> enable plugins -> Click to play 2. hit the back button and go to : https://settings.adobe.com 3. click on where it states "Tap here to activate plugins" 4. select Local Storage Expected: flash will continue on with the app Actual: flash app restarts Note: 1. Samsung Nexus S, 2.3.1, flash 11, 20120105 build 2. also the flash plugin will continue to the next screen if yes was selected for the enable plugins option.
This feels like a problem with the flash settings app. Possibly a dupe of bug 694546?
It's talking about the same portion, but I don't believe it's the same bug. In the bug 694546 it's stating that it doesn't work at all. I believe that it does work, but not when "Click to play" is enabled.
With click to play enabled, I can tap to activate the plugin, but I run into problems when I try to select local storage like you say in comment 0. Does the app work correctly if you just have enable plugins set to "yes"?
Yes, the app seems to work correctly if I just enable the plugins to yes and then restart fennec.
Okay, I'm seeing the same thing. I just discovered if I hit the back button after going to the busted local storage page, then click on local storage again it works. Maybe there's a problem with the order in which we're loading the plugins on the page?
Oh interesting. Is there a way to test order of load easier?
(In reply to Naoki Hirata :nhirata from comment #6) > Oh interesting. Is there a way to test order of load easier? Actually, I think this problem is caused by the same root issue that's causing bug 713080 - we're re-loading all the plugins when the plugin objects are tapped, even after they're loaded. I'm working on a patch now that will hopefully fix both these bugs.
Created attachment 586506 [details] [diff] [review] patch We were reloading the plugins every time the plugin object received a click event. Since we're playing all plugins when one is clicked, it's kind of tricky to remove the click event listeners, so I decided to just clear _pluginsToPlay and check its length when playAllPlugins gets called. I also decided to get rid of the addPluginClickCallback I mostly stole from desktop, since we don't need that generic helper function, and I just replaced it with a normal click listener. I think this will fix bug 713080 and bug 694546 as well. (I was hoping it would also fix bug 715740, but that doesn't seem to be the case.)
Comment on attachment 586506 [details] [diff] [review] patch Test switching between tabs with Flash to make sure we don't lose plugin info
This never got marked as fixed, but it made its way into mozilla-central: https://hg.mozilla.org/mozilla-central/rev/5971546ca35e
Comment on attachment 586506 [details] [diff] [review] patch [Approval Request Comment] User impact if declined: interactive flash objects (like embedded youtube videos) won't work Testing completed (on m-c, etc.): baked on m-c for a few days
Comment on attachment 586506 [details] [diff] [review] patch [Triage Comment] Mobile only - approved for Aurora.
Verified fixed on: Mozilla/5.0 (Android;Linux armv7l;rv:11.0a2)Gecko/20120130 Firefox/11.0a2 Fennec/11.0a2 Device: Samsung Galaxy S OS: Android 2.2 Mozilla/5.0 (Android;Linux armv7l;rv:12.0a1)Gecko/20120130 Firefox/12.0a1 Fennec/12.0a1 Device: Samsung Galaxy S OS: Android 2.2