262134 - Set as Desktop Background (wallpaper) gets zoomed (scaled) image dimensions instead of real ones

Status: VERIFIED FIXED

(Firefox :: General, defect)

Description

[Carlo 'Carcarlo' Carmagnini]

User-Agent:       Mozilla/5.0 (Windows; U; Win98; rv:1.7.3) Gecko/20040913 Firefox/0.10
Build Identifier: Mozilla/5.0 (Windows; U; Win98; rv:1.7.3) Gecko/20040913 Firefox/0.10

Having set monitor to resolution 1024x768 (for example), opening a 1024x768
image with the option "resize large images to fit in the browser window"
enabled, makes Set As Wallpaper believing that image is smaller than it is.
The "Set As Wallpaper" function works right, the only problem is in the preview
window.

Reproducible: Always
Steps to Reproduce:
1. Set 1024x768 as your resolution (it's not related to the bug but only to make
this example work)
2. check Firefox preferences, the option "resize large images to fit in the
browser window" must be enabled.
3. open a 1024x768 image with Firefox and right click "Set As Wallpaper"
4. the browser misunderstands the real image measures and the preview shows a
wallpaper smaller than 1024x768 that doesn't fit the desktop.
Actual Results:  
I'll attach a screen capture that better explains the problem.

Comment 1

[Carlo 'Carcarlo' Carmagnini]

Attachment: Screen capture showing the bug

This is the screen capture of the bug.

Comment 2

[Carlo 'Carcarlo' Carmagnini]

Confirmed in 
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0

Comment 3

[Thomas K. (:tom)]

Confirmed in
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.6) Gecko/20050223
Firefox/1.0.1

Comment 4

[Carlo 'Carcarlo' Carmagnini]

Confirmed in
Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7.8) Gecko/20050511 Firefox/1.0.4

Comment 5

[Thomas K. (:tom)]

Confirmed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b4)
Gecko/20050721 Firefox/1.0+

I used the 1920x1200 version of tbtw and it's "scaled" version was shown in the
preview, even though it was set to "center".  When at original size in the
browser, the preview is much different, only showing the center of the large image.

Changing to WinXP (I would make it all but I don't know how it happens on
Linux/x86Mac) to show that it happens on the 'latest' version of windows as well.

Comment 6

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

*** Bug 303768 has been marked as a duplicate of this bug. ***

Comment 7

[Gabriel Seah]

With the change in terminology I suggest this bug be renamed to "Set as Desktop
Background gets zoomed..."

Comment 8

[Adam Guthrie]

Attachment: Use the actual size of the image when we're doing the preview

When we set the image as the desktop background we use the full size, but here |this._image.width/height| refers to the actual size of the image element (which might be scaled down). This uses the naturalHeight/Width of the image which is irrespective of whether the image is scaled in the browser window or not. (Thanks to biesi for helping with this.)

Comment 9

[Asaf Romano (gone)]

Comment on attachment 236021 [details] [diff] [review]
Use the actual size of the image when we're doing the preview

r=mano.

Comment 10

[Martijn Wargers (dead)]

Checking in setDesktopBackground.js;
/cvsroot/mozilla/browser/components/shell/content/setDesktopBackground.js,v  <--
  setDesktopBackground.js
new revision: 1.5; previous revision: 1.4
done

Checked in on trunk.

Comment 11

[Adam Guthrie]

Comment on attachment 236021 [details] [diff] [review]
Use the actual size of the image when we're doing the preview

Low-risk patch for the Set As Desktop Background dialog that makes sure we always use the full height/width of an image when preview it.

Comment 12

[Mike Connor [:mconnor]]

Comment on attachment 236021 [details] [diff] [review]
Use the actual size of the image when we're doing the preview

a=mconnor on behalf of drivers, please get this in ASAP

Comment 13

[Martijn Wargers (dead)]

Checking in setDesktopBackground.js;
/cvsroot/mozilla/browser/components/shell/content/setDesktopBackground.js,v  <--
  setDesktopBackground.js
new revision: 1.3.4.2; previous revision: 1.3.4.1
done

Checked in on the 1.8.1 branch.