239393 - Set as Wallpaper: re-selecting "Stretch" doesn't update preview

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[José Jeria]

When you select set as wallpaper and try out the different "position":s, the
preview stops working after testing 2 options.

Steps to reproduce:
1. Right click an image an "set as wallpaper"
2. Switch between the 3 positions availabe to see the preview
3. After selecting at least 2 options the other options stop to work

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7b) Gecko/20040330
Firefox/0.8.0+

Comment 1

[Jesse Ruderman]

This regressed in March 8 builds.  (Some builds I tested were unofficial, so
don't assume they were made at 8am.)

02/16: dialog introduced
02/19: selecting "tile" does nothing
03/06: selecting "tile" does nothing
03/07: selecting "tile" does nothing      \ 
03/08: selecting "stretch" does nothing   /  regression window
03/20: selecting "stretch" does nothing

Comment 2

[PikeUK]

In setWallPaper.xul, these constants need to be updated to reflect the changes
made when moving from winhooks to shellservice:

ISS.WALLPAPER_TILE -> ISS.BACKGROUND_TILE

and

ISS.WALLPAPER_STRETCH -> ISS.BACKGROUND_STRETCH

Making those changes fixes it for me.

Comment 3

[Blake Ross]

Fixed.

Comment 4

[José Jeria]

This is only kind of fixed, it updates center and stretch, but not tile...
Reopen or file a new bug?

Comment 5

[Jesse Ruderman]

Tile never worked.  See
http://lxr.mozilla.org/mozilla/source/browser/base/content/setWallpaper.xul#111.
File a new bug.

Comment 6

[José Jeria]

Thanks for the info, bug 239618 was filed.