[Maps] EMS baselayer should load cleanly

[thomasneirynck]

Summary

Closes #88515

This bug was "revealed" by #87310.

vector_tile_layer was loading the EMS-meta twice in short succession due to not correctly setting the metadata of the request. This did not cause end-user errors, just a double refresh cycle internally.

By introducing #87310, the second refresh would throw an error, causing the layer-error to show up in the TOC.

The fix is that vector_tile_layer should correctly set the metadata about the request. This way, the canSkip logic behaves as expected and there is no 2nd redundant refresh.

Checklist

Delete any items that are not applicable to this PR.

• Unit or functional tests were updated or added to match the most common scenarios

For maintainers

• This was checked for breaking API changes and was labeled appropriately

[elasticmachine]

Pinging @elastic/kibana-gis (Team:Geo)

[nickpeihl]

lgtm.

tested in Chrome

[nreese]

LGTM - thanks for fixing this
code review

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: bd1492d

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
maps	2.9MB	2.9MB	-36.0B

History

• 💔 Build #99654 failed 7b1b57e
• 💔 Build #99636 failed fa92213
• 💚 Build #99266 succeeded 9398308
• 💔 Build #99255 failed fcdacab

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream