Closed Bug 1518548 Opened 10 months ago Closed 9 months ago

Remove the sync telemetry histogram

Categories

(Firefox :: Search, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 67
Tracking Status
firefox67 --- fixed

People

(Reporter: mikedeboer, Assigned: manishkk)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

During the review phase of bug 1492475, we noticed that the 'SEARCH_SERVICE_INIT_SYNC' Telemetry Histogram doesn't really serve a purpose anymore, because we simply removed the synchronous initialization path!

Assignee: nobody → 1991manish.kumar
Flags: needinfo?(mdeboer)
Keywords: checkin-needed

We're sorry, Autoland could not rebase your commits for you automatically.
Please manually rebase your commits and try again.
Hunk #1 FAILED at 2733
Hunk #2 succeeded at 2683 with fuzz 2 (offset -95 lines).
1 out of 2 hunks FAILED -- saving rejects to file toolkit/components/search/nsSearchService.js.rej

Flags: needinfo?(1991manish.kumar)
Keywords: checkin-needed
Attachment #9039637 - Attachment description: Bug 1518548 Remove the sync telemetry histogram → Bug 1518548 - Remove the sync telemetry histogram
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/1e95c3246660
Remove the sync telemetry histogram r=mikedeboer
Flags: needinfo?(1991manish.kumar)
Keywords: checkin-needed

Hi Manish, does this still need to be landed after c#4? If that's the case the the modifications made need to be review by mikedeboer in order to be landable via Phabricator. Thanks.

Flags: needinfo?(1991manish.kumar)
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
Flags: needinfo?(1991manish.kumar)
You need to log in before you can comment on or make changes to this bug.