Add a top shadow to material design omnibox dropdown

chrome/browser/ui/views/location_bar/location_bar_view.cc

Index: chrome/browser/ui/views/location_bar/location_bar_view.cc
diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.cc b/chrome/browser/ui/views/location_bar/location_bar_view.cc
index 2d5fd1269c656e4c5bdc2f229d81f7408f2e86cf..5bcfe05151952bb1b0a7f2936ba900318fda6488 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.cc
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.cc
@@ -488,15 +488,15 @@ void LocationBarView::GetOmniboxPopupPositioningInfo(
     gfx::Point* top_left_screen_coord,
     int* popup_width,
     int* left_margin,
-    int* right_margin) {
+    int* right_margin,
+    int top_edge_overlap) {
   // Because the popup might appear atop the attached bookmark bar, there won't
   // necessarily be a client edge separating it from the rest of the toolbar.
   // Therefore we position the popup high enough so it can draw its own client
   // edge at the top, in the same place the toolbar would normally draw the
   // client edge.

[Peter Kasting, 2015/08/29 01:00:23]

Let's remove this comment from this file, as it basically is about why the
overlap has the value it has, which is something this callee doesn't need or
care to know.  This info belongs more properly in the caller; I'll give thoughts
there about how to convey this.

[tdanderson, 2015/08/31 15:23:18]

Done.

-  *top_left_screen_coord = gfx::Point(
-      0,
-      parent()->height() - views::NonClientFrameView::kClientEdgeThickness);
+  *top_left_screen_coord = gfx::Point(0, parent()->height() - top_edge_overlap);
+
   views::View::ConvertPointToScreen(parent(), top_left_screen_coord);
   *popup_width = parent()->width();