From 5b629b0e08aba39a41dca4bc7eeb434d864a97e6 Mon Sep 17 00:00:00 2001
From: Lucian Branescu Mihaila <lucian.branescu@gmail.com>
Date: Fri, 16 Jul 2010 19:39:09 +0000
Subject: Small cleanup with some asignments.

---
diff --git a/webtoolbar.py b/webtoolbar.py
index a708b4f..b71a2db 100644
--- a/webtoolbar.py
+++ b/webtoolbar.py
@@ -395,28 +395,29 @@ class PrimaryToolbar(ToolbarBox):
         history = browser.get_back_forward_list()
         limit = history.get_limit()
 
+        back_palette = self._back.get_palette()
+        forward_palette = self._forward.get_palette()
+
         # clear the palettes
-        for palette in (self._back.get_palette(), self._forward.get_palette()):
+        for palette in (back_palette, forward_palette):
             for menu_item in palette.menu.get_children():
                 palette.menu.remove(menu_item)
 
         # (re)populate the palettes
-        for item in history.get_back_list_with_limit(limit):
+        for item in list(history.get_back_list_with_limit(limit)):
             menu_item = MenuItem(item.get_title(), text_maxlen=60)
             menu_item.connect('activate', self.__history_item_activated_cb,
                               item)
 
-            palette = self._back.get_palette()
-            palette.menu.append(menu_item)
+            back_palette.menu.append(menu_item)
             menu_item.show()
 
-        for item in history.get_forward_list_with_limit(limit):
+        for item in list(history.get_forward_list_with_limit(limit)):
             menu_item = MenuItem(item.get_title(), text_maxlen=60)
             menu_item.connect('activate', self.__history_item_activated_cb,
                               item)
 
-            palette = self._forward.get_palette()
-            palette.menu.append(menu_item)
+            forward_palette.menu.append(menu_item)
             menu_item.show()
 
     def __history_item_activated_cb(self, menu_item, history_item):
--
cgit v0.9.1