Ash:Shelf - Cleanup of Alternate Shelf (part 1)

ash/shelf/shelf_model.cc

Index: ash/shelf/shelf_model.cc
diff --git a/ash/shelf/shelf_model.cc b/ash/shelf/shelf_model.cc
index 7f98bbaf4e7f508fd910c3961628734d0db3cc1c..661b5707fd86d7c66ee7210aa59bd9ae43578ecb 100644
--- a/ash/shelf/shelf_model.cc
+++ b/ash/shelf/shelf_model.cc
@@ -14,44 +14,24 @@ namespace ash {
 namespace {
 
 int ShelfItemTypeToWeight(ShelfItemType type) {
-  if (ash::switches::UseAlternateShelfLayout()) {
-    switch (type) {
-      case TYPE_APP_LIST:
-        // TODO(skuhne): If the app list item becomes movable again, this need
-        // to be a fallthrough.
-        return 0;
-      case TYPE_BROWSER_SHORTCUT:
-      case TYPE_APP_SHORTCUT:
-        return 1;
-      case TYPE_WINDOWED_APP:
-      case TYPE_PLATFORM_APP:
-        return 2;
-      case TYPE_DIALOG:
-        return 3;
-      case TYPE_APP_PANEL:
-        return 4;
-      case TYPE_UNDEFINED:
-        NOTREACHED() << "ShelfItemType must be set";
-        return -1;
-    }
-  } else {
-    switch (type) {
-      case TYPE_BROWSER_SHORTCUT:
-      case TYPE_APP_SHORTCUT:
-        return 0;
-      case TYPE_WINDOWED_APP:
-      case TYPE_PLATFORM_APP:
-        return 1;
-      case TYPE_APP_LIST:
-        return 2;
-      case TYPE_DIALOG:
-        return 3;
-      case TYPE_APP_PANEL:
-        return 4;
-      case TYPE_UNDEFINED:
-        NOTREACHED() << "ShelfItemType must be set";
-        return -1;
-    }
+  switch (type) {
+    case TYPE_APP_LIST:
+      // TODO(skuhne): If the app list item becomes movable again, this need
+      // to be a fallthrough.
+      return 0;
+    case TYPE_BROWSER_SHORTCUT:
+    case TYPE_APP_SHORTCUT:
+      return 1;
+    case TYPE_WINDOWED_APP:
+    case TYPE_PLATFORM_APP:
+      return 2;
+    case TYPE_DIALOG:
+      return 3;
+    case TYPE_APP_PANEL:
+      return 4;
+    case TYPE_UNDEFINED:
+      NOTREACHED() << "ShelfItemType must be set";
+      return -1;
   }
 
   NOTREACHED() << "Invalid type " << type;
@@ -187,8 +167,7 @@ void ShelfModel::RemoveObserver(ShelfModelObserver* observer) {
 }
 
 int ShelfModel::ValidateInsertionIndex(ShelfItemType type, int index) const {
-  DCHECK(index >= 0 && index <= item_count() +
-      (ash::switches::UseAlternateShelfLayout() ? 1 : 0));
+  DCHECK(index >= 0 && index <= item_count() + 1);
 
   // Clamp |index| to the allowed range for the type as determined by |weight|.
   LauncherItem weight_dummy;