Add a ScopedPangoFontDescription helper class and use it.

ui/gfx/render_text_linux.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 "ui/gfx/render_text_linux.h"
 
 #include <fontconfig/fontconfig.h>
 #include <pango/pangocairo.h>
 #include <algorithm>
 #include <string>
@@ skipping 130 matching lines @@
 std::vector<RenderText::FontSpan> RenderTextLinux::GetFontSpansForTesting() {
   EnsureLayout();
 
   std::vector<RenderText::FontSpan> spans;
   for (GSList* it = current_line_->runs; it; it = it->next) {
     PangoItem* item = reinterpret_cast<PangoLayoutRun*>(it->data)->item;
     const int start = LayoutIndexToTextIndex(item->offset);
     const int end = LayoutIndexToTextIndex(item->offset + item->length);
     const ui::Range range(start, end);
 
-    PangoFontDescription* native_font =
-        pango_font_describe(item->analysis.font);
-    spans.push_back(RenderText::FontSpan(Font(native_font), range));
-    pango_font_description_free(native_font);
+    ScopedPangoFontDescription desc(pango_font_describe(item->analysis.font));
+    spans.push_back(RenderText::FontSpan(Font(desc.get()), range));
   }
 
   return spans;
 }
 
 SelectionModel RenderTextLinux::AdjacentCharSelectionModel(
     const SelectionModel& selection,
     VisualCursorDirection direction) {
   GSList* run = GetRunContainingCaret(selection);
   if (!run) {
@@ skipping 169 matching lines @@
   for (StyleRanges::const_iterator i = style_ranges().begin();
        i < style_ranges().end(); ++i) {
     // In Pango, different fonts means different runs, and it breaks Arabic
     // shaping across run boundaries. So, set font only when it is different
     // from the default font.
     // TODO(xji): We'll eventually need to split up StyleRange into components
     // (ColorRange, FontRange, etc.) so that we can combine adjacent ranges
     // with the same Fonts (to avoid unnecessarily splitting up runs).
     if (i->font_style != default_font_style) {
       FontList derived_font_list = font_list().DeriveFontList(i->font_style);
-      PangoFontDescription* desc = pango_font_description_from_string(
-          derived_font_list.GetFontDescriptionString().c_str());
+      ScopedPangoFontDescription desc(pango_font_description_from_string(
+          derived_font_list.GetFontDescriptionString().c_str()));
 
-      PangoAttribute* pango_attr = pango_attr_font_desc_new(desc);
+      PangoAttribute* pango_attr = pango_attr_font_desc_new(desc.get());
       pango_attr->start_index = TextIndexToLayoutIndex(i->range.start());
       pango_attr->end_index = TextIndexToLayoutIndex(i->range.end());
       pango_attr_list_insert(attrs, pango_attr);
-      pango_font_description_free(desc);
     }
   }
 
   pango_layout_set_attributes(layout, attrs);
   pango_attr_list_unref(attrs);
 }
 
 void RenderTextLinux::DrawVisualText(Canvas* canvas) {
   DCHECK(layout_);
 
@@ skipping 42 matching lines @@
     // TODO(asvitkine): Can we avoid looping here, e.g. by caching this per run?
     int style = -1;
     for (size_t i = 0; i < style_ranges_utf8.size(); ++i) {
       if (IndexInRange(style_ranges_utf8[i], first_glyph_byte_index)) {
         style = i;
         break;
       }
     }
     DCHECK_GE(style, 0);
 
-    PangoFontDescription* native_font =
-        pango_font_describe(run->item->analysis.font);
+    ScopedPangoFontDescription desc(
+        pango_font_describe(run->item->analysis.font));
 
     const std::string family_name =
-        pango_font_description_get_family(native_font);
-    renderer.SetTextSize(GetPangoFontSizeInPixels(native_font));
-
-    pango_font_description_free(native_font);
+        pango_font_description_get_family(desc.get());
+    renderer.SetTextSize(GetPangoFontSizeInPixels(desc.get()));
 
     SkScalar glyph_x = x;
     SkScalar start_x = x;
     int start = 0;
 
     glyphs.resize(glyph_count);
     pos.resize(glyph_count);
 
     for (int i = 0; i < glyph_count; ++i) {
       const PangoGlyphInfo& glyph = run->glyphs->glyphs[i];
@@ skipping 114 matching lines @@
   return bounds;
 }
 
 std::vector<Rect> RenderTextLinux::GetSelectionBounds() {
   if (selection_visual_bounds_.empty())
     selection_visual_bounds_ = CalculateSubstringBounds(selection());
   return selection_visual_bounds_;
 }
 
 }  // namespace gfx