Reduce overdraw on bookmark bar

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

Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index 2deae98f3f4f27b3af24bad518cd95c7a44ed0a9..d9cb97fec88d4d6676415f0259a67f2f1118bcba 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -200,10 +200,6 @@ void PaintDetachedBookmarkBar(gfx::Canvas* canvas,
   // Paint background for detached state; if animating, this is fade in/out.
   const ui::ThemeProvider* tp = view->GetThemeProvider();
   gfx::Rect fill_rect = view->GetLocalBounds();
-  // We have to not color the top 1dp, because that should be painted by the
-  // toolbar. We will, however, paint the 1px separator at the bottom of the
-  // first dp. See crbug.com/610359
-  fill_rect.Inset(0, 1, 0, 0);
 
   // In detached mode, the bar is meant to overlap with |contents_container_|.
   // The detached background color may be partially transparent, but the layer
@@ -509,12 +505,7 @@ void BrowserView::InitStatusBubble() {
 }
 
 gfx::Rect BrowserView::GetToolbarBounds() const {
-  gfx::Rect toolbar_bounds(toolbar_->bounds());
-  if (toolbar_bounds.IsEmpty())
-    return toolbar_bounds;
-  // The apparent toolbar edges are outside the "real" toolbar edges.
-  toolbar_bounds.Inset(-views::NonClientFrameView::kClientEdgeThickness, 0);

[Peter Kasting, 2017/05/31 02:16:03]

Can you explain why you removed this horizontal outdent?  Is it no longer needed
in the MD top chrome world, or did you mistake it for a vertical overlap, or
____?

[yiyix, 2017/06/01 21:13:55]

Sorry about it. I will pay more attention where I am replying. I will reply to
this comment to help you to be in context. 

I did more research on those lines. You are right! I have mistake it  for a
vertical overlap! I should not remove those lines! These insets are added to
draw the separator in between the tab and the toolbar in the right corner of the
browser (right to the tab strips). I have reverted them. Thank you so much!

@pkasting, could you please review this patch again? Thank you.

-  return toolbar_bounds;
+  return toolbar_->bounds();
 }
 
 gfx::Rect BrowserView::GetFindBarBoundingBox() const {
@@ -2550,8 +2541,7 @@ int BrowserView::GetRenderViewHeightInsetWithDetachedBookmarkBar() {
   }
   // Don't use bookmark_bar_view_->height() which won't be the final height if
   // the bookmark bar is animating.
-  return chrome::kNTPBookmarkBarHeight -
-         views::NonClientFrameView::kClientEdgeThickness;
+  return chrome::kNTPBookmarkBarHeight;
 }
 
 void BrowserView::ExecuteExtensionCommand(