585371 - Regression: "View Image" -> (automatic) resizing of (large) pictures not working anymore

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[Christian Ascheberg]

User-Agent:       Mozilla/5.0 (Windows NT 6.1; rv:2.0b4pre) Gecko/20100807 Minefield/4.0b4pre
Build Identifier: Mozilla/5.0 (Windows NT 6.1; rv:2.0b4pre) Gecko/20100807 Minefield/4.0b4pre

See steps below. The given URL is just one possible testcase, if your browser window is small enough. I need to reload the image to get the functionality working again (CTRL+R).

Reproducible: Always

Steps to Reproduce:
1. visit a website showing a large picture
2. right click on image, "View Image" (image needs to be bigger than browser window)
Actual Results:  
The image is shown at original size, image is not resized to width of browser window. Clicking on the image has no effect, no icon shown.

Expected Results:  
The image is resized to width of browser window automatically. Hovering over the image shows the Zoom-in-Icon, clicking the image zooms image to its original size.

Comment 1

[Alice0775 White]

Regression window:
Works:
http://hg.mozilla.org/mozilla-central/rev/5f857be14db9
Mozilla/5.0 (Windows; Windows NT 6.1; WOW64; rv:2.0b3pre) Gecko/20100727 Minefield/4.0b3pre ID:20100728145944
Fails:
http://hg.mozilla.org/mozilla-central/rev/b8b62b351c09
Mozilla/5.0 (Windows; Windows NT 6.1; WOW64; rv:2.0b3pre) Gecko/20100728 Minefield/4.0b3pre ID:20100728152620
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5f857be14db9&tochange=b8b62b351c09
Candidate bug:
Bug 572520 - make all imgIDecoderObserver notifications asynchronous

Comment 2

[Bobby Holley (:bholley)]

I wonder if this is the same issue as bug 583825...

Comment 4

[Joe Drew (not getting mail)]

I hope you still love me, Bobby.

Comment 5

[Bobby Holley (:bholley)]

Confirmed that this is fixed by my patch over in bug 592641. Resolving dupe.

Comment 6

[Christian Ascheberg]

(In reply to comment #5)
> Confirmed that this is fixed by my patch over in bug 592641. Resolving dupe.
> 
> *** This bug has been marked as a duplicate of bug 592641 ***

I do not know why this bug was reopened. It seems to be fixed indeed.

Comment 7

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

It was reopened because it isn't a duplicate (describes a different symptom, better to keep those separate). Bug 592641 fixed it, though, so it can be marked FIXED!