Closed
Bug 751609
Opened 13 years ago
Closed 13 years ago
Playing Flash video yields black screen -- Surface::lock failed, already locked messages
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox14 verified, firefox15 verified, blocking-fennec1.0 +)
VERIFIED
FIXED
Firefox 15
People
(Reporter: aaronmt, Assigned: snorp)
References
()
Details
(Keywords: regression)
Attachments
(2 files)
165.31 KB,
text/plain
|
Details | |
2.63 KB,
patch
|
blassey
:
review+
blassey
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
After a half dozen attempts to trigger the play button after activating plugins on Android.com, the entire viewport turns black and my log is continuously flooded with
E/Surface ( 9926): Surface::lock failed, already locked
--
Nightly (05/03), Aurora (05/03)
Samsung Galaxy Note (Android 2.3)
Reporter | ||
Comment 1•13 years ago
|
||
This leaves Fennec completely non-functional; new load attempts at new sites leave the throbber spinning and nothing happening, looping:
E/Surface ( 9926): Surface::lock failed, already locked
Comment 2•13 years ago
|
||
I suspect this is a regression from bug 727116, can QA please confirm that?
Assignee: nobody → snorp
Reporter | ||
Comment 3•13 years ago
|
||
Reporter | ||
Updated•13 years ago
|
Keywords: regression
Updated•13 years ago
|
No longer blocks: 750963
blocking-fennec1.0: ? → +
Summary: Playing Flash video on Android.com on the Galaxy Note yields black screen -- Surface::lock failed, already locked messages → Playing Flash video yields black screen -- Surface::lock failed, already locked messages
Assignee | ||
Comment 5•13 years ago
|
||
Attachment #621137 -
Flags: review?(blassey.bugs)
Assignee | ||
Comment 6•13 years ago
|
||
With the above patch, we block for a maximum of ~5ms and give up after that. The result is that you won't see the placeholder which looks nice during pan/zoom and instead the plugin SurfaceView will kind of lag behind. Sucks, but apparently there is something going on with the Note that causes a problem here.
Reporter | ||
Comment 7•13 years ago
|
||
Can reproduce on an inbound from this afternoon so 'the looper patch' didn't fix this.
Updated•13 years ago
|
Attachment #621137 -
Flags: review?(blassey.bugs) → review+
Assignee | ||
Comment 8•13 years ago
|
||
Comment 9•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
Comment 10•13 years ago
|
||
Comment on attachment 621137 [details] [diff] [review]
Give up on locking a Surface after 10 failures
[Triage Comment]
Attachment #621137 -
Flags: approval-mozilla-aurora+
Comment 11•13 years ago
|
||
Comment 12•13 years ago
|
||
This issue is not reproducible anymore on the latest Nightly and Aurora builds. Closing bug as verified fixed on:
Firefox 15.0a1 (2012-05-30)
Firefox 14.0a2 (2012-05-30)
Device: Galaxy Nexus
OS: Android 4.0.2
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•