Replace use of EXIT_* constants to 0 and 1

Fix WM_CLASS and WM_WINDOW_ROLE properties

Instead of forcing class and instance names, which is what GTK does by
default anyway, allow the user to set the instance name, but keep the
general class as “Surf”.
Is we won't support a “-name” parameter and don't implement a parsing of
RESOURCE_NAME env variable, let's fallback on the third behaviour, use
the name of the calling executable.
That would let the user do things like 'ln -s ./surf ./surf-other;
./surf-other' and set different parameters for the two instances (in dwm
for example).
Try to generate a unique WM_WINDOW_ROLE (within the surf process)
composed of “Surf” and the view ID.

Remove window hints settings

Setting minimum size to 1x1 is not necessary as this is the default.

Style indentation fix

Continue making chars const

Correct and simplify buttonreleased() gdk event

The passed event is a GdkEvent (or GdkEventButton), but not a
GdkEventKey.
Access the struct directly.

Un(g)boolify to separate GTK dependant code from the rest

Use char instead of gchar and make them const when possible

Style update for indentation

Add mouse function to play medias in external player

Control + left click launches mpv with the target media url.

Update credits

Style change in includes order

Style change in functions ordering

Try to group and order functions in a logical manner.
Same thing for config keybindings.

Cleanup remaining unused variables and copystr()

Adapt newwindow()

Reorder cmdline options alphabetically and try to keep settings as they
have been toggled in this surf client.

Adapt user agent handling

Now we can only append Surf version to the default WebKit user agent
instead of setting the whole string (while this remains possible to do).

Adapt updatetitle()

Now always show loading progress, still print mouse over link uri before
page uri. Process simplyfied because we use const strings.

Adapt toggle statistics

Rename *togglestat to plural *togglestats, add frame flatenning
indicator, resize array in consequence.
Use a static index instead of a dynamic one as we always use all values
anyway.

Adapt page statistics

Rename *pagestat to plural *pagestats, and resize array to reflect
missing proxy parameter.

Adapt loaduri()

Better handling of different URIs. Filter out “about:” scheme, dont
touch URI if it contains a complete scheme (we assume "://", denotes
one), else test if given path is an actual reachable file on the
filesystem, else prepend arbitrary http:// scheme.

Adapt config MACROs SETPROP, DOWNLOAD, PLUMB

As we now use directly const strings from WebKitGTK, “constify” arg void
pointer.

Rename mouse functions to clickFUNCTION()

Adapt toggle(), clean some config parameters

Regroup all toggles in an enum and handle them with a unique function
via a switch. That lets us take different actions for each toggle.
Add a frame flatenning and a dns preteching options.

Adapt find()

Slightly new behaviour: searching again for the same string (via MOD+/)
resets the search (ie restarts search from document top).
Searching for an empty string stops the search (ie all highlights are
removed).

Adapt zoom()

Adapt navigate()

Remove togglescrollbars()

We do not have access to scrollbars and will have to manipulate DOM to
do that.

Adapt scrolling

In fact, we have a scrolling handle ersatz for now using JavaScript
calls as we don't have access anymore to scrollbars.
We'll have to manipulate the DOM directly (later).

Adapt inspector handling

The inspector is now easily manageable via API, there's no need for
keeping its state in the Client.

Adapt print()

Simplify building of stylefiles path

We can directly use style file pointers instead of using temporary
string pointers.

Adapt setstyle()

We can't anymore pass a simple path to a stylefile, we have to read the
entire file to create a stylesheet and apply it to the view.

Adapt clipboard()

Replace linkhover with target uri and add some comments

Rename fullscreen() to togglefullscreen()

And handle c->fullscreen value in winevent(). This way we keep track of
fullscreen state even if we did not directly initiate the fullscreen.

Replace titlechangeleave() with catch-all winevent()

All GtkWidget events have the same function prototype with the generic
GdkEvent as parameter. This will let us handle everything in the switch.

Adapt JavaScript functions

Removed eval(), evalscript() is now sufficient and can directly execute
passed JavaScript string.

Remove windowobjectcleared(), execute scripts on load finished

The “window-object-cleared” signal is not directly accessible for
specific WebViews now. We'll have to use an Extension or rewrite the
JavaScript running functions.

Add createwindow() for GtkWindow creation

Move there window creation from showview().

Add closeview() for JavaScript window closing

Simply call gtk_widget_destroy() on the window. Then GtkWindow closing
process will take care of the rest.

Remove (delete :>) deletion_interface()

Integrate beforerequest() into decidepolicy()

Remove contextmenu() and menuactivate() for "context-menu" signal

Those were added because “right click menu to copy the link URI will now
work” (would not work) in 2013. It's been a while since that works without
intervention.

Adapted buttonrelease()

Use the current hit test (c->mousepos) to determine where the mouse
pointer is.
It is possible to link an action to a click and still propagate the
event after that by setting the “stop event” parameter of a Button to 0.

Replace initdownload() and intercept global download requests.

Attach to download requests on WebKitContext, this way we can easily
manage all types of downloads (coming from views or not).

Adapted progresschanged()

Manage ssl errors with GTlsCertificateFlags

Keep a GTlsCertificateFlags in Client for TLS errors.
If we don't use a ssl connection, set it to a value greater than maximum
error flag value (which is G_TLS_CERTIFICATE_VALIDATE_ALL for all
possible errors).

Replace loadstatuschanged() with loadchanged()

Replace decidewindow() and decidedownload() with decidepolicy()

Now all the requests are triggered by one signal.
We then handle each type, resource download, navigation, newwindow, in
separate functions.

Replace createwindow() with createview()

We can now perform some filtering before opening a new view (window)
when requested by JavaScript (either random or by a click).

Replace geopolicyrequested() with permissionrequested()

The permission-request signal can now ask for than a geolocation
authorisation (though we only handle that for now).

Replace linkhover() with mousetargetchanged()

The “linkhover” can now be more than a simple link (image, video, etc.).
As we can't anymore perform a hit test when we want, we have to keep the
last known hit test to be able to know where the mouse is on the next
click event.

Modify the context name of the hit tests

These relate more to the position of the pointer when an event occurs.

Adapted titlechange()

Do not copy title strings, just carry over the const pointer.

Move all necessary initialisation to newview()

Most all the settings and callbacks are set before view creation.
Create a related view (with inherited settings) if asked to.

Add newview() and forward related views

View creation is now done in a separate function.
That helps identifying the workflow and let us easily carry related
view (client) when asked by the WebKit for a new one.

Add showview()

Only show the window when and if the view is ready.
Move all the window rendering there.

Separate the rendering engine process

Use one WebKitWebProcess per view instead of one for all views.
More information in the API documentation.

No option to enforce 96DPI anymore on WebKit2

Scrolling is now handled through WebKit

We don't have access anymore to the scrolling process and have to rely
entirely on WebKit.
We could be able to manage scrolling and bars directly by manipulating
the DOM via a WebKitExtension. To be continued…

SSL policy is now handled through WebKit

Disk cache is now handled through WebKit

Cookies are now handled through WebKit

Remove all soup cookie handling and use provided interface.

Remove source(), it has been removed from webkit2gtk

WebKit2GTK doesn't provide a direct way to view source anymore, we'll
have to do that ourselves with a GtkSourceView if we still need that.

Remove proxy stuff, it's now handled by WebView

Switch headers to webkit2

Port surf to gtk3

buildfile: don't forget to free temporary string bpath

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Be more specific about what enablestyles parameter does.

The enablestyles configuration variable and parameter flag manages the
global application of styles, not just the site-specific styles.

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Fix inspector handling when inspector is disabled.

Do not try to show the inspector window when the inspector is disabled.

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Fix usage(), -m and -M for styles were absent.

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Fix fullscreen handling

Handle the fullscreen in a more consistent way with other options:
-f disables fullscreen while -F enables fullscreen.

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Add setstyle()

Regroup style uri setting application calls under one function.

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Fix style files handling: stop leaking strings.

Everytime getstyle() was being called, we returned newly allocated
strings without ever freing them.
Now uri stylefiles only get allocated once at setup().

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Style changes in config.def.h.

Continue style fixing: function declarations, code alignement

Here are the changes we discussed on the mailing-list.
Alignement is pushed to the first parameter of the function, which seems
to be what people agrees on.

Signed-off-by: Christoph Lohmann <20h@r-36.net>

style fixes: space after keywords, () with sizeof

Signed-off-by: Christoph Lohmann <20h@r-36.net>

whitespace fixes

Signed-off-by: Christoph Lohmann <20h@r-36.net>

fix style path generation

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Style cleanup.

Fix the way system files are handled at surf startup.

1. Do not chmod existing directories.
2. Fix the handling of tilde expansion in paths, don't expand ~foo to
   $HOME/foo but to foo's home directory.
3. Separate the creation of files and directories. We don't have to
   worry anymore about pathnames having to end with a '/' to be correctly
   handled.

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Fix atom value parsing in SETPROP

xprop(1) encloses the returned atom string value in double quotes while
it doesn't when the value is unset. Original simple parsing would fail
and parse the atom name instead of getting an empty value.

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Remove the pax intermediary.

Compile webkit without jit and you don't need to disable mprotect.

Fixing the paxctl check.

Thanks quinq for the tip.

Add PaX support.

Mprotect needs to be turned off for Webkit.

Torified surf is unstable

On Tue, Aug 18, 2015 at 12:38:40PM +0200, Gabriel Pérez-Cerezo wrote:
> It really seems  to be a problem with torsocks.  I have already solved
> this problem with a patch GhostAV just  sent me on this list that adds
> SOCKS support to surf. It works fine now.

As i failed to send the patch to the list, i resend it now:
-- >8 --
From: Jonas Rabenstein <jonas.rabenstein@studium.uni-erlangen.de>
Subject: [PATCH] allow sock proxies

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Fix type of strictssl property.

Thanks to Mark Edgar <medgar123@gmail.com> for having spotted this.

Removing the debug print from the last commit.

allow buttonrelease customization in config.h

Signed-off-by: Christoph Lohmann <20h@r-36.net>

surf: see hover URL without changing title

On Tue, Jan 27, 2015, at 11:52 AM, Christoph Lohmann wrote:
> Then hook to the window leaving event in GTK or X11 and set the title to
> your needs. I will welcome a patch.

Patch is attached.  I am a total novice in GTK programming, so I don't
know what type the callback function is supposed to be (hence the void
pointers) or whether I registered it properly.  But it does work well
for me.

--
http://www.fastmail.com - Access your email from home and the web

From a33f06da092bf920b6a286ea7688b32944d79a50 Mon Sep 17 00:00:00 2001
From: Greg Reagle <greg.reagle@umbc.edu>
Date: Tue, 27 Jan 2015 13:22:15 -0500
Subject: [PATCH] ensure that window title is web page title (not hover link)
 when leaving window

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Some cleanup in style.

Describe the web page indicators too.

It wasn't really clear what was meant with site indicators in the manpage.

surf: documented indicators in man page

Patch attached.

--
http://www.fastmail.com - The professional email service

From 4a64624fee94c16c093d2b192389bad9bd6675d8 Mon Sep 17 00:00:00 2001
From: Greg Reagle <greg.reagle@umbc.edu>
Date: Mon, 26 Jan 2015 14:29:05 -0500
Subject: [PATCH] Documented indicators

Signed-off-by: Christoph Lohmann <20h@r-36.net>

fix stylesheet interna.

* no more segfault when running `surf -m`
* allow to enable custom styles after `surf -m` with mod+shift+m
* use enablestyles instead of the webkit-setting, which clears things up a bit

Signed-off-by: Christoph Lohmann <20h@r-36.net>

Only plumb some URI, when it's ASCII.

Oh my blob!

Adding blob: to the URIS handled by the browser.

Data: is part of the browser too.

file:// should be handled in surf too.

Remove the debugging from the testing.

Add some description for the plumb feature.