41602 - plugins don't size to expand the box

Status: VERIFIED FIXED

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

Description

[David Ascher]

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.01; Windows NT 5.0)
BuildID:    2000042708

The following XUL demonstrates that the plugins don't size the way one would 
expect them to internally, even though they appear to claim the right amount of 
the parent box.  They don't resize well either.  Tested with the sample plugin 
and with our plugin.

Reproducible: Always
Steps to Reproduce:
Load XUL file below.  One would expect the plugin area to act much like the box 
on the left of the splitter.  

Actual Results:  The splitter is at 50% of the width, as expected.
The embed area is small, unlike what is expected (maybe 50px by 50px).
When the splitter is moved by dragging the grippy, the sizing is wrong (the 
splitter does not follow the mouse)

Expected Results:  A really boring window with a grey box on the right which 
grew as the splitter is dragged.

<?xml version="1.0"?>
<?xml-stylesheet href="chrome://global/skin" type="text/css"?>
<window
  xmlns:html="http://www.w3.org/TR/REC-html40"
  xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
>
<box orient="horizontal" flex="1" width="100"/>
<splitter state="open" collapse="before" resizebefore="closest" 
resizeafter="closest"/>
<html:embed type="netscape-plugin" flex="1"/%

Comment 1

[Jesse Ruderman]

this might be related to bug 37137

Comment 2

[Eric Vaughan]

This test case is invalid. Splitters are placed between 2 elements you wish to 
size not one. If you just have one you will indeed get very strange behavior. 
Try placing a spring with flex 1 to the right of the splitter.

Comment 3

[David Ascher]

I'm sorry, I don't understand.

The intended element structure in my test case (which was apparently truncated 
in the submission process was:

<box flex="1"/><splitter .../><html:embed ... flex="1"/>

How is that not putting a splitter between two elements?

In other words I want the embed to resize just like the box.

--david ascher

Comment 4

[Eric Vaughan]

This example seems to work:

<?xml version="1.0"?> 
<?xml-stylesheet href="chrome://global/skin/" type="text/css"?> 
<?xml-stylesheet href="chrome://communicator/skin/" type="text/css"?> 

<!DOCTYPE window> 

<window 
xmlns:html="http://www.w3.org/TR/REC-html40"
        xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"> 

	<box orient="horizontal" flex="1" width="100"/>
	<splitter/>
        <html:embed type="netscape-plugin" flex="1"/>
    
</window>

For me the plugin comes up and fills the right area. It places an icon in the 
upper left corner because this example does not specify it. Is there a live 
working plug in I can try?

Comment 5

[Eric Vaughan]

targeting

Comment 6

[John Morrison]

moving evaughan non-nsbeta3 bugs to Future milestone, per xptoolkit triage. 

Is this still a problem, or a misunderstanding? Is there a fuller testcase?

Comment 7

[David Ascher]

This problem doesn't occur with recent bugs.  Sorry for not updating this bug 
report earlier.  Should I close it, being the reporter?

Comment 8

[Braden]

David: Yup. If it's nolonger occurring, please mark it FIXED.

Comment 9

[David Ascher]

Problem no longer occurs.

Comment 10

[shrirang khanzode]

marking verified based on reporters' comment.