GitLab

In CL:2350026, I tried to be clever and eliminate the unused
FidoDiscoveryFactory argument from SetPINHandler. However, its base
FidoRequestHandlerBase ctor expects a pointer to the discovery factory
and is run before initialization of the SetPINHandler data members, and
so we now pass it the address of an uninitialized member.  Instead,
inject the FidoDiscoveryFactory from the outside, like before.

Bug: 1122892
Change-Id: I8ea881c57c9dc2b3c33c1bda2f0512459f3a7c47
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2384133Reviewed-by: Adam Langley <agl@chromium.org>
Commit-Queue: Martin Kreichgauer <martinkr@google.com>
Cr-Commit-Position: refs/heads/master@{#803148}