654444 - Only include URL of active tab in content crash reports

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[(not currently active) Ted Mielczarek]

+++ This bug was initially created as a clone of Bug #654312 +++

The patch in bug 654312 sends the URLs of all tabs with content crash reports. This is bad, and we shouldn't do this.

a) We explicitly rejected this as too invasive of users' privacy when we designed desktop crash reporting. Yes, it might be useful, but we have to respect our users and this is just too much information.
b) Our infrastructure only expects a single URL in the URL field anyway, so topcrash by URL reports etc won't work properly with this data.

We should just send the URL of the active tab. On desktop we try to be smart about load events etc, and save the most recently active tab, or something like that. It's always going to be imprecise, but it's the right thing to do.

Comment 1

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 529722 [details] [diff] [review]
patch

I don't like accessing __SS_data, but we need to refactor the nsISessionStroe interface to fix that. Of course, this code should be removed as soon as the crashreporter annotations work for the child crashes.

Comment 3

[Brad Lassey [:blassey] (use needinfo?)]

pushed http://hg.mozilla.org/mozilla-central/rev/0a0081de66e3

Comment 4

[Anna (Waverley)]

Can someone please verify this bug ?