gtk: Fix memory leak in ThemeServiceGtk::LoadGtkValues.

chrome/browser/ui/gtk/theme_service_gtk.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/gtk/theme_service_gtk.h"
 
 #include <gtk/gtk.h>
 
 #include <set>
 #include <string>
@@ skipping 760 matching lines @@
 
   SetThemeColorFromGtk(ThemeService::COLOR_NTP_LINK,
                        link_color);
   SetThemeColorFromGtk(ThemeService::COLOR_NTP_LINK_UNDERLINE,
                        link_color);
   SetThemeColorFromGtk(ThemeService::COLOR_NTP_SECTION_LINK,
                        link_color);
   SetThemeColorFromGtk(ThemeService::COLOR_NTP_SECTION_LINK_UNDERLINE,
                        link_color);
 
+  gdk_color_free(const_cast<GdkColor*>(link_color));

[John Knottenbelt, 2012/03/27 09:30:42]

We shouldn't free link_color if it is &kDefaultLinkColor. 

+
   // Generate the colors that we pass to WebKit.
   focus_ring_color_ = gfx::GdkColorToSkColor(frame_color);
   GdkColor thumb_active_color, thumb_inactive_color, track_color;
   ThemeServiceGtk::GetScrollbarColors(&thumb_active_color,
                                       &thumb_inactive_color,
                                       &track_color);
   thumb_active_color_ = gfx::GdkColorToSkColor(thumb_active_color);
   thumb_inactive_color_ = gfx::GdkColorToSkColor(thumb_inactive_color);
   track_color_ = gfx::GdkColorToSkColor(track_color);
 
@@ skipping 335 matching lines @@
   cairo_new_path(cr);
   cairo_set_line_width(cr, 1.0);
   cairo_move_to(cr, start_x, allocation.y);
   cairo_line_to(cr, start_x, allocation.y + allocation.height);
   cairo_stroke(cr);
   cairo_destroy(cr);
   cairo_pattern_destroy(pattern);
 
   return TRUE;
 }