Gtk3: Render a GtkHeaderBar as the background of the tab strip

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

Index: chrome/browser/ui/views/frame/opaque_browser_frame_view.cc
diff --git a/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc b/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc
index c3623f8093da0bcaa83dee045fb13ced6a57600d..198f762d1ad1cc62023ecdc17b6fe2c79c325fa1 100644
--- a/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc
+++ b/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc
@@ -507,9 +507,8 @@ int OpaqueBrowserFrameView::GetTopAreaHeight() const {
   const gfx::ImageSkia frame_image = GetFrameImage();
   int top_area_height = frame_image.isNull() ? kHeight : frame_image.height();
   if (browser_view()->IsTabStripVisible()) {
-    top_area_height =
-        std::max(top_area_height,
-                 GetBoundsForTabStrip(browser_view()->tabstrip()).bottom());
+    // TODO(thomasanderson): Figure out what this should be.  Do no land as is.
+    top_area_height = GetBoundsForTabStrip(browser_view()->tabstrip()).bottom();

[Tom (Use chromium acct), 2017/01/11 20:07:27]

pkasting@:  The "kHeight = 64" was messing this up.  The top area height should
be 44 when restored and 29 when maximized, but this code was causing it to
always be 64, which was causing the GtkHeaderBar to render stretched out.  Is
the fix just a matter of hard-coding these new constants?

[Peter Kasting, 2017/01/12 18:53:52]

44 and 29 are what the GetBoundsForTabStrip() call will end up returning, right?

I don't think we want to hardcode constants like those.  It's not those specific
values you want, it's "always use the height of the tabstrip bottom".  In other
words, I think you basically want what my TODO above suggests: to just remove
kHeight.

Looking at how this value will be used, I think it's safe to go ahead and do
this.

Effectively, you'll want to do:

  int top_area_height = frame_image.height();  // Returns 0 if isNull()

...and then leave the old body of this conditional.

What you'll want to check is what happens in windows with no tabstrip.  Reading
the above, I think this will return 0 then, and I'm not sure whether that's
right.  It depends how we draw things afterwards.

[Tom (Use chromium acct), 2017/01/12 21:51:09]

Yes
Done
I tested with a fullscreen window and this function never gets called.

[Peter Kasting, 2017/01/12 22:02:56]

Fullscreen wouldn't be the thing to check; you'd want a case where this gets
called with IsTabStripVisible() false, so a window without a tabstrip.  An app
window ought to work for this, I think?

[Tom (Use chromium acct), 2017/01/12 22:54:13]

Hm.. I tried with many types of windows and this still doesn't get called :S
I tried with:
* Chrome Remote Desktop window
* gnubbyd window
* Wrench menu
* Context menus
* Tooltips
* Chrome task manager
* JS alerts

[Peter Kasting, 2017/01/12 23:14:04]

Things like menus, tooltips, and alerts wouldn't get a frame, so you'd be
looking at app windows mostly; I dunno whether the remote desktop/gnubbyd cases
would fit this.  I'd do app>More Tools>Add To Desktop and make sure "open as
window" is checked.  Make sure to test in an environment where the opaque frame
is the native frame; I think for example Win 7 Aero Basic (non-glass) is this
case.

[Tom (Use chromium acct), 2017/01/13 00:59:51]

Ok that seemed to work.  It does return 0, but nothing is visually wrong

   }
   return top_area_height;
 }
@@ -517,6 +516,7 @@ int OpaqueBrowserFrameView::GetTopAreaHeight() const {
 void OpaqueBrowserFrameView::PaintRestoredFrameBorder(
     gfx::Canvas* canvas) const {
   frame_background_->set_frame_color(GetFrameColor());
+  frame_background_->set_is_active(ShouldPaintAsActive());
   frame_background_->set_theme_image(GetFrameImage());
   frame_background_->set_theme_overlay_image(GetFrameOverlayImage());
   frame_background_->set_top_area_height(GetTopAreaHeight());
@@ -543,6 +543,7 @@ void OpaqueBrowserFrameView::PaintRestoredFrameBorder(
 void OpaqueBrowserFrameView::PaintMaximizedFrameBorder(
     gfx::Canvas* canvas) const {
   frame_background_->set_frame_color(GetFrameColor());
+  frame_background_->set_is_active(ShouldPaintAsActive());
   frame_background_->set_theme_image(GetFrameImage());
   frame_background_->set_theme_overlay_image(GetFrameOverlayImage());
   frame_background_->set_top_area_height(GetTopAreaHeight());