189581 - disabling auto image resizing does not take effect immediately

Status: RESOLVED WONTFIX

(Core :: Layout: Images, Video, and HTML Frames, enhancement, P4)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

0. make sure auto image resizing is on
1. load an image, e.g. http://dilbert.gemal.dk
2. make the browser window small enough that the image gets scaled
3. disable auto image resizing
4. notice that the image is still not the real size. works after a reload, though.

Comment 1

[Hixie (not reading bugmail)]

I suggest WONTFIX. We should remove the pref anyway. See bug 190545.

Comment 2

[Brian 'netdragon' Bober]

The UI for the pref was removed (not checked in yet), but there are now ways to
change the pref without the actual UI in Edit > Preferences. I will see if that
still shows this issue.

Comment 3

[Samir Gehani]

adt: nsbeta1-

Comment 4

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

Attachment: patch (untested)

Local problems kept me from testing this.  Missing the boat for 1.8b2 :(

Comment 5

[Christian :Biesinger (don't email me, ping me on IRC)]

+    CheckOverflowing(false);

PR_FALSE?

Comment 6

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

Comment on attachment 179818 [details] [diff] [review]
patch (untested)

(In reply to comment #5)
> +    CheckOverflowing(false);
> 
> PR_FALSE?

Something more complicated is going on here - I'm not changing enough state.

Comment 7

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

Now that bug 197263 is fixed, does this matter?

Comment 8

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

(In reply to comment #7)
> Now that bug 197263 is fixed, does this matter?
> 

jst agreed that this is now irrelevant.