Restoring a session should restore window minimization state on Windows

chrome/browser/sessions/session_service.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/sessions/session_service.h"
 
 #include <algorithm>
 #include <limits>
 #include <set>
 #include <vector>
@@ skipping 954 matching lines @@
       case kCommandSetWindowBounds3: {
         WindowBoundsPayload3 payload;
         if (!command->GetPayload(&payload, sizeof(payload)))
           return true;
         GetWindow(payload.window_id, windows)->bounds.SetRect(payload.x,
                                                               payload.y,
                                                               payload.w,
                                                               payload.h);
         ui::WindowShowState show_state = ui::SHOW_STATE_NORMAL;
         if (payload.show_state > ui::SHOW_STATE_DEFAULT &&
-            payload.show_state < ui::SHOW_STATE_MAX) {
+            payload.show_state < ui::SHOW_STATE_MAX &&
+            payload.show_state != ui::SHOW_STATE_INACTIVE) {

[sky, 2011/08/26 16:26:20]

Can you add a comment here saying INACTIVE isn't persisted. I know you have it
in the enum too, but it's easily missed.

[dhollowa, 2011/08/26 22:13:50]

Done.

           show_state = static_cast<ui::WindowShowState>(payload.show_state);
         } else {
           NOTREACHED();
         }
         GetWindow(payload.window_id, windows)->show_state = show_state;
         break;
       }
 
       case kCommandSetTabIndexInWindow: {
         TabIndexInWindowPayload payload;
@@ skipping 528 matching lines @@
       50);
   if (use_long_period) {
     std::string long_name_("SessionRestore.SaveLongPeriod");
     UMA_HISTOGRAM_CUSTOM_TIMES(long_name_,
         delta,
         save_delay_in_mins_,
         save_delay_in_hrs_,
         50);
   }
 }