Closed Bug 1350724 Opened 6 years ago Closed 6 years ago

Remove code for recording tab cache position (bug 1242013)

Categories

(Firefox :: Tabbed Browser, enhancement)

Product:
Firefox
Component:
Tabbed Browser
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: billm, Assigned: billm)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

patch
5.86 KB, patch
mconley
: review+
Details | Diff | Splinter Review
Attached patch patch — — Splinter Review 
We're not using this anymore and bug 1349822 shows that it's kinda slow.
Attachment #8851385 - Flags: review?(mconley)
Comment on attachment 8851385 [details] [diff] [review]
patch

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

Thanks billm.

Noticed that there's a reference to _recordTabAccess in devtools/client/responsive.html/docs/browser-swap.md that we should remove while we're at it.
Attachment #8851385 - Flags: review?(mconley) → review+
billm, should we set checkin-needed here?
Flags: needinfo?(wmccloskey)
Pushed by wmccloskey@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5fe4faa141d4
Remove telemetry for tab cache position (r=mconley)
https://hg.mozilla.org/mozilla-central/rev/5fe4faa141d4
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Flags: needinfo?(wmccloskey)
You need to log in before you can comment on or make changes to this bug.