Closed
Bug 415168
Opened 17 years ago
Closed 17 years ago
Disable <a ping> for Firefox 3
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla1.9beta3
People
(Reporter: sicking, Assigned: sicking)
Details
Attachments
(1 file)
1.30 KB,
patch
|
vlad
:
review+
vlad
:
superreview+
beltzner
:
approval1.9b3+
|
Details | Diff | Splinter Review |
The spec has changed under us so we're now at a state of no UI, no good proposal for a UI, and no compliant implementation. Let's just disable it and see if we can get a good implementation for next firefox instead.
(note that this change is firefox specific. It's already disabled by default for all other gecko apps, and they will still be able to enable it if they so wish)
Flags: blocking1.9+
Attachment #300754 -
Flags: superreview?(vladimir)
Attachment #300754 -
Flags: review?(vladimir)
Comment on attachment 300754 [details] [diff] [review]
Turn it off
Jonas was supposed to ask beltzner for r/a+, setting approval request here.
Attachment #300754 -
Flags: superreview?(vladimir)
Attachment #300754 -
Flags: superreview+
Attachment #300754 -
Flags: review?(vladimir)
Attachment #300754 -
Flags: review+
Attachment #300754 -
Flags: approval1.9b3?
Updated•17 years ago
|
Target Milestone: --- → mozilla1.9beta3
Comment 2•17 years ago
|
||
Comment on attachment 300754 [details] [diff] [review]
Turn it off
a=beltzner, alas!
Attachment #300754 -
Flags: approval1.9b3? → approval1.9b3+
Assignee | ||
Comment 3•17 years ago
|
||
Checked in
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 4•17 years ago
|
||
This caused the unit test from bug 375314 to fail. sicking said to just remove <a ping> testing from the test, so I did so. If this ever comes back, the test for <a ping> needs to be re-added.
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&subdir=mozilla/content/base/test&command=DIFF_FRAMESET&file=test_bug375314.html&rev1=1.1&rev2=1.2&root=/cvsroot
Comment 5•11 years ago
|
||
Google have recently begun using ping to speed up mobile search results: https://plus.google.com/+IlyaGrigorik/posts/fPJNzUf76Nx
Comment 6•11 years ago
|
||
And also pushing this as a wished-for performance improvement generally, not just on mobile, though it's a win there:
http://calendar.perfplanet.com/2013/browser-wishlist-2013/
I'd assume the argument is that any <a ping= href=...></a> would be replaced by the website that wants to track people following the link with a redirect, and thus there's no true privacy difference here. If anything, ping support is a privacy win (even with no UI) in that if sites start using ping, then the user *can* stop sending them (and a bigger win as ping use spreads and if UI is added). If ping isn't supported and the sites use redirects, the user has no option.
Flags: needinfo?(blassey.bugs)
Assignee | ||
Comment 7•11 years ago
|
||
Me and Doug recently worked with Microsoft and Google to specify a "beacon" spec which should be a more general, easier to implement and less controversial feature.
Spec is here: https://dvcs.w3.org/hg/webperf/raw-file/tip/specs/Beacon/Overview.html
I think Doug is working on implementing, but I don't know if he has a bug yet.
Comment 8•11 years ago
|
||
Comment 9•11 years ago
|
||
(In reply to Randell Jesup [:jesup] from comment #6)
> And also pushing this as a wished-for performance improvement generally, not
> just on mobile, though it's a win there:
> http://calendar.perfplanet.com/2013/browser-wishlist-2013/
>
> I'd assume the argument is that any <a ping= href=...></a> would be replaced
> by the website that wants to track people following the link with a
> redirect, and thus there's no true privacy difference here. If anything,
> ping support is a privacy win (even with no UI) in that if sites start using
> ping, then the user *can* stop sending them (and a bigger win as ping use
> spreads and if UI is added). If ping isn't supported and the sites use
> redirects, the user has no option.
I generally agree that at worst this is a wash privacy-wise.
Doug, care to expand on what Jonas said?
Flags: needinfo?(blassey.bugs) → needinfo?(doug.turner)
Comment 11•11 years ago
|
||
(In reply to Doug Turner (:dougt) from comment #10)
> yeah, we're working on it.
was more looking for a bug number where this is being discussed
Flags: needinfo?(doug.turner)
You need to log in
before you can comment on or make changes to this bug.
Description
•