Fix calculation of the bubble view height.

ash/common/system/tray/system_tray.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/common/system/tray/system_tray.h"
 
 #include <algorithm>
 #include <map>
 #include <vector>
 
@@ skipping 660 matching lines @@
   else if (system_bubble_)
     bubble_view = system_bubble_->bubble_view();
 
   int height = 0;
   if (bubble_view) {
     gfx::Rect work_area =
         display::Screen::GetScreen()
             ->GetDisplayNearestWindow(bubble_view->GetWidget()->GetNativeView())
             .work_area();
     height =
-        std::max(0, work_area.height() - bubble_view->GetBoundsInScreen().y());
+        std::max(0, work_area.bottom() - bubble_view->GetBoundsInScreen().y());

[Mr4D (OOO till 08-26), 2017/01/10 15:08:26]

Just wondering - should there be a minumum height instead of 0?

[yhanada, 2017/01/11 01:18:55]

This height doesn't include a height of tray. I think there is no problem that a
minimum height is 0 when no bubble view (e.g. message center or menu) is shown.

   }
   if (web_notification_tray_)
     web_notification_tray_->SetTrayBubbleHeight(height);
 }
 
 base::string16 SystemTray::GetAccessibleTimeString(
     const base::Time& now) const {
   base::HourClockType hour_type =
       WmShell::Get()->system_tray_controller()->hour_clock_type();
   return base::TimeFormatTimeOfDayWithHourClockType(now, hour_type,
@@ skipping 203 matching lines @@
           .work_area()
           .height();
   if (work_area_height > 0) {
     UMA_HISTOGRAM_CUSTOM_COUNTS(
         "Ash.SystemMenu.PercentageOfWorkAreaHeightCoveredByMenu",
         100 * bubble_view->height() / work_area_height, 1, 300, 100);
   }
 }
 
 }  // namespace ash