LocationObserverModule will Fail Thread Checks

[NickGerleman]

This uses react::uwp::BatchingMessageQueue created on the UI thread, and will fail thread checks as BatchingMessageQueue is itself weirdly thread-affined. We also seem to be using it like it should be the NM thread, even though it's not. Likely an artifact of the MSRN migration.

[NickGerleman]

There's a fun comment at the bottom here

  modules.emplace_back(
      LocationObserverModule::name,
      [messageQueue]() { return std::make_unique<LocationObserverModule>(messageQueue); },
      MakeSerialQueueThread()); // TODO: figure out threading

[chrisglein]

Duplicate of #6038