479421 - saner default handling for sizemode in SM sessionstore

Status: RESOLVED FIXED

(SeaMonkey :: Session Restore, defect)

Description

[Misak Khachatryan]

Attachment: patch [Checkin: Comment 2]

This is port of part of bug 477657, which is landed by another checkin (bug 476463). Some related comments from there:

Omitting a window's sizemode property should preserve the current state (and
not restore to a non-maximized, non-minimized state).

Comment 1

[[e-mail]]

Comment on attachment 363295 [details] [diff] [review]
patch
[Checkin: Comment 2]

>-    case "maximized":
>-       if (win_("sizemode") != "maximized")
>-         aWindow.maximize();
>-    break
>-    case "minimized":
>-       if (win_("sizemode") != "minimized")
>-         aWindow.minimize();
>-       break
>-    default:
>-       if (win_("sizemode") != "normal")
>-         aWindow.restore();
>-       break
Whoa, the coding style here is all out of whack. I want to get rid of it ASAP!

Comment 2

[Serge Gautherie (:sgautherie)]

Comment on attachment 363295 [details] [diff] [review]
patch
[Checkin: Comment 2]


http://hg.mozilla.org/comm-central/rev/2532cb89212e