302930 - sizeToContent() gives wrong size on full-page plugins

Status: RESOLVED INVALID

(Core :: Layout, defect)

Description

[Torben]

(Build id: Current trunk builds of Camino and Firefox)

If a page only hosts a full-page plugin sizeToContent() resizes the window so
the content is approx. 100 x 100 px no matter how big the plugin is.

Reproducible: Always

Steps to reproduce:
1. Load URL
2. Type javascript:sizeToContent() in the location bar and hit enter (you need
   to allow javascript to resize windows for this to work)

Actual Results:  the window resizes so the content is 100 X 100 px

Expected Results:  resize to the size of the plugin, or don't resize at all.

Since I don't think there is a way for us to get the actual size of the plugin,
I suggest we do nothing (thats way better than the current behaviour).

I have a proposed fix for this in bug 155956 attachment 191111 [details] [diff] [review] (scroll to the
bottom).

Comment 1

[Boris Zbarsky [:bzbarsky]]

Actually, I was pretty sure we had an existing bug on extending NPAPI to allow a
plugin to report an intrinsic size...

Comment 2

[Torben]

You're probably thinking of bug 70976. Fixing that would be better, but until
then I think we should just stop the window from resizing. We probably will need
to spwcial-case plugins anyway.

Comment 3

[Mats Palmgren (inactive)]

We're in the process of removing support for plugins (bug 1677160) and bug 1687239 has removed the relevant Layout code, so this bug is irrelevant now.