Cache the GdkSkia object in PlatformContextSkia so that we aren't creating

webkit/port/platform/chromium/RenderThemeGtk.cpp

 /*
  * Copyright (C) 2007 Apple Inc.
  * Copyright (C) 2007 Alp Toker <alp@atoker.com>
  * Copyright (C) 2008 Collabora Ltd.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 11 matching lines @@
 
 #include "config.h"
 #include "RenderThemeGtk.h"
 
 #include "AffineTransform.h"
 #include "GraphicsContext.h"
 #include "NotImplemented.h"
 #include "PlatformContextSkia.h"
 #include "RenderObject.h"
 #include "gtkdrawing.h"
-#include "gdkskiadrawable.h"
+#include "GdkSkia.h"
 
 #include <gdk/gdk.h>
 
 namespace WebCore {
 
 static Color makeColor(const GdkColor& c)
 {
   return Color(makeRGB(c.red >> 8, c.green >> 8, c.blue >> 8));
 }
 
@@ skipping 133 matching lines @@
     const SkIRect clip_region = canvas->getTotalClip().getBounds();
 
     GdkRectangle gdkClipRect;
     gdkClipRect.x = clip_region.fLeft;
     gdkClipRect.y = clip_region.fTop;
     gdkClipRect.width = clip_region.width();
     gdkClipRect.height = clip_region.height();
 
     gdk_rectangle_intersect(&gdkRect, &gdkClipRect, &gdkClipRect);
 
-    // TODO(agl): we should look at not creating this anew every time.
-    GdkSkia* skiadrawable = gdk_skia_new(canvas);
+    const gint r =
+      moz_gtk_widget_paint(type, pcs->gdk_skia(), &gdkRect, &gdkClipRect, &mozState, flags, direction) != MOZ_GTK_SUCCESS;
 
-    const gint r =
-      moz_gtk_widget_paint(type, skiadrawable, &gdkRect, &gdkClipRect, &mozState, flags, direction) != MOZ_GTK_SUCCESS;
-
-    g_object_unref(skiadrawable);
     return r;
 }
 
 static void setButtonPadding(RenderStyle* style)
 {
     // FIXME: This looks incorrect.
     const int padding = 8;
     style->setPaddingLeft(Length(padding, Fixed));
     style->setPaddingRight(Length(padding, Fixed));
     style->setPaddingTop(Length(padding / 2, Fixed));
@@ skipping 276 matching lines @@
 
     m_gtkTreeView = gtk_tree_view_new();
     g_signal_connect(m_gtkTreeView, "style-set", G_CALLBACK(gtkStyleSetCallback), theme());
     gtk_container_add(gtkContainer(), m_gtkTreeView);
     gtk_widget_realize(m_gtkTreeView);
 
     return m_gtkTreeView;
 }
 
 }