Use white header for app windows part #2

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

 // Copyright (c) 2012 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_non_client_frame_view_ash.h"
 
 #include "ash/ash_switches.h"
 #include "ash/wm/caption_buttons/frame_caption_button_container_view.h"
 #include "ash/wm/frame_border_hit_test_controller.h"
 #include "ash/wm/header_painter.h"
@@ skipping 225 matching lines @@
 
 void BrowserNonClientFrameViewAsh::OnPaint(gfx::Canvas* canvas) {
   if (!ShouldPaint())
     return;
 
   if (UseImmersiveLightbarHeaderStyle()) {
     PaintImmersiveLightbarStyleHeader(canvas);
     return;
   }
 
+  caption_button_container_->SetPaintAsActive(ShouldPaintAsActive());
+
   // The primary header image changes based on window activation state and
   // theme, so we look it up for each paint.
-  int theme_frame_image_id = GetThemeFrameImageId();
-  int theme_frame_overlay_image_id = GetThemeFrameOverlayImageId();
+  int theme_frame_image_id = 0;
+  int theme_frame_overlay_image_id = 0;
+  GetFrameImageIds(&theme_frame_image_id, &theme_frame_overlay_image_id);
 
-  ui::ThemeProvider* theme_provider = GetThemeProvider();
-  if (!theme_provider->HasCustomImage(theme_frame_image_id) &&
-      (theme_frame_overlay_image_id == 0 ||
-       !theme_provider->HasCustomImage(theme_frame_overlay_image_id))) {
-    if (frame()->IsMaximized() || frame()->IsFullscreen())
-      theme_frame_image_id = IDR_AURA_WINDOW_HEADER_BASE_MINIMAL;
-  }
+  ash::HeaderPainter::Mode header_mode = ShouldPaintAsActive() ?
+      ash::HeaderPainter::MODE_ACTIVE : ash::HeaderPainter::MODE_INACTIVE;
   header_painter_->PaintHeader(
       canvas,
+      header_mode,
       theme_frame_image_id,
       theme_frame_overlay_image_id);
   if (browser_view()->ShouldShowWindowTitle())
     header_painter_->PaintTitleBar(canvas, BrowserFrame::GetTitleFontList());
   if (browser_view()->IsToolbarVisible())
     PaintToolbarBackground(canvas);
   else
     PaintContentEdge(canvas);
 }
 
@@ skipping 159 matching lines @@
       browser_view()->immersive_mode_controller();
   return immersive_mode_controller->IsEnabled() &&
       (immersive_mode_controller->IsRevealed() ||
        UseImmersiveLightbarHeaderStyle());
 }
 
 void BrowserNonClientFrameViewAsh::PaintImmersiveLightbarStyleHeader(
     gfx::Canvas* canvas) {
   // The light bar header is not themed because theming it does not look good.
   gfx::ImageSkia* frame_image = GetThemeProvider()->GetImageSkiaNamed(
-      IDR_AURA_WINDOW_HEADER_BASE_MINIMAL);
+      IDR_AURA_BROWSER_WINDOW_HEADER_BASE_MAXIMIZED);
   canvas->TileImageInt(*frame_image, 0, 0, width(), frame_image->height());
 }
 
 void BrowserNonClientFrameViewAsh::PaintToolbarBackground(gfx::Canvas* canvas) {
   gfx::Rect toolbar_bounds(browser_view()->GetToolbarBounds());
   if (toolbar_bounds.IsEmpty())
     return;
   gfx::Point toolbar_origin(toolbar_bounds.origin());
   View::ConvertPointToTarget(browser_view(), this, &toolbar_origin);
   toolbar_bounds.set_origin(toolbar_origin);
@@ skipping 59 matching lines @@
       gfx::Rect(x + kClientEdgeThickness,
                 toolbar_bounds.bottom() - kClientEdgeThickness,
                 w - (2 * kClientEdgeThickness),
                 kClientEdgeThickness),
       ThemeProperties::GetDefaultColor(
           ThemeProperties::COLOR_TOOLBAR_SEPARATOR));
 }
 
 void BrowserNonClientFrameViewAsh::PaintContentEdge(gfx::Canvas* canvas) {
   if (UsePackagedAppHeaderStyle()) {
-    header_painter_->PaintHeaderContentSeparator(canvas);
+    ash::HeaderPainter::Mode header_mode = ShouldPaintAsActive() ?
+        ash::HeaderPainter::MODE_ACTIVE : ash::HeaderPainter::MODE_INACTIVE;
+    header_painter_->PaintHeaderContentSeparator(canvas, header_mode);
   } else {
     canvas->FillRect(gfx::Rect(0, caption_button_container_->bounds().bottom(),
                                width(), kClientEdgeThickness),
         ThemeProperties::GetDefaultColor(
             ThemeProperties::COLOR_TOOLBAR_SEPARATOR));
   }
 }
 
-int BrowserNonClientFrameViewAsh::GetThemeFrameImageId() const {
+void BrowserNonClientFrameViewAsh::GetFrameImageIds(

[James Cook, 2014/02/03 18:46:35]

This function is long and complex. I suggest breaking it into smaller parts
somehow. For example, you could provide little functions for the
BrownserTypeNormal case, PackagedApp case, etc.

+    int* frame_image_id,
+    int* frame_overlay_image_id) const {
+  *frame_overlay_image_id = 0;
+
   bool is_incognito = !browser_view()->IsRegularOrGuestSession();
   if (browser_view()->IsBrowserTypeNormal()) {
-    // Use the standard resource ids to allow users to theme the frames.
+    // Tabbed browser windows.
+    ui::ThemeProvider* tp = GetThemeProvider();
+    if (tp->HasCustomImage(IDR_THEME_FRAME_OVERLAY) &&
+        !is_incognito) {
+      *frame_overlay_image_id = ShouldPaintAsActive() ?
+          IDR_THEME_FRAME_OVERLAY : IDR_THEME_FRAME_OVERLAY_INACTIVE;
+    }
+
     if (ShouldPaintAsActive()) {
-      return is_incognito ?
+      *frame_image_id = is_incognito ?
           IDR_THEME_FRAME_INCOGNITO : IDR_THEME_FRAME;
+    } else {
+      *frame_image_id = is_incognito ?
+          IDR_THEME_FRAME_INCOGNITO_INACTIVE : IDR_THEME_FRAME_INACTIVE;
     }
-    return is_incognito ?
-        IDR_THEME_FRAME_INCOGNITO_INACTIVE : IDR_THEME_FRAME_INACTIVE;
+
+    if ((frame()->IsMaximized() || frame()->IsFullscreen()) &&
+        !tp->HasCustomImage(IDR_THEME_FRAME) &&
+        !tp->HasCustomImage(*frame_image_id) &&
+        *frame_overlay_image_id == 0) {
+      *frame_image_id = IDR_AURA_BROWSER_WINDOW_HEADER_BASE_MAXIMIZED;
+    }
+  } else if (UsePackagedAppHeaderStyle()) {
+    // Apps are not themeable.
+    *frame_image_id = ShouldPaintAsActive() ?
+        IDR_AURA_WINDOW_HEADER_BASE_ACTIVE :
+        IDR_AURA_WINDOW_HEADER_BASE_INACTIVE;
+  } else {
+    // Browser popups are not themeable.
+    if (frame()->IsMaximized() || frame()->IsFullscreen()) {
+      *frame_image_id = IDR_AURA_BROWSER_WINDOW_HEADER_BASE_MAXIMIZED;
+    } else if (ShouldPaintAsActive()) {
+      *frame_image_id = is_incognito ?
+          IDR_AURA_BROWSER_WINDOW_HEADER_BASE_RESTORED_INCOGNITO_ACTIVE :
+          IDR_AURA_BROWSER_WINDOW_HEADER_BASE_RESTORED_ACTIVE;
+    } else {
+      *frame_image_id = is_incognito ?
+          IDR_AURA_BROWSER_WINDOW_HEADER_BASE_RESTORED_INCOGNITO_INACTIVE :
+          IDR_AURA_BROWSER_WINDOW_HEADER_BASE_RESTORED_INACTIVE;
+    }
   }
-  // Never theme app and popup windows.
-  if (ShouldPaintAsActive()) {
-    return is_incognito ?
-        IDR_AURA_WINDOW_HEADER_BASE_INCOGNITO_ACTIVE :
-        IDR_AURA_WINDOW_HEADER_BASE_ACTIVE;
-  }
-  return is_incognito ?
-      IDR_AURA_WINDOW_HEADER_BASE_INCOGNITO_INACTIVE :
-      IDR_AURA_WINDOW_HEADER_BASE_INACTIVE;
 }
-
-int BrowserNonClientFrameViewAsh::GetThemeFrameOverlayImageId() const {
-  ui::ThemeProvider* tp = GetThemeProvider();
-  if (tp->HasCustomImage(IDR_THEME_FRAME_OVERLAY) &&
-      browser_view()->IsBrowserTypeNormal() &&
-      !browser_view()->IsOffTheRecord()) {
-    return ShouldPaintAsActive() ?
-        IDR_THEME_FRAME_OVERLAY : IDR_THEME_FRAME_OVERLAY_INACTIVE;
-  }
-  return 0;
-}