Make the minimized home card not overlap activities on Athena

extensions/shell/browser/shell_desktop_controller.h

 // 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.
 
 #ifndef EXTENSIONS_SHELL_BROWSER_SHELL_DESKTOP_CONTROLLER_H_
 #define EXTENSIONS_SHELL_BROWSER_SHELL_DESKTOP_CONTROLLER_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 12 matching lines @@
 class DefaultCaptureClient;
 class FocusClient;
 }
 }
 
 namespace content {
 class BrowserContext;
 }
 
 namespace gfx {
+class Insets;
 class Size;
 }
 
 #if defined(OS_CHROMEOS)
 namespace ui {
 class UserActivityPowerManagerNotifier;
 }
 #endif
 
 namespace wm {
@@ skipping 35 matching lines @@
   // |app_window_controller|.
   void SetAppWindowController(ShellAppWindowController* app_window_controller);
 
   // Creates a new app window and adds it to the desktop. The desktop maintains
   // ownership of the window.
   ShellAppWindow* CreateAppWindow(content::BrowserContext* context);
 
   // Closes and destroys the app windows.
   void CloseAppWindows();
 
+#if defined(OS_CHROMEOS)

[oshima, 2014/08/14 00:19:02]

do you need this ifdef?

I believe the CHROMEOS below are for ozone. This code should work fine on ozone.

[pkotwicz, 2014/08/14 15:49:39]

Removed the ifdef. Once app shell is for more than just CrOS, we will want an
ifdef because the app setting the workarea only makes sense on CrOS.

+  // Sets the screen's work area insets.

[pkotwicz, 2014/08/13 13:48:55]

There is no app API for setting the work area bounds so
SetDisplayWorkAreaInsets() does not really belong here. However, I could not
think of something better (without needing a lot of new code)

[oshima, 2014/08/14 00:19:02]

I was fine with something like this,

static_cast<TestScreen>(gfx::Screen::GetNativeScreen())->SetWorkAreaInsets()

but your approach is better.

+  void SetDisplayWorkAreaInsets(const gfx::Insets& insets);
+#endif
+
   // Overridden from aura::client::WindowTreeClient:
   virtual aura::Window* GetDefaultParent(aura::Window* context,
                                          aura::Window* window,
                                          const gfx::Rect& bounds) OVERRIDE;
 
 #if defined(OS_CHROMEOS)
   // ui::DisplayConfigurator::Observer overrides.
   virtual void OnDisplayModeChanged(const std::vector<
       ui::DisplayConfigurator::DisplayState>& displays) OVERRIDE;
 #endif
@@ skipping 42 matching lines @@
 
   // The desktop supports a single app window.
   scoped_ptr<ShellAppWindowController> app_window_controller_;
 
   DISALLOW_COPY_AND_ASSIGN(ShellDesktopController);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_SHELL_BROWSER_SHELL_DESKTOP_CONTROLLER_H_