<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/107867/">http://git.reviewboard.kde.org/r/107867/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On December 23rd, 2012, 12:57 p.m., <b>Anders Lund</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Wouldn't it be better to improve the userscripts plugin for KHTML? I have auserscript that removes the google tracking URLS in khtml, and there are probably similar scripts eg for facebook and apart from that a lot of other usefull scripts in userscripts.org.
I do not understand the rationale behind targeting one specific website this way! Just my 2c :)</pre>
</blockquote>
<p>On December 23rd, 2012, 2:35 p.m., <b>Thomas Fischer</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">> userscripts plugin for KHTML
Do you mean this one here?
http://kde-apps.org/content/show.php?content=140676
It says it is no longer maintained. I will have a look ...
My code is fairly simple and more likely (I assume) to get accepted than a "large" solution like userscript.
> rationale behind targeting one specific website this way
It was my itch to scratch. Google is just the start.
As I stated in the code as a TODO comment: more cases to add!
</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hello Anders, your comment on my posting two days ago isn't here, but I'll answer it here. I agree that hardcoding replacement patterns is inflexible regarding future changes on Google's (or anyone else's) homepage.
I already fetched the latest sources from KHTML-userscript and imported them into my git scratch. I'll have a closer look during the next few days.</pre>
<br />
<p>- Thomas</p>
<br />
<p>On December 23rd, 2012, 11:09 a.m., Thomas Fischer wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kdelibs.</div>
<div>By Thomas Fischer.</div>
<p style="color: grey;"><i>Updated Dec. 23, 2012, 11:09 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This patch adds the feature to KHTML to rewrite URLs that are used to track users. Right now, only tracking URLs from Google's search result are supported, but the list can be expanded (hard-coded right now).
Example: A search for "KDE" may result in a list of links, including a link like
http://www.google.com/url?q=http://www.kde.org/&sa=U&ei=YsYFfgOqAZzBQBC&ved=GEFANYNoNG&usg=Y8BfN6qj0QYNHYJQQBEB
When you follow this link, Google will transparently redirect you to http://www.kde.org, but still record your behaviour.
The patch rewrites such links already in the HTML parsing phase, i.e. you never see the tracking URL, but instead the final URL only.
The rewrite feature can be disabled through a setting, but there is no GUI for that yet.
I was thinking about automatically detecting tracking URLs through a regular expression, but I guess running a regular expression check for every URL would be too time-consuming.
I wrote the patch for 4.9.3 as this is the version I am using on the testing machine. I assume the affected classes haven't changed much in recent months, so it should be fairly simple to port to HEAD or future 4.11.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>khtml/khtml_settings.h <span style="color: grey">(0faec6d)</span></li>
<li>khtml/khtml_settings.cpp <span style="color: grey">(b5693b4)</span></li>
<li>khtml/xml/dom_docimpl.cpp <span style="color: grey">(bb65a89)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107867/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>