ash:Shelf - Enable Alternate Shelf and Side Shelf by default.

ash/shelf/shelf_layout_manager.cc

Index: ash/shelf/shelf_layout_manager.cc
diff --git a/ash/shelf/shelf_layout_manager.cc b/ash/shelf/shelf_layout_manager.cc
index a507da648a839e58de1b06987e4e344ea9226e16..9d3b52ef14ea56e08cb9acb0e445d1c08b0ef530 100644
--- a/ash/shelf/shelf_layout_manager.cc
+++ b/ash/shelf/shelf_layout_manager.cc
@@ -892,10 +892,10 @@ void ShelfLayoutManager::UpdateTargetBoundsForGesture(
 
     if (ash::switches::UseAlternateShelfLayout()) {
       if (alignment_ == SHELF_ALIGNMENT_RIGHT) {
-        target_bounds->shelf_bounds_in_root.set_x(
-            available_bounds.right() - shelf_width + kStatusAreaInset);
+        target_bounds->status_bounds_in_shelf.set_x(kStatusAreaInset);

[James Cook, 2013/08/22 00:19:12]

I don't understand why you are reversing this.  Why does SHELF_ALIGNMENT_LEFT
need to do the math with available_bounds.right()?

[Harry McCleave, 2013/08/27 00:37:00]

The code here was wrong (caught by GestureDrag unit tests), the inset needs to
be kept constant on they inside edge of the shelf in the alternate shelf layout
so when the shelf is on the left the right side of the status area is what is
being positioned.

       } else {
-        target_bounds->shelf_bounds_in_root.set_x(kStatusAreaInset);
+        target_bounds->status_bounds_in_shelf.set_x(
+            available_bounds.right() - shelf_width + kStatusAreaInset);
       }
     } else {
       // The statusbar should be in the center of the shelf.