Closed Bug 735769 Opened 12 years ago Closed 12 years ago

Texturing of <video> elements in WebGL broken by bug 714408 (Video cube demo got broken after adding omx-plugin)

Categories

(Firefox OS Graveyard :: General, defect)

Product:
Firefox OS Graveyard
Component:
General
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: romaxa, Assigned: eflores)

References

Details

(Keywords: regression) 

Simple bisect endup with:
bad: [0724b0c0d886613840c1b979b1217bf94a7852cd] code cleanup, getting ready to use overlays
Disabling plugin prefs does not help, also does not help to disable in configure option...
only full backout of that change helps to make video cube working again
Strange. Can you try to poke at the code to see why this happens? I will see if I can help this weekend.
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #3)
> Where is this demo?

Chris Double had trouble loading this bug, so he passed along this info in irc:

"The demo mentioned is http://gaiamobile.org/apps/cubevid/
The source is at https://github.com/andreasgal/gaia/tree/master/apps/cubevid"

I appreciate the additional info, but can someone who has installed this patch comment on what the problem is?   Thanks.

(I also added Chris D to the Cc list of this bug.)
To reproduce this problem

 (1) Make a b2g build, on any phone
 (2) Launch the CubeVid demo on the first page of the gaia homescreen

You'll notice audio plays, but no pixels are displayed.  Prior to bug 714408 landing, a rotating 3d cube with <video> mapped to each face was visible.
Summary: Video cube demo got broken after adding omx-plugin → Texturing of <video> elements in WebGL broken by bug 714408 (Video cube demo got broken after adding omx-plugin)
Shouldn't be hard to fix. Let's fix it and land with 714408.
Tried reproducing this on a Samsung Galaxy S2 running Boot to Gecko with a Gingerbread base.

I had the opposite experience---without the patches in bug 714408, I would get sound but no video; with the patches, the video would play as well.

Are you still able to repro? If so, need more details about phone, the android version you started with, etc.
Er, scratch that. Just retested it: it works both before and after the patch.
Which m-c repository were you testing with?
This was on latest m-c out of hg-- not sure how it differs from the git gecko repo, if at all.
Argh, scratch that entirely. I mustn't have updated Gecko properly on the phone before. Broken for me as well now.
Is this still reproducible? On what phone?

I may have some time now to help with this if that's useful.
Yes, any b2g phone built from the configuration at https://github.com/andreasgal/b2g .  Not sure if the NZ folks have poked at this.
Stand down, Edwin figured this out.

The patches in bug 714408 reduced the media cache size a lot for B2G, which caused the "canplaythrough" event to not fire due to bug 755533. The video cube demo relies on "canplaythrough" firing. We'll fix that in bug 755533, and in the meantime, we've split out the cache size change to bug 755609.

So I'm actually going to mark this fixed.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.