Closed Bug 1234729 Opened 8 years ago Closed 8 years ago

Add telemetry probe for when we load a page from cache

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox46 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 46
Tracking Flags:
Tracking Status
firefox46 --- fixed

People

(Reporter: mfinkle, Assigned: mfinkle)

Details

Attachments

(1 file)

usecache-telemetry v0.1
1.15 KB, patch
sebastian
: review+
Margaret
: review+
Details | Diff | Splinter Review 
Simple probe to track the snackbar/toast displayed when we load a page from the cache. I decided to not use "loadurl.1" because we already triggered a probe for that somewhere else in the app and I don't want to double count. We just need to note that the page was loaded from cache and we already use the "neterror.1" Event for other offline probes.

r? sebastian
p? margaret (data collection probe)
Attachment #8701295 - Flags: review?(s.kaspari)
Attachment #8701295 - Flags: review?(margaret.leibovic)
Attachment #8701295 - Flags: review?(s.kaspari) → review+
Assignee: nobody → mark.finkle
Comment on attachment 8701295 [details] [diff] [review]
usecache-telemetry v0.1

Review of attachment 8701295 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #8701295 - Flags: review?(margaret.leibovic) → review+
https://hg.mozilla.org/mozilla-central/rev/869deb549119
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.