The support in metacity for mapping a window initially iconified/minimized
is not working properly.

I'll attach a short test case program that demonstrates this.

The obvious problem is:

  /* See bug 334899; the window may have minimized ancestors
   * which need to be shown.
   *
   * However, we shouldn't unminimize windows here when opening
   * a new display because that breaks passing _NET_WM_STATE_HIDDEN
   * between window managers when replacing them; see bug 358042.
   */
  if (!display->display_opening)
    unminimize_window_and_all_transient_parents (window);

Which gets called even if earlier in the same function we did:

  if (window->initially_iconic)
    {
      /* WM_HINTS said minimized */
      window->minimized = TRUE;
      meta_verbose ("Window %s asked to start out minimized\n", window->desc);
    }

I think that the right fix is:

-  if (!display->display_opening)
+  if (!display->display_opening && !window->minimized)
    unminimize_window_and_all_transient_parents (window);

The display->display_opening check is still needed because it applies to the
case where the window isn't minimized but some parent is ... we still want
to preserve that previous state. (probably ... it's a little pathological)

"unminimize_window_and_all_transient_parents" is used elsewhere, so it 
doesn't need to be changed to "unminimize_all_transient_parents".