Bug 30458

Summary:

Implement <a ping>

Product:

WebKit

Reporter:

John Abd-El-Malek <jam>

Component:

WebCore Misc.

Assignee:

Nate Chapin <japhet>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, commit-queue, darin, emacemac7, eric, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
ping support (disabled by default)	fishd: review+

John Abd-El-Malek

Reported 2009-10-16 14:13:54 PDT

Creating a bug from the webkit-dev discussion. It seems there's consensus on doing this. The main reasons are that without a robust way to handle tracking, ad networks and other sites are resorting to redirects and busy loops in unload handlers after image loads. This hurts the user since it slows down their navigation and consumes battery on mobile/laptop devices. Adding ping doesn't reduce the privacy, since it's something that's already done using hacks. This should improve privacy since user agents can then expose options to the user to disable this feature, just like for referral headers. This is part of HTML5: http://www.whatwg.org/specs/web-apps/current-work/#hyperlink-auditing Note: once we do this, there are a few people who think we should implement window.sendPing() as well. The reasons are that <a ping> won't work for iframes, which is how it's used by ad networks.

Attachments
ping support (disabled by default) (39.26 KB, patch) 2010-09-22 11:52 PDT, Nate Chapin	fishd: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

John Abd-El-Malek

Comment 1 2009-10-16 14:52:56 PDT

The webkit-dev thread: https://lists.webkit.org/pipermail/webkit-dev/2009-September/thread.html#9925

Nate Chapin

Comment 2 2010-09-22 11:52:40 PDT

Created attachment 68406 [details] ping support (disabled by default)

Darin Fisher (:fishd, Google)

Comment 3 2010-09-22 12:40:55 PDT

Comment on attachment 68406 [details] ping support (disabled by default) r=me

Nate Chapin

Comment 4 2010-09-22 16:51:44 PDT

Comment on attachment 68406 [details] ping support (disabled by default) Clearing cq+ because it won't land before I leave for the day. Will land tomorrow.

Alexey Proskuryakov

Comment 5 2010-09-22 17:33:08 PDT

Comment on attachment 68406 [details] ping support (disabled by default) + * WebKit.order: You needn't modify this. Only Apple uses this file, and we update it when we need to.

Nate Chapin

Comment 6 2010-09-23 12:55:18 PDT

http://trac.webkit.org/changeset/68166

WebKit Review Bot

Comment 7 2010-09-23 13:17:13 PDT

http://trac.webkit.org/changeset/68166 might have broken GTK Linux 32-bit Debug The following changes are on the blame list: http://trac.webkit.org/changeset/68163 http://trac.webkit.org/changeset/68164 http://trac.webkit.org/changeset/68165 http://trac.webkit.org/changeset/68166 http://trac.webkit.org/changeset/68167 http://trac.webkit.org/changeset/68168 http://trac.webkit.org/changeset/68169

Note You need to log in before you can comment on or make changes to this bug.