GTK: Fix overflow chevron for RTL locales.

chrome/browser/gtk/bookmark_bar_gtk.cc

Index: chrome/browser/gtk/bookmark_bar_gtk.cc
===================================================================
--- chrome/browser/gtk/bookmark_bar_gtk.cc	(revision 26890)
+++ chrome/browser/gtk/bookmark_bar_gtk.cc	(working copy)
@@ -466,11 +466,18 @@
       gtk_container_get_children(GTK_CONTAINER(bookmark_toolbar_.get()));
   for (GList* iter = toolbar_items; iter; iter = g_list_next(iter)) {
     GtkWidget* tool_item = reinterpret_cast<GtkWidget*>(iter->data);
-    if (tool_item->allocation.x + tool_item->allocation.width >
-        bookmark_toolbar_.get()->allocation.width + extra_space) {
-      overflow = true;
+    if (gtk_widget_get_direction(tool_item) == GTK_TEXT_DIR_RTL) {
+      overflow = (tool_item->allocation.x <
+                  bookmark_toolbar_.get()->allocation.x - extra_space);

[Evan Martin, 2009/09/24 21:12:05]

It's confusing to me what origins these allocation.x values are relative to.  :(

[Evan Stade, 2009/09/24 21:15:42]

they are relative to the parent GdkWindow that they both share. It was in error
that I didn't take into account the toolbar's allocation.x before, but it didn't
matter because it's always (?) 0 for LTR since the toolbar is the first thing on
the bookmark bar.

+    } else {
+      overflow =
+          (tool_item->allocation.x + tool_item->allocation.width >
+           bookmark_toolbar_.get()->allocation.width +
+           bookmark_toolbar_.get()->allocation.x + extra_space);
+    }
+    if (overflow)
       break;
-    }
+
     if (showing_folders &&
         model_->GetBookmarkBarNode()->GetChild(rv)->is_folder()) {
       showing_folders->push_back(gtk_bin_get_child(GTK_BIN(tool_item)));