Bug 15756 – skin switching on desktop squeezeplay has regressed

Bug 15756 - skin switching on desktop squeezeplay has regressed

Summary:

skin switching on desktop squeezeplay has regressed

Status:	CLOSED FIXED

Product:	SqueezePlay
Classification:	Unclassified
Component:	UI Skin
Version:	7.5.x
Platform:	PC Other

Importance:	-- normal (vote)
Target Milestone:	7.6.0
Assigned To:	Alan Young

URL:
Keywords:

Depends on:	10001
Blocks:
	Show dependency tree / graph

Reported:	2010-02-22 15:06 UTC by Ben Klaas
Modified:	2011-05-16 09:41 UTC (History)
CC List:	2 users (show)

See Also:
Category:	Bug

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Ben Klaas 2010-02-22 15:06:39 UTC

This is probably low priority, but changing skins on desktop squeezeplay is often requring an application restart. These are the errors that are thrown to screen when the bad state is hit after a skin switch...

20100222 22:29:44.856 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/jive_wallpaper_clock_dotmatrix.png

20100222 22:29:44.856 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_0.png

20100222 22:29:44.856 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_1.png

20100222 22:29:44.856 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_2.png

20100222 22:29:44.856 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_3.png

20100222 22:29:44.856 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_4.png

20100222 22:29:44.857 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_5.png

20100222 22:29:44.857 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_6.png

20100222 22:29:44.857 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_7.png

20100222 22:29:44.857 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_8.png

20100222 22:29:44.857 ERROR  squeezeplay.ui.draw - _new_image:98 Maximum number of images (500) exceeded for /Users/bklaas/svk/squeezeplay/7.5/trunk/squeezeplay/src/squeezeplay/share/applets/QVGAportraitSkin/images/Clocks/Dot_Matrix/dotmatrix_clock_9.png

20100222 22:29:44.939 ERROR  squeezeplay.ui.draw - jive_tile_load_tiles:490 Can't find image applets/QVGAportraitSkin/images/Clocks/DotMatrix/dot_matrix_alarm_on.png

Comment 1 Alan Young 2010-02-22 23:39:15 UTC

Which skins were you switching between? I though that I confirmed that switching between the normal and 'touch' skins was no problem. The 500-image limit is pretty arbitrary. It should probably be made dynamic but I was wary of the performance cost. Let's tackle this post 7.5 when I will rewrite the whole Surface/Tile mess anyway.

Comment 2 Ben Klaas 2010-02-23 07:20:04 UTC

I was going between a Touch skin to Controller, which I recognize is not really a valid skin switch (esp. on our hardware), but I regularly do this to test how different skins render a particular window/menu style.

This is fine for 8.0.0. Restarting desktop SP is an effective workaround in the meantime

Comment 3 Alan Young 2010-05-10 01:45:35 UTC

I believe that this should now be fixed via the changes integrated for bug 10001 in 7.6. Ben, please let me know if this is still a problem.

Comment 4 Alan Young 2010-08-24 00:22:33 UTC

Ben, can you confirm that this is fixed in 7.6?

Comment 5 Ben Klaas 2010-08-24 06:24:05 UTC

Thanks Alan, this appears to be looking good now.

Comment 6 Paul Chandler 2011-05-16 09:41:29 UTC

Skin switching verified on Squeezeplay build: 7.6.0 r 9441   WIn 7