Do not create a workspace for a maximized window.

ash/shelf/shelf_layout_manager_unittest.cc

 // Copyright (c) 2012 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 "ash/shelf/shelf_layout_manager.h"
 
 #include "ash/accelerators/accelerator_controller.h"
 #include "ash/accelerators/accelerator_table.h"
 #include "ash/ash_switches.h"
 #include "ash/display/display_manager.h"
@@ skipping 1306 matching lines @@
   views::Widget* widget_one = CreateTestWidgetWithParams(params);
   widget_one->Maximize();
 
   views::Widget* widget_two = CreateTestWidgetWithParams(params);
   widget_two->Maximize();
   widget_two->Activate();
 
   // Both windows are maximized. They should be of the same size.
   EXPECT_EQ(widget_one->GetNativeWindow()->bounds().ToString(),
             widget_two->GetNativeWindow()->bounds().ToString());
+  int area_when_shelf_shown =
+      widget_one->GetNativeWindow()->bounds().size().GetArea();
 
   // Now hide the shelf.
   shelf->SetAutoHideBehavior(SHELF_AUTO_HIDE_BEHAVIOR_ALWAYS);
 
-  // The active maximized window will get resized to the new work area. However,
-  // the inactive window should not get resized.
-  EXPECT_NE(widget_one->GetNativeWindow()->bounds().ToString(),
-            widget_two->GetNativeWindow()->bounds().ToString());
-
-  // Activate the first window. Now, both windows should be of the same size
-  // again.
-  widget_one->Activate();
+  // Both windows should be resized according to the shelf status.
   EXPECT_EQ(widget_one->GetNativeWindow()->bounds().ToString(),
             widget_two->GetNativeWindow()->bounds().ToString());
+  // Resized to small.
+  EXPECT_LT(area_when_shelf_shown,
+            widget_one->GetNativeWindow()->bounds().size().GetArea());
 
   // Now show the shelf.
   shelf->SetAutoHideBehavior(SHELF_AUTO_HIDE_BEHAVIOR_NEVER);
 
-  // The active maximized window will get resized to the new work area. However,
-  // the inactive window should not get resized.
-  EXPECT_NE(widget_one->GetNativeWindow()->bounds().ToString(),
-            widget_two->GetNativeWindow()->bounds().ToString());
-
-  // Activate the first window. Now, both windows should be of the same size
-  // again.
-  widget_two->Activate();
+  // Again both windows should be of the same size.
   EXPECT_EQ(widget_one->GetNativeWindow()->bounds().ToString(),
             widget_two->GetNativeWindow()->bounds().ToString());
+  EXPECT_EQ(area_when_shelf_shown,
+            widget_one->GetNativeWindow()->bounds().size().GetArea());
 }
 
 // Confirm that the shelf is dimmed only when content is maximized and
 // shelf is not autohidden.
 TEST_F(ShelfLayoutManagerTest, Dimming) {
   GetShelfLayoutManager()->SetAutoHideBehavior(SHELF_AUTO_HIDE_BEHAVIOR_NEVER);
   scoped_ptr<aura::Window> w1(CreateTestWindow());
   w1->Show();
   wm::ActivateWindow(w1.get());
 
@@ skipping 66 matching lines @@
       EXPECT_TRUE(status_area_widget->IsMessageBubbleShown());
     } else {
       EXPECT_FALSE(shelf->IsVisible());
       EXPECT_FALSE(status_area_widget->IsMessageBubbleShown());
     }
   }
 }
 
 }  // namespace internal
 }  // namespace ash