Improve the look of the Linux omnibox.

chrome/browser/autocomplete/autocomplete_popup_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/autocomplete/autocomplete_popup_view_gtk.h"
 
 #include "base/gfx/gtk_util.h"
 #include "base/logging.h"
 #include "base/scoped_ptr.h"
 #include "base/string_util.h"
@@ skipping 23 matching lines @@
       window_(gtk_window_new(GTK_WINDOW_POPUP)),
       vbox_(NULL),
       opened_(false) {
   GTK_WIDGET_UNSET_FLAGS(window_, GTK_CAN_FOCUS);
   // Don't allow the window to be resized.  This also forces the window to
   // shrink down to the size of its child contents.
   gtk_window_set_resizable(GTK_WINDOW(window_), FALSE);
   // Set up a 1 pixel border around the popup.
   gtk_container_set_border_width(GTK_CONTAINER(window_), 1);
   gtk_widget_modify_bg(window_, GTK_STATE_NORMAL, &kPopupBorderColor);
-  gtk_widget_modify_base(window_, GTK_STATE_NORMAL, &kPopupBorderColor);
 }
 
 AutocompletePopupViewGtk::~AutocompletePopupViewGtk() {
   // Explicitly destroy our model here, before we destroy our GTK widgets.
   // This is because the model destructor can call back into us, and we need
   // to make sure everything is still valid when it does.
   model_.reset();
   if (vbox_)
     gtk_widget_destroy(vbox_);
   gtk_widget_destroy(window_);
@@ skipping 11 matching lines @@
   }
 
   // TODO(deanm): This could be done better, but the code is temporary
   // and will need to be replaced with custom drawing and not widgets.
   if (vbox_)
     gtk_widget_destroy(vbox_);
 
   vbox_ = gtk_vbox_new(FALSE, 0);
 
   for (size_t i = 0; i < result.size(); ++i) {
-    std::string utf8;
-    utf8.append(WideToUTF8(result.match_at(i).contents));
-    utf8.append(" - ");
-    utf8.append(WideToUTF8(result.match_at(i).description));
-    GtkWidget* label = gtk_label_new(utf8.c_str());
+    GtkWidget* label = gtk_label_new(NULL);
+    char* markup = g_markup_printf_escaped(
+        "%s   <span weight=\"light\" size=\"small\">%s</span>",
+        WideToUTF8(result.match_at(i).contents).c_str(),
+        WideToUTF8(result.match_at(i).description).c_str());
+    gtk_label_set_markup(GTK_LABEL(label), markup);
+    g_free (markup);

[Evan Martin, 2009/03/13 16:33:29]

extra space

+
     // We need to put the labels in an event box for background painting.
     GtkWidget* ebox = gtk_event_box_new();
     gtk_misc_set_alignment(GTK_MISC(label), 0, 0);
     gtk_container_add(GTK_CONTAINER(ebox), label);
     gtk_box_pack_start(GTK_BOX(vbox_), ebox, FALSE, FALSE, 0);
     gtk_widget_show(label);
     gtk_widget_show(ebox);
     if (i == model_->selected_line()) {
       gtk_widget_modify_bg(ebox, GTK_STATE_NORMAL, &kHighlightColor);
     } else {
@@ skipping 20 matching lines @@
   gtk_window_move(GTK_WINDOW(window_), x, y);
   gtk_widget_set_size_request(window_, width, -1);
   gtk_widget_show(window_);
   opened_ = true;
 }
 
 void AutocompletePopupViewGtk::Hide() {
   gtk_widget_hide(window_);
   opened_ = false;
 }