Adapt newwindow()
[surf.git] / surf.c
diff --git a/surf.c b/surf.c
index 1d91a92..7ba892e 100644 (file)
--- a/surf.c
+++ b/surf.c
@@ -107,11 +107,12 @@ static Client *clients = NULL;
 static Window embed = 0;
 static gboolean showxid = FALSE;
 static char winid[64];
-static char togglestat[9];
-static char pagestat[3];
+static char togglestats[10];
+static char pagestats[2];
 static GTlsDatabase *tlsdb;
 static int cookiepolicy;
 static char *stylefile = NULL;
+static const char *useragent;
 
 static void addaccelgroup(Client *c);
 static char *buildfile(const char *path);
@@ -142,8 +143,8 @@ static void togglefullscreen(Client *c, const Arg *a);
 static gboolean permissionrequested(WebKitWebView *v,
                WebKitPermissionRequest *r, Client *c);
 static const char *getatom(Client *c, int a);
-static void gettogglestat(Client *c);
-static void getpagestat(Client *c);
+static void gettogglestats(Client *c);
+static void getpagestats(Client *c);
 static char *geturi(Client *c);
 static const gchar *getstyle(const char *uri);
 static void setstyle(Client *c, const char *stylefile);
@@ -162,7 +163,7 @@ static gboolean keypress(GtkAccelGroup *group, GObject *obj, guint key,
 static void mousetargetchanged(WebKitWebView *v, WebKitHitTestResult *h,
                guint modifiers, Client *c);
 static void loadchanged(WebKitWebView *v, WebKitLoadEvent e, Client *c);
-static void loaduri(Client *c, const Arg *arg);
+static void loaduri(Client *c, const Arg *a);
 static void navigate(Client *c, const Arg *a);
 static void clicknavigate(Client *c, const Arg *a, WebKitHitTestResult *h);
 static Client *newclient(Client *c);
@@ -825,38 +826,35 @@ loadchanged(WebKitWebView *v, WebKitLoadEvent e, Client *c)
 }
 
 void
-loaduri(Client *c, const Arg *arg)
+loaduri(Client *c, const Arg *a)
 {
-       char *u = NULL, *rp;
-       const char *uri = (char *)arg->v;
-       Arg a = { .b = FALSE };
        struct stat st;
+       char *url, *path;
+       const char *uri = (char *)a->v;
 
-       if (strcmp(uri, "") == 0)
+       if (g_strcmp0(uri, "") == 0)
                return;
 
-       /* In case it's a file path. */
-       if (stat(uri, &st) == 0) {
-               rp = realpath(uri, NULL);
-               u = g_strdup_printf("file://%s", rp);
-               free(rp);
+       if (g_strrstr(uri, "://") || g_str_has_prefix(uri, "about:")) {
+               url = g_strdup(uri);
+       } else if (!stat(uri, &st) && (path = realpath(uri, NULL))) {
+               url = g_strdup_printf("file://%s", path);
+               free(path);
        } else {
-               u = g_strrstr(uri, "://") ? g_strdup(uri)
-                   : g_strdup_printf("http://%s", uri);
+               url = g_strdup_printf("http://%s", uri);
        }
 
-       setatom(c, AtomUri, uri);
+       setatom(c, AtomUri, url);
 
-       /* prevents endless loop */
-       if (strcmp(u, geturi(c)) == 0) {
-               reload(c, &a);
+       if (strcmp(url, geturi(c)) == 0) {
+               reload(c, a);
        } else {
-               webkit_web_view_load_uri(c->view, u);
-               c->progress = 0;
-               c->title = copystr(&c->title, u);
+               webkit_web_view_load_uri(c->view, url);
+               c->title = geturi(c);
                updatetitle(c);
        }
-       g_free(u);
+
+       g_free(url);
 }
 
 void
@@ -902,7 +900,6 @@ newview(Client *c, WebKitWebView *rv)
        WebKitSettings *settings;
        WebKitUserContentManager *contentmanager;
        WebKitWebContext *context;
-       char *ua;
 
        /* Webview */
        if (rv) {
@@ -921,12 +918,17 @@ newview(Client *c, WebKitWebView *rv)
                    "enable-javascript", enablescripts,
                    "enable-plugins", enableplugins,
                    NULL);
-               if (!(ua = getenv("SURF_USERAGENT")))
-                       ua = useragent;
-               webkit_settings_set_user_agent(settings, ua);
                /* Have a look at http://webkitgtk.org/reference/webkit2gtk/stable/WebKitSettings.html
                 * for more interesting settings */
 
+               if (strcmp(fulluseragent, "")) {
+                       webkit_settings_set_user_agent(settings, fulluseragent);
+               } else if (surfuseragent) {
+                       webkit_settings_set_user_agent_with_application_details(
+                           settings, "Surf", VERSION);
+               }
+               useragent = webkit_settings_get_user_agent(settings);
+
                contentmanager = webkit_user_content_manager_new();
 
                context = webkit_web_context_new_with_website_data_manager(
@@ -1050,45 +1052,56 @@ showview(WebKitWebView *v, Client *c)
 }
 
 void
-newwindow(Client *c, const Arg *arg, gboolean noembed)
+newwindow(Client *c, const Arg *a, int noembed)
 {
-       guint i = 0;
-       const char *cmd[18], *uri;
-       const Arg a = { .v = (void *)cmd };
+       int i = 0;
        char tmp[64];
+       const char *cmd[26], *uri;
+       const Arg arg = { .v = cmd };
 
        cmd[i++] = argv0;
        cmd[i++] = "-a";
        cmd[i++] = cookiepolicies;
-       if (!enablescrollbars)
-               cmd[i++] = "-b";
+       cmd[i++] = enablescrollbars ? "-B" : "-b";
+       if (cookiefile && g_strcmp0(cookiefile, "")) {
+               cmd[i++] = "-c";
+               cmd[i++] = cookiefile;
+       }
+       cmd[i++] = enablecache ? "-D" : "-d";
        if (embed && !noembed) {
                cmd[i++] = "-e";
-               snprintf(tmp, LENGTH(tmp), "%u", (int)embed);
+               snprintf(tmp, LENGTH(tmp), "%lu", embed);
                cmd[i++] = tmp;
        }
-       if (!allowgeolocation)
-               cmd[i++] = "-g";
-       if (!loadimages)
-               cmd[i++] = "-i";
-       if (kioskmode)
-               cmd[i++] = "-k";
-       if (!enableplugins)
-               cmd[i++] = "-p";
-       if (!enablescripts)
-               cmd[i++] = "-s";
+       cmd[i++] = runinfullscreen ? "-F" : "-f";
+       cmd[i++] = allowgeolocation ? "-G" : "-g";
+       cmd[i++] = loadimages ? "-I" : "-i";
+       cmd[i++] = kioskmode ? "-K" : "-k";
+       cmd[i++] = enablestyle ? "-M" : "-m";
+       cmd[i++] = enableinspector ? "-N" : "-n";
+       cmd[i++] = enableplugins ? "-P" : "-p";
+       if (scriptfile && g_strcmp0(scriptfile, "")) {
+               cmd[i++] = "-r";
+               cmd[i++] = scriptfile;
+       }
+       cmd[i++] = enablescripts ? "-S" : "-s";
+       if (stylefile && g_strcmp0(stylefile, "")) {
+               cmd[i++] = "-t";
+               cmd[i++] = stylefile;
+       }
+       if (fulluseragent && g_strcmp0(fulluseragent, "")) {
+               cmd[i++] = "-u";
+               cmd[i++] = fulluseragent;
+       }
        if (showxid)
                cmd[i++] = "-x";
-       if (enablecache)
-               cmd[i++] = "-D";
-       cmd[i++] = "-c";
-       cmd[i++] = cookiefile;
+       /* do not keep zoom level */
        cmd[i++] = "--";
-       uri = arg->v ? (char *)arg->v : c->linkhover;
-       if (uri)
+       if ((uri = a->v))
                cmd[i++] = uri;
-       cmd[i++] = NULL;
-       spawn(NULL, &a);
+       cmd[i] = NULL;
+
+       spawn(c, &arg);
 }
 
 GtkWidget *
@@ -1392,72 +1405,52 @@ togglestyle(Client *c, const Arg *arg)
 }
 
 void
-gettogglestat(Client *c)
+gettogglestats(Client *c)
 {
-       gboolean value;
-       int p = 0;
-       WebKitWebSettings *settings = webkit_web_view_get_settings(c->view);
-
-       togglestat[p++] = cookiepolicy_set(cookiepolicy_get());
-
-       g_object_get(G_OBJECT(settings), "enable-caret-browsing", &value,
-                    NULL);
-       togglestat[p++] = value? 'C': 'c';
-
-       togglestat[p++] = allowgeolocation? 'G': 'g';
-
-       togglestat[p++] = enablecache? 'D': 'd';
-
-       g_object_get(G_OBJECT(settings), "auto-load-images", &value, NULL);
-       togglestat[p++] = value? 'I': 'i';
-
-       g_object_get(G_OBJECT(settings), "enable-scripts", &value, NULL);
-       togglestat[p++] = value? 'S': 's';
-
-       g_object_get(G_OBJECT(settings), "enable-plugins", &value, NULL);
-       togglestat[p++] = value? 'V': 'v';
-
-       togglestat[p++] = enablestyle ? 'M': 'm';
-
-       togglestat[p] = '\0';
+       togglestats[0] = cookiepolicy_set(cookiepolicy_get());
+       togglestats[1] = enablecaretbrowsing ? 'C' : 'c';
+       togglestats[2] = allowgeolocation ? 'G' : 'g';
+       togglestats[3] = enablecache ? 'D' : 'd';
+       togglestats[4] = loadimages ? 'I' : 'i';
+       togglestats[5] = enablescripts ? 'S': 's';
+       togglestats[6] = enableplugins ? 'V' : 'v';
+       togglestats[7] = enablestyle ? 'M' : 'm';
+       togglestats[8] = enableframeflattening ? 'F' : 'f';
+       togglestats[9] = '\0';
 }
 
 void
-getpagestat(Client *c)
+getpagestats(Client *c)
 {
-       const char *uri = geturi(c);
-
        pagestats[0] = c->tlsflags > G_TLS_CERTIFICATE_VALIDATE_ALL ? '-' :
            c->tlsflags > 0 ? 'U' : 'T';
-       pagestat[1] = '\0';
+       pagestats[1] = '\0';
 }
 
 void
 updatetitle(Client *c)
 {
-       char *t;
+       char *title;
 
        if (showindicators) {
-               gettogglestat(c);
-               getpagestat(c);
-
-               if (c->linkhover) {
-                       t = g_strdup_printf("%s:%s | %s", togglestat, pagestat,
-                                           c->linkhover);
-               } else if (c->progress != 100) {
-                       t = g_strdup_printf("[%i%%] %s:%s | %s", c->progress,
-                                           togglestat, pagestat,
-                                           c->title == NULL ? "" : c->title);
+               gettogglestats(c);
+               getpagestats(c);
+
+               if (c->progress != 100) {
+                       title = g_strdup_printf("[%i%%] %s:%s | %s",
+                           c->progress, togglestats, pagestats,
+                           c->targeturi ? c->targeturi : c->title);
                } else {
-                       t = g_strdup_printf("%s:%s | %s", togglestat, pagestat,
-                                           c->title == NULL ? "" : c->title);
+                       title = g_strdup_printf("%s:%s | %s",
+                           togglestats, pagestats,
+                           c->targeturi ? c->targeturi : c->title);
                }
 
-               gtk_window_set_title(GTK_WINDOW(c->win), t);
-               g_free(t);
+               gtk_window_set_title(GTK_WINDOW(c->win), title);
+               g_free(title);
        } else {
-               gtk_window_set_title(GTK_WINDOW(c->win), (c->title == NULL) ?
-                                    "" : c->title);
+               gtk_window_set_title(GTK_WINDOW(c->win), c->title ?
+                   c->title : "");
        }
 }