Open Bug 1311691 Opened 3 years ago Updated 3 years ago
Clean up uses of |_selection
Details/telemetry Search Details | in autocomplete
From bug 1303333 comment 16: > The entire _selectionDetails/telemetrySearchDetails stuff in autocomplete looks hackish to me, I'm not sure why search needs autocomplete to save this info, rather than overriding "handleEnter" and storing this info locally. The original changesets introducing this doesn't make sense to me. > I also don't know why part of the telemetry collection is in handleSearchCommandWhere and part is in doSearch, rather than being well collected together in a method. We should investigate why we're using _selectionDetails/telemetrySearchDetails this way instead of overriding handleEnter and eventually clean up the code, if needed.
You need to log in before you can comment on or make changes to this bug.