Make popups closer to pixel-perfect. The main problem is that the toolbar is...

chrome/browser/views/frame/opaque_non_client_view.h

 // Copyright (c) 2006-2008 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 CHROME_BROWSER_VIEWS_FRAME_OPAQUE_NON_CLIENT_VIEW_H_
 #define CHROME_BROWSER_VIEWS_FRAME_OPAQUE_NON_CLIENT_VIEW_H_
 
 #include "chrome/browser/views/frame/opaque_frame.h"
 #include "chrome/browser/views/tab_icon_view.h"
 #include "chrome/views/non_client_view.h"
@@ skipping 77 matching lines @@
   int NonClientBorderWidth() const;
 
   // Returns the height of the entire nonclient top border, including the window
   // frame, any title area, and any connected client edge.
   int NonClientTopBorderHeight() const;
 
   // Returns the height of the entire nonclient bottom border, including both
   // the window frame and the client edge.
   int NonClientBottomBorderHeight() const;
 
-  // For windows without a toolbar, restored mode also draws a client edge below
-  // the titlebar which is considered part of the nonclient height.  This
-  // returns the height of any such edge.
-  int ClientEdgeThicknessWithinNonClientHeight() const;
+  // For windows without a toolbar, a bottom border, and, in restored mode, a
+  // client edge are drawn at the bottom of the titlebar.  When a toolbar is
+  // present, neither of these are drawn, as the toolbar itself will edge the
+  // titlebar area.  This returns the height of any such edge.
+  int BottomEdgeThicknessWithinNonClientHeight() const;
 
   // Calculates multiple values related to title layout.  Returns the height of
   // the entire titlebar including any connected client edge.
   int TitleCoordinates(int* title_top_spacing,
                        int* title_thickness) const;
 
   // Paint various sub-components of this view.  The *FrameBorder() functions
   // also paint the background of the titlebar area, since the top frame border
   // and titlebar background are a contiguous component.
   void PaintRestoredFrameBorder(ChromeCanvas* canvas);
@@ skipping 61 matching lines @@
   static views::WindowResources* inactive_resources_;
   static views::WindowResources* active_otr_resources_;
   static views::WindowResources* inactive_otr_resources_;
   static ChromeFont title_font_;
 
   DISALLOW_EVIL_CONSTRUCTORS(OpaqueNonClientView);
 };
 
 #endif  // #ifndef CHROME_BROWSER_VIEWS_FRAME_OPAQUE_NON_CLIENT_VIEW_H_