Consolidate debugging in midori_debug()/ MIDORI_DEBUG

diff --git a/INSTALL b/INSTALL
index ad1dddbb6b8a911e483b0524ae969d8085cf1b68..05a3eb9450efcf0061aea3d381fe77cbe9f50cfc 100644 (file)
--- a/INSTALL
+++ b/INSTALL
@@ -53,9 +53,9 @@ If the problem is a warning and not a crash, try this:
 
 If you are interested in HTTP communication, try this:
 
-'MIDORI_SOUP_DEBUG=2 _build/default/midori/midori'
+'MIDORI_DEBUG=soup:2 _build/default/midori/midori'
 
-Where '2' can be a level between 0 and 3.
+Where '2' can be a level between 1 and 3.
 
 If you are interested in (non-) touchscreen behaviour, try this:
 

diff --git a/extensions/adblock.c b/extensions/adblock.c
index b35ae79310457d994991f4973e4b2f0e6d077cd6..96d414c53fef4831fdd2e39a34e5eecf4ff052a2 100644 (file)
--- a/extensions/adblock.c
+++ b/extensions/adblock.c
@@ -29,7 +29,7 @@
     (__filter[4] != '-' && __filter[5] != '-')
 #ifdef G_ENABLE_DEBUG
     #define adblock_debug(dmsg, darg1, darg2) \
-        do { if (debug == 1) g_debug (dmsg, darg1, darg2); } while (0)
+        do { if (midori_debug ("adblock:1")) g_debug (dmsg, darg1, darg2); } while (0)
 #else
     #define adblock_debug(dmsg, darg1, darg2) /* nothing */
 #endif
@@ -41,9 +41,6 @@ static GHashTable* urlcache = NULL;
 static GHashTable* blockcssprivate = NULL;
 static GHashTable* navigationwhitelist = NULL;
 static GString* blockcss = NULL;
-#ifdef G_ENABLE_DEBUG
-static guint debug;
-#endif
 
 static gboolean
 adblock_parse_file (gchar* path);
@@ -884,7 +881,7 @@ adblock_resource_request_starting_cb (WebKitWebView*         web_view,
     }
 
     #ifdef G_ENABLE_DEBUG
-    if (debug == 2)
+    if (midori_debug ("adblock:2"))
         g_test_timer_start ();
     #endif
     if (adblock_is_matched (req_uri, page_uri))
@@ -895,7 +892,7 @@ adblock_resource_request_starting_cb (WebKitWebView*         web_view,
         g_object_set_data (G_OBJECT (web_view), "blocked-uris", blocked_uris);
     }
     #ifdef G_ENABLE_DEBUG
-    if (debug == 2)
+    if (midori_debug ("adblock:2"))
         g_debug ("match: %f%s", g_test_timer_elapsed (), "seconds");
     #endif
 
@@ -1587,25 +1584,9 @@ static void
 adblock_activate_cb (MidoriExtension* extension,
                      MidoriApp*       app)
 {
-    #ifdef G_ENABLE_DEBUG
-    const gchar* debug_mode;
-    #endif
     KatzeArray* browsers;
     MidoriBrowser* browser;
 
-    #ifdef G_ENABLE_DEBUG
-    debug_mode = g_getenv ("MIDORI_ADBLOCK");
-    if (debug_mode)
-    {
-        if (*debug_mode == '1')
-            debug = 1;
-        else if (*debug_mode == '2')
-            debug = 2;
-        else
-            debug = 0;
-    }
-    #endif
-
     adblock_reload_rules (extension, FALSE);
 
     browsers = katze_object_get_object (app, "browsers");

diff --git a/katze/katze-http-cookies-sqlite.c b/katze/katze-http-cookies-sqlite.c
index bf4b5036ee5893e0d74acd12e99d87ea63ac4966..10eb3f33196e48c336926f1350119e0823985a98 100644 (file)
--- a/katze/katze-http-cookies-sqlite.c
+++ b/katze/katze-http-cookies-sqlite.c
@@ -207,7 +207,7 @@ katze_http_cookies_sqlite_jar_changed_cb (SoupCookieJar*    jar,
         }
     }
 
-    if (g_getenv ("MIDORI_COOKIES_DEBUG") != NULL)
+    if (!g_strcmp0 (g_getenv ("MIDORI_DEBUG"), "cookies"))
         http_cookies->counter++;
 
     if (old_cookie) {

diff --git a/katze/katze-http-cookies.c b/katze/katze-http-cookies.c
index a25f62a17eec414b3b7fd1efdc0f4017782541e9..1ccb8d9aff64cd2e0f7875392332a0f41423e9f2 100644 (file)
--- a/katze/katze-http-cookies.c
+++ b/katze/katze-http-cookies.c
@@ -211,7 +211,7 @@ katze_http_cookies_update_jar (KatzeHttpCookies* http_cookies)
         goto failed;
     g_free (temporary_filename);
 
-    if (g_getenv ("MIDORI_COOKIES_DEBUG") != NULL)
+    if (!g_strcmp0 (g_getenv ("MIDORI_DEBUG"), "cookies"))
     {
         g_print ("KatzeHttpCookies: %d cookies changed\n", http_cookies->counter);
         http_cookies->counter = 0;
@@ -223,7 +223,7 @@ failed:
         fclose (f);
     g_unlink (temporary_filename);
     g_free (temporary_filename);
-    if (g_getenv ("MIDORI_COOKIES_DEBUG") != NULL)
+    if (!g_strcmp0 (g_getenv ("MIDORI_DEBUG"), "cookies"))
         g_print ("KatzeHttpCookies: Failed to write '%s'\n",
                  http_cookies->filename);
     return FALSE;
@@ -263,7 +263,7 @@ katze_http_cookies_jar_changed_cb (SoupCookieJar*    jar,
         }
     }
 
-    if (g_getenv ("MIDORI_COOKIES_DEBUG") != NULL)
+    if (!g_strcmp0 (g_getenv ("MIDORI_DEBUG"), "cookies"))
         http_cookies->counter++;
 
     if (!http_cookies->timeout && (old_cookie || new_cookie->expires))

diff --git a/midori/main.c b/midori/main.c
index 3044809746b61beb8840e99a98912ec60bc72a5e..46d2f30e8101a71b6decae38d263d671cf0e3d8b 100644 (file)
--- a/midori/main.c
+++ b/midori/main.c
@@ -937,7 +937,7 @@ midori_soup_session_settings_accept_language_cb (SoupSession*       session,
                 soup_uri_set_query (stripped_uri, NULL);
                 stripped_referer = soup_uri_to_string (stripped_uri, FALSE);
                 soup_uri_free (stripped_uri);
-                if (g_getenv ("MIDORI_SOUP_DEBUG"))
+                if (midori_debug ("soup"))
                     g_message ("Referer %s stripped to %s", referer, stripped_referer);
                 soup_message_headers_replace (msg->request_headers, "Referer",
                                               stripped_referer);
@@ -950,11 +950,15 @@ midori_soup_session_settings_accept_language_cb (SoupSession*       session,
 static void
 midori_soup_session_debug (SoupSession* session)
 {
-    const char* soup_debug = g_getenv ("MIDORI_SOUP_DEBUG");
-
-    if (soup_debug)
+    gint soup_debug_level = 0;
+    if (midori_debug ("soup:1"))
+        soup_debug_level = 1;
+    else if (midori_debug ("soup:2"))
+        soup_debug_level = 2;
+    else if (midori_debug ("soup:3"))
+        soup_debug_level = 3;
+    if (soup_debug_level > 0)
     {
-        gint soup_debug_level = atoi (soup_debug);
         SoupLogger* logger = soup_logger_new (soup_debug_level, -1);
         soup_logger_attach (logger, session);
         g_object_unref (logger);
@@ -1327,7 +1331,7 @@ midori_load_extensions (gpointer data)
     gchar** keys = g_object_get_data (G_OBJECT (app), "extensions");
     KatzeArray* extensions;
     #ifdef G_ENABLE_DEBUG
-    gboolean startup_timer = g_getenv ("MIDORI_STARTTIME") != NULL;
+    gboolean startup_timer = midori_debug ("startup");
     GTimer* timer = startup_timer ? g_timer_new () : NULL;
     #endif
 
@@ -1396,7 +1400,7 @@ midori_load_session (gpointer data)
     gint64 current;
     gchar** command = g_object_get_data (G_OBJECT (app), "execute-command");
     #ifdef G_ENABLE_DEBUG
-    gboolean startup_timer = g_getenv ("MIDORI_STARTTIME") != NULL;
+    gboolean startup_timer = midori_debug ("startup");
     GTimer* timer = startup_timer ? g_timer_new () : NULL;
     #endif
 
@@ -1978,7 +1982,7 @@ main (int    argc,
     gint max_history_age;
     gint clear_prefs = MIDORI_CLEAR_NONE;
     #ifdef G_ENABLE_DEBUG
-        gboolean startup_timer = g_getenv ("MIDORI_STARTTIME") != NULL;
+        gboolean startup_timer = midori_debug ("startup");
         #define midori_startup_timer(tmrmsg) if (startup_timer) \
             g_debug (tmrmsg, (g_test_timer_last () - g_test_timer_elapsed ()) * -1)
     #else

diff --git a/midori/midori-app.c b/midori/midori-app.c
index 4e8eb8be84124a0a3ab146e5e77f85259725b772..a7bce86ba85084ba34856362d62d72bd19e19ca5 100644 (file)
--- a/midori/midori-app.c
+++ b/midori/midori-app.c
@@ -1527,3 +1527,37 @@ midori_app_setup (gchar** argument_vector)
     #endif
 }
 
+gboolean
+midori_debug (const gchar* token)
+{
+    static const gchar* debug_token = NULL;
+    g_return_val_if_fail (token != NULL, FALSE);
+    if (debug_token == NULL)
+    {
+        const gchar* debug = g_getenv ("MIDORI_DEBUG");
+        const gchar* debug_tokens = "soup:1 soup:2 soup:3 cookies";
+        const gchar* full_debug_tokens = "adblock:1 adblock:2 startup bookmarks";
+        if (debug && strstr (full_debug_tokens, debug))
+        {
+            #ifdef G_ENABLE_DEBUG
+            debug_token = g_intern_static_string (debug);
+            #else
+            g_warning ("Value '%s' for MIDORI_DEBUG requires a full debugging build.", debug);
+            #endif
+        }
+        else if (debug && strstr (debug_tokens, debug))
+            debug_token = g_intern_static_string (debug);
+        else if (debug)
+            g_warning ("Unrecognized value '%s' for MIDORI_DEBUG.", debug);
+        else
+            debug_token = "NONE";
+        if (!debug_token)
+        {
+            debug_token = "INVALID";
+            g_print ("Supported values: %s\nWith full debugging: %s\n",
+                     debug_tokens, full_debug_tokens);
+        }
+    }
+    return debug_token == g_intern_static_string (token);
+}
+

diff --git a/midori/midori-app.h b/midori/midori-app.h
index 5a4b9b35a671c277d331863b534b7f34aca18461..a8fae5eb9887d2b6be43dbec2e38490cc7764601 100644 (file)
--- a/midori/midori-app.h
+++ b/midori/midori-app.h
@@ -94,6 +94,9 @@ midori_app_find_res_filename      (const gchar* filename);
 gchar*
 midori_app_get_lib_path           (const gchar* package);
 
+gboolean
+midori_debug                      (const gchar*       token);
+
 G_END_DECLS
 
 #endif /* __MIDORI_APP_H__ */

diff --git a/midori/midori-bookmarks.c b/midori/midori-bookmarks.c
index 6e694b6caa5238fbdacf403e08fe4a7cb72bac5c..035c36a3cda9f717bc845e04db52afaaa841a99a 100644 (file)
--- a/midori/midori-bookmarks.c
+++ b/midori/midori-bookmarks.c
@@ -160,7 +160,7 @@ midori_bookmarks_initialize (KatzeArray*  array,
     }
 
 #ifdef G_ENABLE_DEBUG
-    if (g_getenv ("MIDORI_BOOKMARKS_DEBUG"))
+    if (midori_debug ("bookmarks")
         sqlite3_trace (db, midori_bookmarks_dbtracer, NULL);
 #endif
 

diff --git a/midori/midori-view.c b/midori/midori-view.c
index e14a61ab4597ba7cd93e4cae649fd90eac33c686..f434566279e58b8dcb2f779fed52442a8657b107 100644 (file)
--- a/midori/midori-view.c
+++ b/midori/midori-view.c
@@ -4232,7 +4232,7 @@ midori_view_set_uri (MidoriView*  view,
             #ifdef G_ENABLE_DEBUG
             GTimer* timer = NULL;
 
-            if (g_getenv ("MIDORI_STARTTIME") != NULL)
+            if (midori_debug ("startup")
                 timer = g_timer_new ();
             #endif
 
@@ -4247,7 +4247,7 @@ midori_view_set_uri (MidoriView*  view,
                 speeddial_markup ? speeddial_markup : "", "about:blank", NULL);
 
             #ifdef G_ENABLE_DEBUG
-            if (g_getenv ("MIDORI_STARTTIME") != NULL)
+            if (midori_debug ("startup")
             {
                 g_debug ("Speed Dial: \t%fs", g_timer_elapsed (timer, NULL));
                 g_timer_destroy (timer);