Subject: Re: bug#14032: 24.3.50; regression: `modify-frame-parameters' no longer works correctly

From: Eli Zaretskii <[email-masked]>
Date: Sat, 23 Mar 2013 11:07:12 +0200

> From: "Drew Adams" <[email-masked]>
> Cc: <[email-masked]>
> Date: Fri, 22 Mar 2013 14:40:52 -0700
> 
> OK, here is a recipe from emacs -Q:
> 
> Load these three source files, in order: frame-fns.el, frame-cmds.el,
> thumb-frm.el.  They are available here:
> http://www.emacswiki.org/emacs/?action=elisp-area;context=0
> 
> Visit a lisp file in a separate frame (e.g. C-x 5 f).
> 
> Manually resize the lisp-file frame to be wider and taller than the default
> size, enough so you will notice the difference (quite large, if you want).
> 
> C-z to thumbify the frame.  C-z to dethumbify it.  It should be as it was before
> thumbification: same position, height, and width.  It is not - it seems to be
> the default frame size.
> 
> That's the regression.

Thanks.  For the record, here's a much easier way of reproducing the
bug:

  emacs -Q
  Resize the frame by dragging one of its edges with the mouse
  M-: (modify-frame-parameters nil '((fullscreen))) RET

IOW, the problem was that when 'fullscreen' was present in the frame
parameters with a nil value, the frame dimensions were reset to the
"original" size, which was recorded only during startup, and thus
disregarded later changes of the frame dimensions.

I think I fixed this in revision 112114 on the trunk.