Remove docked windows entirely in M59.

components/sessions/core/session_service_commands.cc

 // Copyright 2014 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 "components/sessions/core/session_service_commands.h"
 
 #include <stdint.h>
 #include <string.h>
 
 #include <utility>
@@ skipping 95 matching lines @@
 // Persisted versions of ui::WindowShowState that are written to disk and can
 // never change.
 enum PersistedWindowShowState {
   // SHOW_STATE_DEFAULT (0) never persisted.
   PERSISTED_SHOW_STATE_NORMAL = 1,
   PERSISTED_SHOW_STATE_MINIMIZED = 2,
   PERSISTED_SHOW_STATE_MAXIMIZED = 3,
   // SHOW_STATE_INACTIVE (4) never persisted.
   PERSISTED_SHOW_STATE_FULLSCREEN = 5,
   PERSISTED_SHOW_STATE_DETACHED_DEPRECATED = 6,
-  PERSISTED_SHOW_STATE_DOCKED_DEPRECATED = 7,
-  PERSISTED_SHOW_STATE_END = 7
+  PERSISTED_SHOW_STATE_END = 6

[varkha, 2017/02/28 00:59:03]

I don't think you can remove DOCKED_DEPRECATED here without breaking the profile
history. You can remove the handling of docked state in
ShowStateToPersistedShowState() (since no window will ever be docked) but I
think you need to leave the handler for the deprecated state in
PersistedShowStateToShowState() and simply return ui:SHOW_STATE_NORMAL there in
old line 171.

[afakhry, 2017/03/09 22:28:27]

Done.

 };
 
 using IdToSessionTab =
     std::map<SessionID::id_type, std::unique_ptr<SessionTab>>;
 using IdToSessionWindow =
     std::map<SessionID::id_type, std::unique_ptr<SessionWindow>>;
 
 // Assert to ensure PersistedWindowShowState is updated if ui::WindowShowState
 // is changed.
 static_assert(ui::SHOW_STATE_END ==
                   static_cast<ui::WindowShowState>(PERSISTED_SHOW_STATE_END),
               "SHOW_STATE_END must equal PERSISTED_SHOW_STATE_END");
 
 // Returns the show state to store to disk based |state|.
 PersistedWindowShowState ShowStateToPersistedShowState(
     ui::WindowShowState state) {
   switch (state) {
     case ui::SHOW_STATE_NORMAL:
       return PERSISTED_SHOW_STATE_NORMAL;
     case ui::SHOW_STATE_MINIMIZED:
       return PERSISTED_SHOW_STATE_MINIMIZED;
     case ui::SHOW_STATE_MAXIMIZED:
       return PERSISTED_SHOW_STATE_MAXIMIZED;
     case ui::SHOW_STATE_FULLSCREEN:
       return PERSISTED_SHOW_STATE_FULLSCREEN;
-
-    // TODO(afakhry): Remove Docked Windows in M58.
-    case ui::SHOW_STATE_DOCKED:
-      return PERSISTED_SHOW_STATE_DOCKED_DEPRECATED;
-
     case ui::SHOW_STATE_DEFAULT:
     case ui::SHOW_STATE_INACTIVE:
       return PERSISTED_SHOW_STATE_NORMAL;
 
     case ui::SHOW_STATE_END:
       break;
   }
   NOTREACHED();
   return PERSISTED_SHOW_STATE_NORMAL;
 }
 
 // Lints show state values when read back from persited disk.
 ui::WindowShowState PersistedShowStateToShowState(int state) {
   switch (state) {
     case PERSISTED_SHOW_STATE_NORMAL:
       return ui::SHOW_STATE_NORMAL;
     case PERSISTED_SHOW_STATE_MINIMIZED:
       return ui::SHOW_STATE_MINIMIZED;
     case PERSISTED_SHOW_STATE_MAXIMIZED:
       return ui::SHOW_STATE_MAXIMIZED;
     case PERSISTED_SHOW_STATE_FULLSCREEN:
       return ui::SHOW_STATE_FULLSCREEN;
-    case PERSISTED_SHOW_STATE_DOCKED_DEPRECATED:
-      return ui::SHOW_STATE_DOCKED;
     case PERSISTED_SHOW_STATE_DETACHED_DEPRECATED:
       return ui::SHOW_STATE_NORMAL;
   }
   NOTREACHED();
   return ui::SHOW_STATE_NORMAL;
 }
 
 // Iterates through the vector updating the selected_tab_index of each
 // SessionWindow based on the actual tabs that were restored.
 void UpdateSelectedTabIndex(
@@ skipping 713 matching lines @@
     SortTabsBasedOnVisualOrderAndClear(&windows, valid_windows);
     UpdateSelectedTabIndex(valid_windows);
   }
   // AddTabsToWindows should have processed all the tabs.
   DCHECK_EQ(0u, tabs.size());
   // SortTabsBasedOnVisualOrderAndClear should have processed all the windows.
   DCHECK_EQ(0u, windows.size());
 }
 
 }  // namespace sessions