The shadow drawn by the browser window was being drawn over by

the status area view.
Review URL: http://codereview.chromium.org/249002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27243 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 7 insertions, 3 deletions

diff --git a/chrome/browser/chromeos/status_area_view.cc b/chrome/browser/chromeos/status_area_view.cc
index 7b41040..7cbc28b 100644
--- a/chrome/browser/chromeos/status_area_view.cc
+++ b/chrome/browser/chromeos/status_area_view.cc
@@ -251,8 +251,10 @@ gfx::Size StatusAreaView::GetPreferredSize() {
   // TODO(brettw) do we need to use result_h? This is currently hardcoded
   // becuase the menu button really wants to be larger, but we don't want
   // the status bar to force the whole tab strip to be larger. Making it
-  // "small" just means that we'll expand to the height, which we want.
-  return gfx::Size(result_w - kStatusItemSeparation, 10);
+  // "smaller" just means that we'll expand to the height, which we want.
+  // The height of 24 makes it just big enough to sit on top of the shadow
+  // drawn by the browser window.
+  return gfx::Size(result_w - kStatusItemSeparation, 24);
 }
 
 void StatusAreaView::Layout() {
@@ -262,7 +264,9 @@ void StatusAreaView::Layout() {
     gfx::Size cur_size = cur->GetPreferredSize();
 
     // Put next in row horizontally, and center vertically.
-    cur->SetBounds(cur_x, (height() - cur_size.height()) / 2,
+    // Padding the y position by 1 balances the space above and
+    // below the icons, but still allows the shadow to be drawn.
+    cur->SetBounds(cur_x, (height() - cur_size.height()) / 2 + 1,
                    cur_size.width(), cur_size.height());
     cur_x += cur_size.width() + kStatusItemSeparation;
   }