1486799 - Refactor highlighters to use fronts methods for show/hide/destroy

Status: NEW

(DevTools :: Inspector, enhancement, P4)

Description

[Yulia Startsev [:yulia]]

in https://bugzilla.mozilla.org/show_bug.cgi?id=1447490 we introduced memoization directly on the inspector front for the highlighters. This makes more sense than the local memoization we have been doing throughout the codebase:

https://searchfox.org/mozilla-central/search?q=getHighlighterByType(&case=false&regexp=false&path=

And we may be able to drop this function: https://searchfox.org/mozilla-central/source/devtools/client/inspector/shared/highlighters-overlay.js#603-632 

It may be that highlighterOverlay was taking over a lot of tasks that should have gone to fronts.