2118 - OBJECTs degrade incorrectly

Status: VERIFIED FIXED

(Core Graveyard :: Plug-ins, defect, P1)

Description

[Antti Näyhä]

NGLayout nightly build (Dec-31) degrades OBJECT elements incorrectly.  For
example: if the OBJECT has width=100 and height=100, the alternative content is
also forced into this space.  The size of the alternative content should *not*
be dependent in any way on the OBJECT element's width and height attributes.
See the URL for several examples of this behaviour and the problems it causes.

Comment 1

[bkdelong]

Don't forget to implement the attribute TABINDEX for element OBJECT.

Comment 2

[leger]

Setting all current Open Critical and Major to M3

Comment 3

[Paul MacQuiddy]

per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 4

[av (gone)]

I think we can consider this fixed as to original incorrect behaviour.
Should be in after 4.30pm March 10 build.

Comment 5

[rubydoo123]

attempted to access the above URI, selecting any of the test from that page and
the 3/16, apprunner build crashes:
APPRUNNER caused an invalid page fault in
module RAPTORHTML.DLL at 014f:01059675.
Registers:
EAX=00000000 CS=014f EIP=01059675 EFLGS=00010246
EBX=009ec170 SS=0157 ESP=0076fb48 EBP=0076fb9c
ECX=0076fb8c DS=0157 ESI=0109d878 FS=0e8f
EDX=816be444 ES=0157 EDI=008f6a90 GS=0000

Reopening bug until the crashing is resolved. I suspect we have more OBJECT
element related issues.

Comment 6

[av (gone)]

Could be not OBJECT tag related.  I noticed, it does not crash on all tests
except #4 if the Viewer window is large enough. Just maximize it first. Maybe
general layout problem. I am adding troy and nisheeth to the cc list.

Comment 7

[Alex Musil]

Crashing in: nsCSSFrameConstructor::CantRenderReplacedElement()

Comment 8

[Alex Musil]

Bumping to M4

Comment 9

[av (gone)]

I do not see crash any more

Comment 10

[Antti Näyhä]

All five tests still crash my April 5th Win32 build.
Changing Severity to "critical".

Comment 11

[av (gone)]

My April 5th build does not crash on either test. I am using checked build
though.

Comment 12

[Antti Näyhä]

My apologies, it seems that I was using an old March build by mistake.
The extracting directory in the new nightly .zip's seems to have changed,
which confused me.

My April 6th build _does_ pass the alternative renderings test - it's actually
the first OBJECT-supporting browser to do so. Congrats to everyone involved!

Comment 13

[Alex Musil]

Awesome!

Comment 14

[rubydoo123]

will verify when 5363 gets fixed

Comment 15

[rubydoo123]

marking as verified, using the 1999042608 build on winNT and checked on win95
too.