GTK: Use background images for location bar, instead of faking it.

chrome/browser/gtk/location_bar_view_gtk.cc

 // Copyright (c) 2009 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/gtk/location_bar_view_gtk.h"
 
 #include <string>
 
 #include "app/gfx/canvas_paint.h"
 #include "app/gfx/gtk_util.h"
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/string_util.h"
 #include "chrome/app/chrome_dll_resource.h"
 #include "chrome/browser/alternate_nav_url_fetcher.h"
 #include "chrome/browser/autocomplete/autocomplete_edit_view_gtk.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/command_updater.h"
 #include "chrome/browser/extensions/extension_browser_event_router.h"
 #include "chrome/browser/extensions/extension_tabs_module.h"
+#include "chrome/browser/gtk/cairo_cached_surface.h"
 #include "chrome/browser/gtk/first_run_bubble.h"
 #include "chrome/browser/gtk/gtk_theme_provider.h"
 #include "chrome/browser/gtk/rounded_window.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/search_engines/template_url.h"
 #include "chrome/browser/search_engines/template_url_model.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/common/gtk_util.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/common/page_transition_types.h"
@@ skipping 454 matching lines @@
     gtk_util::SetLabelColor(tab_to_search_label_, &gfx::kGdkBlack);
     gtk_util::SetLabelColor(tab_to_search_hint_leading_label_,
                             &gfx::kGdkBlack);
     gtk_util::SetLabelColor(tab_to_search_hint_trailing_label_,
                             &gfx::kGdkBlack);
   }
 }
 
 gboolean LocationBarViewGtk::HandleExpose(GtkWidget* widget,
                                           GdkEventExpose* event) {
-  GdkDrawable* drawable = GDK_DRAWABLE(event->window);
   GdkRectangle* alloc_rect = &hbox_->allocation;
 
-  // The area outside of our margin, which includes the border.
-  GdkRectangle inner_rect = {
-      alloc_rect->x,
-      alloc_rect->y + kTopMargin,
-      alloc_rect->width,
-      alloc_rect->height - kTopMargin - kBottomMargin};
-
   // If we're not using GTK theming, draw our own border over the edge pixels
   // of the background.
   if (!profile_ ||
       !GtkThemeProvider::GetFrom(profile_)->UseGtkTheme()) {
-    GdkGC* gc = gdk_gc_new(drawable);
+    cairo_t* cr = gdk_cairo_create(GDK_DRAWABLE(event->window));
+    cairo_rectangle(cr, event->area.x, event->area.y, event->area.width,

[Evan Stade, 2009/10/30 23:24:52]

can use gdk_cairo_rectangle(cr, &event->area)

+                    event->area.height);
+    cairo_clip(cr);
+    CairoCachedSurface* background = theme_provider_->GetSurfaceNamed(
+        popup_window_mode_ ? IDR_LOCATIONBG_POPUPMODE_CENTER : IDR_LOCATIONBG,
+        widget);
 
-    // Some of our calculations are a bit sloppy.  Since we draw on our parent
-    // window, set a clip to make sure that we don't draw outside.
-    gdk_gc_set_clip_rectangle(gc, &inner_rect);
+    // We paint the source to the "outer" rect, which is the size of the hbox's
+    // allocation. This image blends with whatever is behind it as the top and
+    // bottom fade out.
+    background->SetSource(cr, alloc_rect->x, alloc_rect->y);
+    cairo_pattern_set_extend(cairo_get_source(cr), CAIRO_EXTEND_REPEAT);
+    cairo_rectangle(cr, alloc_rect->x, alloc_rect->y,
+                    alloc_rect->width, alloc_rect->height);
+    cairo_fill(cr);
 
-    // Draw the background.
-    gdk_gc_set_rgb_fg_color(gc,
-        &kBackgroundColorByLevel[toolbar_model_->GetSchemeSecurityLevel()]);
-    gdk_draw_rectangle(drawable, gc, TRUE,
-                       inner_rect.x,
-                       inner_rect.y,
-                       inner_rect.width,
-                       inner_rect.height);
+    // But on top of that, we also need to draw the "inner" rect, which is all
+    // the color that the background should be.
+    cairo_rectangle(cr, alloc_rect->x,
+                    alloc_rect->y + kTopMargin + kBorderThickness,
+                    alloc_rect->width,
+                    alloc_rect->height - kTopMargin -
+                    kBottomMargin - 2 * kBorderThickness);
+    gdk_cairo_set_source_color(cr, const_cast<GdkColor*>(
+        &kBackgroundColorByLevel[toolbar_model_->GetSchemeSecurityLevel()]));
+    cairo_fill(cr);
 
-    // Draw our 1px border.  TODO(deanm): Maybe this would be cleaner as an
-    // overdrawn stroked rect with a clip to the allocation?
-    gdk_gc_set_rgb_fg_color(gc, &kBorderColor);
-    gdk_draw_rectangle(drawable, gc, TRUE,
-                       inner_rect.x,
-                       inner_rect.y,
-                       inner_rect.width,
-                       kBorderThickness);
-    gdk_draw_rectangle(drawable, gc, TRUE,
-                       inner_rect.x,
-                       inner_rect.y + inner_rect.height - kBorderThickness,
-                       inner_rect.width,
-                       kBorderThickness);
-    if (popup_window_mode_) {
-      gdk_draw_rectangle(drawable, gc, TRUE,
-                         inner_rect.x,
-                         inner_rect.y,
-                         kBorderThickness,
-                         inner_rect.height);
-      gdk_draw_rectangle(drawable, gc, TRUE,
-                         inner_rect.x + inner_rect.width - kBorderThickness,
-                         inner_rect.y,
-                         kBorderThickness,
-                         inner_rect.height);
-    }
-
-    g_object_unref(gc);
+    cairo_destroy(cr);
   }
 
   return FALSE;  // Continue propagating the expose.
 }
 
 void LocationBarViewGtk::SetSecurityIcon(ToolbarModel::Icon icon) {
   gtk_widget_hide(GTK_WIDGET(security_lock_icon_image_));
   gtk_widget_hide(GTK_WIDGET(security_warning_icon_image_));
   if (icon != ToolbarModel::NO_ICON)
     gtk_widget_show(GTK_WIDGET(security_icon_event_box_));
@@ skipping 293 matching lines @@
 
   std::string badge_text = view->page_action_->GetBadgeText(tab_id);
   if (badge_text.empty())
     return FALSE;
 
   gfx::CanvasPaint canvas(event, false);
   gfx::Rect bounding_rect(widget->allocation);
   view->page_action_->PaintBadge(&canvas, bounding_rect, tab_id);
   return FALSE;
 }