Fixes BrowserViewHostedAppTest.Layout when using MD

chrome/browser/ui/views/frame/browser_view_unittest.cc

 // Copyright 2013 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 "chrome/browser/ui/views/frame/browser_view.h"
 
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/views/bookmarks/bookmark_bar_view.h"
 #include "chrome/browser/ui/views/frame/browser_view_layout.h"
@@ skipping 182 matching lines @@
 
   // The position of the bottom of the header (the bar with the window
   // controls) in the coordinates of BrowserView.
   int bottom_of_header = browser_view()->frame()->GetTopInset() -
       header_offset.y();
 
   // The web contents should be flush with the bottom of the header.
   EXPECT_EQ(bottom_of_header, contents_container->y());
 
   // The find bar should overlap the 1px header/web-contents separator at the
-  // bottom of the header.
-  EXPECT_EQ(browser_view()->frame()->GetTopInset() - 1,
+  // bottom of the header or be a few pixels higher still with Material Design.

[tdanderson, 2015/10/09 18:12:09]

nit: I don't think changing the comment here is necessary.

[varkha, 2015/10/09 18:20:35]

Done.

+  EXPECT_EQ(browser_view()->frame()->GetTopInset() -
+                GetLayoutConstant(FIND_BAR_VERTICAL_OFFSET),
             browser_view()->GetFindBarBoundingBox().y());
 }