2007-10-19 Eric Seidel <eric@webkit.org>
authoreseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Oct 2007 08:58:14 +0000 (08:58 +0000)
committereseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Oct 2007 08:58:14 +0000 (08:58 +0000)
        Reviewed by Mark Rowe.

        Make the GtkLauncher code slightly more readable.

        * GtkLauncher/main.cpp:
        (setupMainMenu): Added.
        (setupMainWindowUI): Added.
        (main):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@26767 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKitTools/ChangeLog
WebKitTools/GtkLauncher/main.cpp

index 1088bbba8d2ea242b9ba62cfadfc5380a7593cc2..c975bbcd0f4d827778fee8920ed1544eb084852e 100644 (file)
@@ -1,3 +1,14 @@
+2007-10-19  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Mark Rowe.
+        
+        Make the GtkLauncher code slightly more readable.
+
+        * GtkLauncher/main.cpp:
+        (setupMainMenu): Added.
+        (setupMainWindowUI): Added.
+        (main):
+
 2007-10-19  Maciej Stachowiak  <mjs@apple.com>
 
         Rubber stamped by Adam.
index e7b8f42431b1d0e38b1b24a6de69741460827238..f0d072e116dfe1387e5af67ae1e11e8f77f965a3 100644 (file)
@@ -13,11 +13,6 @@ static gchar* gTitle;
 static gint gProgress;
 static guint gStatusbarContextId;
 
-static bool stringIsEqual(const char* str1, const char* str2)
-{
-    return 0 == strcmp(str1, str2);
-}
-
 static gchar* autocorrectURL(const gchar* url)
 {
     if (strncmp("http://", url, 7) != 0 && strncmp("https://", url, 8) != 0 && strncmp("file://", url, 7) != 0 && strncmp("ftp://", url, 6) != 0) {
@@ -105,34 +100,8 @@ static void menuMainQuitCallback(gpointer)
     gtk_main_quit();
 }
 
-int main(int argc, char* argv[]) 
+static GtkWidget* setupMainMenu()
 {
-    gtk_init(&argc, &argv);
-    webkit_init();
-
-    gchar* url = "http://www.google.com";
-    bool exitAfterLoading = false;
-    bool dumpRenderTree = false;
-    for (int argPos = 1; argPos < argc; ++argPos) {
-        char *currArg = argv[argPos];
-        if (stringIsEqual(currArg, "-exit-after-loading"))
-            exitAfterLoading = true;
-        else if (stringIsEqual(currArg, "-exitafterloading"))
-            exitAfterLoading = true;
-        else if (stringIsEqual(currArg, "-exitafterload"))
-            exitAfterLoading = true;
-        else if (stringIsEqual(currArg, "-exit-after-load"))
-            exitAfterLoading = true;
-        else if (stringIsEqual(currArg, "-drt"))
-            dumpRenderTree = true;
-        else if (stringIsEqual(currArg, "-dump-render-tree"))
-            dumpRenderTree = true;
-        else if (stringIsEqual(currArg, "-dumprendertree"))
-            dumpRenderTree = true;
-        else
-            url = autocorrectURL(currArg);
-    }
-
     GtkWidget* menuMain = gtk_menu_new();
     GtkWidget* menuMainBack = gtk_menu_item_new_with_label("Back");
     gtk_menu_shell_append(GTK_MENU_SHELL(menuMain), menuMainBack);
@@ -152,6 +121,13 @@ int main(int argc, char* argv[])
     GtkWidget* menuMainRoot = gtk_menu_item_new_with_label("Main");
     gtk_widget_show(menuMainRoot);
     gtk_menu_item_set_submenu(GTK_MENU_ITEM(menuMainRoot), menuMain);
+    
+    return menuMainRoot;
+}
+
+static void setupMainWindowUI()
+{
+    GtkWidget* menuMainRoot = setupMainMenu();
 
     GtkWidget* menuBar = gtk_menu_bar_new();
     gtk_menu_shell_append(GTK_MENU_SHELL(menuBar), menuMainRoot);
@@ -179,20 +155,34 @@ int main(int argc, char* argv[])
     GtkWidget* scrolledWindow = gtk_scrolled_window_new(NULL,NULL);
     gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(scrolledWindow),
                                    GTK_POLICY_AUTOMATIC, GTK_POLICY_AUTOMATIC);
+    
+    
     gPage = WEBKIT_PAGE(webkit_page_new());
     gtk_container_add(GTK_CONTAINER(scrolledWindow), GTK_WIDGET(gPage));
     gtk_box_pack_start(GTK_BOX(vbox), scrolledWindow, TRUE, TRUE, 0);
 
     gStatusbar = GTK_STATUSBAR(gtk_statusbar_new());
     gStatusbarContextId = gtk_statusbar_get_context_id(gStatusbar, "Link Hover");
-    gtk_box_pack_start(GTK_BOX(vbox), GTK_WIDGET(gStatusbar), FALSE, FALSE, 0); 
+    gtk_box_pack_start(GTK_BOX(vbox), GTK_WIDGET(gStatusbar), FALSE, FALSE, 0);
     
     g_signal_connect(gPage, "title-changed", G_CALLBACK(titleChanged), gPage);
     g_signal_connect(gPage, "load-progress-changed", G_CALLBACK(progressChanged), gPage);
     g_signal_connect(gPage, "hovering-over-link", G_CALLBACK(hoveringOverLink), gPage);
+}
+
+int main(int argc, char* argv[]) 
+{
+    gtk_init(&argc, &argv);
+    webkit_init();
+
+    gchar* url = autocorrectURL((argc > 1) ? argv[1] : "http://www.google.com");
+    
+    setupMainWindowUI();
+
     webkit_page_open(gPage, url);
 
     gtk_widget_show_all(gTopLevelWindow);
+    g_free(url);
     gtk_main();
     return 0;
 }