Add some extra height to bookmark bar if the font wants to be larger

chrome/browser/ui/webui/ntp/ntp_resource_cache.cc

Index: chrome/browser/ui/webui/ntp/ntp_resource_cache.cc
diff --git a/chrome/browser/ui/webui/ntp/ntp_resource_cache.cc b/chrome/browser/ui/webui/ntp/ntp_resource_cache.cc
index e67d28ea4152f4aa4785c2b8c15b344cbd83a248..1f0066862d7243572f52ab31d1cf53eb515239f8 100644
--- a/chrome/browser/ui/webui/ntp/ntp_resource_cache.cc
+++ b/chrome/browser/ui/webui/ntp/ntp_resource_cache.cc
@@ -140,7 +140,7 @@ std::string GetNewTabBackgroundCSS(const ui::ThemeProvider& theme_provider,
   if (alignment & ThemeProperties::ALIGN_TOP) {
     // The bar is detached, so we must offset the background by the bar size
     // if it's a top-aligned bar.

[Evan Stade, 2016/08/17 06:36:35]

ruh roh. This is now broken since we aren't hard-coding the bar size.

[kylix_rd, 2016/08/17 15:39:02]

It is only broken if kNTPMinimumBookmarkBarHeight isn't sufficiently tall to to
display the fonts. Possible, but not likely, IMO.

It doesn't look like this function has enough information to determine the
proper offset if the NTP bookmark bar is taller than
kNTPMinimumBookmarkBarHeight.

I can investigate.

[Evan Stade, 2016/08/17 15:47:29]

oh wait, that's right, the ntp version is taller and shouldn't have a problem
even with Hindi. Forgive me if this was covered up thread, but why do we need to
apply the same logic to the NTP bookmark bar then, instead of leaving its size
hardcoded (to simplify stuff like this)?

now that I look in the other files, it seems like you haven't changed how this
is used, so why the name change?

[kylix_rd, 2016/08/17 16:13:51]

Consistency? A little overzealous? Good question. I'll change it back.

-    int offset = chrome::kNTPBookmarkBarHeight;
+    int offset = chrome::kNTPMinimumBookmarkBarHeight;
 
     if (alignment & ThemeProperties::ALIGN_LEFT)
       return "left " + base::IntToString(-offset) + "px";