escapeAdminPath sessionStorage should be namespaced

I have attached patch for same, please review

Seems to have some failures.

Tried fixing failures, attached interdiff for same

Seems fine.

Looks like there are couple contrib modules using this session storage entry. Should we deprecate the old key before removing it?

Seems minor but probably the safest route

How can we apply javascript deprecation → on top of this session storage key?

Just a small change.

Deprecation key seems straight forward. But should the message include a link to a CR or mention what should be used instead?

Added the CR.Thanks!

Short and to the point!

Kernel failure appeared to be random, re-ran the tests all are green.

Believe this one is good to go.

CR was a bit too short, added a bit of context :) Since this is sessionStorage there is no need to get rid of the old variable, it will be deleted when the tab closes.

I know it's now in the toolbar module but it's something that is used in gin, will be used by the navigation module eventually, and the toolbar module will be deprecated/removed at some point. So not sure it's necessary to namespace it in the toolbar. We'd need to do another deprecation later and we don't need to go through that I think.

Can we do Drupal.escapeAdminPath instead?