455956 - embed/objects set width & height to 'auto' instead of default values of 240x200

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P3)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.1) Gecko/2008070208 Firefox/3.0.1
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b1pre) Gecko/20080916182923 Fennec/1.0a1pre

in Fx3.1 an embedded object which doesn't have a width/height defined has a value of 240x200.  In Fennec, this is set to auto.  The mochitests have 10 cases which fail in /tests/content/base/test/test_bug268972.html

I have created a simple html test case which will pop up an alert box if it receives auto for the width or height.  This will work fine in Fx3.1 and fail in Fennec

Reproducible: Always

Steps to Reproduce:
1. run attached test_fail10.html in fennec
2. observe two popups indicating 'auto' for width and height
3. run attached test_fail10.html in Firefox 3.1 and observe no popups
Actual Results:  
style.width returns auto
style.height returns auto

Expected Results:  
style.width return 240 if there is nothing set
style.height returns 200 if there is nothing set

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: Test case to exhibit default width/height on embedded objects

this will display a popup if it fails (i.e. has the value of 'auto' instead of 240px or 200px).

Comment 2

[Stuart Parmenter]

i have no idea why this would happen only in fennec.  anyone have any thoughts/ideas?

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Is this the missing plugin stuff?  There are some style rules for that that live in the theme; not sure if that's the right place for them.

See also bug 368972.

Comment 4

[:Gavin Sharp [email: [e-mail]]]

This is possibly related to bug 455866. Seems like the missing plugin styling (including the binding) just isn't being applied in Fennec for some reason (i.e. the rules in toolkit/mozapps/plugins/content/missingPluginBinding.css aren't matching).

Comment 5

[David Bengoa]

The patch that I am making for the bug 455866 also fix this issue.

Comment 6

[:Gavin Sharp [email: [e-mail]]]

Fixed by patch in bug 455866:
https://hg.mozilla.org/mobile-browser/rev/6bfcbaf1c420

Comment 7

[Aakash Desai [:aakashd]]

verified FIXED On builds: 

Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.2a2pre) Gecko/20090921 Fennec/1.0a3


and

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.2a2pre) Gecko/20090921
Fennec/1.0b4pre