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..79666d8fb850ceffb207559385cb2c95adef5ed6 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() << "LauncherItemType must be set";

[Mr4D (OOO till 08-26), 2014/01/23 19:25:08]

ShelfItemType - not LauncherItemType any more.

[Harry McCleave, 2014/01/24 22:48:47]

Done, grumble grumble rebase...

+      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;