Use function to access browser settings an panel

This commit is contained in:
Christian Dywan 2010-11-01 16:47:32 +01:00
parent 6ea63d2f70
commit ed6d44df5f
8 changed files with 12 additions and 27 deletions

View file

@ -293,7 +293,7 @@ midori_addons_open_in_editor_clicked_cb (GtkWidget* toolitem,
gchar* text_editor; gchar* text_editor;
browser = midori_browser_get_for_widget (GTK_WIDGET (addons->treeview)); browser = midori_browser_get_for_widget (GTK_WIDGET (addons->treeview));
settings = katze_object_get_object (browser, "settings"); settings = midori_browser_get_settings (browser);
gtk_tree_model_get (model, &iter, 0, &element, -1); gtk_tree_model_get (model, &iter, 0, &element, -1);

View file

@ -334,15 +334,11 @@ feed_panel_row_activated_cb (GtkTreeView* treeview,
MidoriWebSettings* settings; MidoriWebSettings* settings;
MidoriBrowser* browser; MidoriBrowser* browser;
gint n; gint n;
browser = midori_browser_get_for_widget (GTK_WIDGET (panel)); browser = midori_browser_get_for_widget (GTK_WIDGET (panel));
n = midori_browser_add_item (browser, item); n = midori_browser_add_item (browser, item);
settings = midori_browser_get_settings (browser);
settings = katze_object_get_object (browser, "settings");
if (!katze_object_get_boolean (settings, "open-tabs-in-the-background")) if (!katze_object_get_boolean (settings, "open-tabs-in-the-background"))
midori_browser_set_current_page (browser, n); midori_browser_set_current_page (browser, n);
g_object_unref (settings);
} }
g_object_unref (item); g_object_unref (item);
} }
@ -485,11 +481,9 @@ feed_panel_open_in_tab_activate_cb (GtkWidget* menuitem,
browser = midori_browser_get_for_widget (GTK_WIDGET (panel)); browser = midori_browser_get_for_widget (GTK_WIDGET (panel));
n = midori_browser_add_item (browser, item); n = midori_browser_add_item (browser, item);
settings = midori_browser_get_settings (browser);
settings = katze_object_get_object (browser, "settings");
if (!katze_object_get_boolean (settings, "open-tabs-in-the-background")) if (!katze_object_get_boolean (settings, "open-tabs-in-the-background"))
midori_browser_set_current_page (browser, n); midori_browser_set_current_page (browser, n);
g_object_unref (settings);
} }
} }
@ -584,10 +578,9 @@ feed_panel_button_release_event_cb (GtkWidget* widget,
browser = midori_browser_get_for_widget (GTK_WIDGET (panel)); browser = midori_browser_get_for_widget (GTK_WIDGET (panel));
n = midori_browser_add_item (browser, item); n = midori_browser_add_item (browser, item);
settings = katze_object_get_object (browser, "settings"); settings = midori_browser_get_settings (browser);
if (!katze_object_get_boolean (settings, "open-tabs-in-the-background")) if (!katze_object_get_boolean (settings, "open-tabs-in-the-background"))
midori_browser_set_current_page (browser, n); midori_browser_set_current_page (browser, n);
g_object_unref (settings);
} }
} }
else else

View file

@ -28,9 +28,8 @@ page_holder_notebook_append_view (GtkWidget* notebook)
view = midori_view_new (NULL); view = midori_view_new (NULL);
browser = midori_browser_get_for_widget (notebook); browser = midori_browser_get_for_widget (notebook);
settings = katze_object_get_object (browser, "settings"); settings = midori_browser_get_settings (browser);
midori_view_set_settings (MIDORI_VIEW (view), settings); midori_view_set_settings (MIDORI_VIEW (view), settings);
g_object_unref (settings);
gtk_widget_show (view); gtk_widget_show (view);
label = midori_view_get_proxy_tab_label (MIDORI_VIEW (view)); label = midori_view_get_proxy_tab_label (MIDORI_VIEW (view));
return gtk_notebook_append_page (GTK_NOTEBOOK (notebook), view, label); return gtk_notebook_append_page (GTK_NOTEBOOK (notebook), view, label);

View file

@ -79,7 +79,7 @@ statusbar_features_app_add_browser_cb (MidoriApp* app,
statusbar = katze_object_get_object (browser, "statusbar"); statusbar = katze_object_get_object (browser, "statusbar");
bbox = gtk_hbox_new (FALSE, 0); bbox = gtk_hbox_new (FALSE, 0);
settings = katze_object_get_object (browser, "settings"); settings = midori_browser_get_settings (browser);
toolbar = katze_object_get_object (browser, "navigationbar"); toolbar = katze_object_get_object (browser, "navigationbar");
button = katze_property_proxy (settings, "auto-load-images", "toggle"); button = katze_property_proxy (settings, "auto-load-images", "toggle");
g_object_set_data (G_OBJECT (button), "feature-label", _("Images")); g_object_set_data (G_OBJECT (button), "feature-label", _("Images"));
@ -112,7 +112,6 @@ statusbar_features_app_add_browser_cb (MidoriApp* app,
gtk_box_pack_start (GTK_BOX (bbox), button, FALSE, FALSE, 2); gtk_box_pack_start (GTK_BOX (bbox), button, FALSE, FALSE, 2);
gtk_widget_show_all (bbox); gtk_widget_show_all (bbox);
gtk_box_pack_start (GTK_BOX (statusbar), bbox, FALSE, FALSE, 3); gtk_box_pack_start (GTK_BOX (statusbar), bbox, FALSE, FALSE, 3);
g_object_unref (settings);
g_object_unref (statusbar); g_object_unref (statusbar);
g_signal_connect (extension, "deactivate", g_signal_connect (extension, "deactivate",

View file

@ -396,7 +396,7 @@ tab_panel_browser_add_tab_cb (MidoriBrowser* browser,
{ {
GtkWidget* notebook = katze_object_get_object (browser, "notebook"); GtkWidget* notebook = katze_object_get_object (browser, "notebook");
gint page = gtk_notebook_page_num (GTK_NOTEBOOK (notebook), view); gint page = gtk_notebook_page_num (GTK_NOTEBOOK (notebook), view);
MidoriWebSettings* settings = katze_object_get_object (browser, "settings"); MidoriWebSettings* settings = midori_browser_get_settings (browser);
gboolean minimized = katze_object_get_boolean (view, "minimized"); gboolean minimized = katze_object_get_boolean (view, "minimized");
GdkPixbuf* icon = midori_view_get_icon (MIDORI_VIEW (view)); GdkPixbuf* icon = midori_view_get_icon (MIDORI_VIEW (view));
const gchar* title = midori_view_get_display_title (MIDORI_VIEW (view)); const gchar* title = midori_view_get_display_title (MIDORI_VIEW (view));
@ -444,7 +444,6 @@ tab_panel_browser_add_tab_cb (MidoriBrowser* browser,
} }
g_object_unref (notebook); g_object_unref (notebook);
g_object_unref (settings);
} }
static void static void

View file

@ -119,9 +119,8 @@ static GSList *tb_editor_parse_active_items(MidoriBrowser *browser)
GSList *list = NULL; GSList *list = NULL;
MidoriWebSettings *settings; MidoriWebSettings *settings;
settings = katze_object_get_object(browser, "settings"); settings = midori_browser_get_settings(browser);
g_object_get(settings, "toolbar-items", &items, NULL); g_object_get(settings, "toolbar-items", &items, NULL);
g_object_unref(settings);
names = g_strsplit(items ? items : "", ",", 0); names = g_strsplit(items ? items : "", ",", 0);
list = tb_editor_array_to_list((const gchar **) names); list = tb_editor_array_to_list((const gchar **) names);
@ -356,9 +355,8 @@ static void tb_editor_update_toolbar(TBEditorWidget *tbw)
gtk_tree_model_foreach(GTK_TREE_MODEL(tbw->store_used), tb_editor_foreach_used, str); gtk_tree_model_foreach(GTK_TREE_MODEL(tbw->store_used), tb_editor_foreach_used, str);
settings = katze_object_get_object(tbw->browser, "settings"); settings = midori_browser_get_settings(tbw->browser);
g_object_set(settings, "toolbar-items", str->str, NULL); g_object_set(settings, "toolbar-items", str->str, NULL);
g_object_unref(settings);
g_string_free(str, TRUE); g_string_free(str, TRUE);
} }

View file

@ -134,7 +134,6 @@ settings_and_accels_new (const gchar* config,
GEnumClass* enum_class = G_ENUM_CLASS ( GEnumClass* enum_class = G_ENUM_CLASS (
g_type_class_peek (pspec->value_type)); g_type_class_peek (pspec->value_type));
GEnumValue* enum_value; GEnumValue* enum_value;
str = g_key_file_get_string (key_file, "settings", property, NULL); str = g_key_file_get_string (key_file, "settings", property, NULL);
enum_value = g_enum_get_value_by_name (enum_class, str); enum_value = g_enum_get_value_by_name (enum_class, str);
if (enum_value) if (enum_value)
@ -142,9 +141,7 @@ settings_and_accels_new (const gchar* config,
else else
g_warning (_("Value '%s' is invalid for %s"), g_warning (_("Value '%s' is invalid for %s"),
str, property); str, property);
g_free (str); g_free (str);
g_type_class_unref (enum_class);
} }
else else
g_warning (_("Invalid configuration value '%s'"), property); g_warning (_("Invalid configuration value '%s'"), property);
@ -230,7 +227,7 @@ settings_save_to_file (MidoriWebSettings* settings,
else if (type == G_TYPE_PARAM_ENUM) else if (type == G_TYPE_PARAM_ENUM)
{ {
GEnumClass* enum_class = G_ENUM_CLASS ( GEnumClass* enum_class = G_ENUM_CLASS (
g_type_class_ref (pspec->value_type)); g_type_class_peek (pspec->value_type));
gint integer; gint integer;
GEnumValue* enum_value; GEnumValue* enum_value;
g_object_get (settings, property, &integer, NULL); g_object_get (settings, property, &integer, NULL);
@ -1858,7 +1855,7 @@ main (int argc,
} }
else else
{ {
settings = katze_object_get_object (browser, "settings"); settings = g_object_ref (midori_browser_get_settings (browser));
g_object_set (settings, g_object_set (settings,
"show-menubar", FALSE, "show-menubar", FALSE,
"show-navigationbar", FALSE, "show-navigationbar", FALSE,

View file

@ -4584,7 +4584,7 @@ midori_panel_notify_page_cb (MidoriPanel* panel,
GParamSpec* pspec, GParamSpec* pspec,
MidoriBrowser* browser) MidoriBrowser* browser)
{ {
gint page = katze_object_get_boolean (panel, "page"); gint page = midori_panel_get_current_page (panel);
if (page > -1) if (page > -1)
g_object_set (browser->settings, "last-panel-page", page, NULL); g_object_set (browser->settings, "last-panel-page", page, NULL);
} }